diff options
| author | Claudio Takahasi <claudio.takahasi@openbossa.org> | 2007-11-12 13:17:16 +0000 |
|---|---|---|
| committer | Claudio Takahasi <claudio.takahasi@openbossa.org> | 2007-11-12 13:17:16 +0000 |
| commit | f89c7796c7882c65e829e5203a79606c3d1d63b0 (patch) | |
| tree | 0636d10882316f10fd2b5f4105a6b051014bfbd5 /hcid | |
| parent | 312471773c71bfc1e64d9d6e8a71e933d3b9cc98 (diff) | |
hcid: check the scanned svc record value
Diffstat (limited to 'hcid')
| -rw-r--r-- | hcid/dbus-sdp.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/hcid/dbus-sdp.c b/hcid/dbus-sdp.c index 73dfcbe7..6d7e9b5b 100644 --- a/hcid/dbus-sdp.c +++ b/hcid/dbus-sdp.c @@ -447,7 +447,7 @@ static void remote_svc_rec_completed_cb(uint8_t type, uint16_t err, uint8_t *rsp, size_t size, void *udata) { struct transaction_context *ctxt = udata; - sdp_record_t *rec = NULL; + sdp_record_t *rec; DBusMessage *reply; DBusMessageIter iter, array_iter; const char *src, *dst; @@ -494,8 +494,8 @@ static void remote_svc_rec_completed_cb(uint8_t type, uint16_t err, DBUS_TYPE_BYTE_AS_STRING, &array_iter); rec = sdp_extract_pdu(rsp, &scanned); - if (rec == NULL) { - error("SVC REC is null"); + if (rec == NULL || size != scanned) { + error("Invalid service record!"); goto done; } @@ -519,7 +519,7 @@ static void remote_svc_rec_completed_xml_cb(uint8_t type, uint16_t err, void *udata) { struct transaction_context *ctxt = udata; - sdp_record_t *rec = NULL; + sdp_record_t *rec; DBusMessage *reply; const char *src, *dst; int scanned; @@ -563,8 +563,8 @@ static void remote_svc_rec_completed_xml_cb(uint8_t type, uint16_t err, reply = dbus_message_new_method_return(ctxt->rq); rec = sdp_extract_pdu(rsp, &scanned); - if (rec == NULL) { - error("SVC REC is null"); + if (rec == NULL || size != scanned) { + error("Invalid service record!"); goto done; } |