osmith/wireshark
1
0
Fork 0
forked from osmocom/wireshark
Code Pull requests Releases Wiki Activity

Bluetooth: ATT: Fix not missing break

Browse source 
"Break" is not missing, "error opcode" is part of opcode "Error Response"
that is needed to fetch request data.

Change-Id: I35432b22fae492a93332a8787213dd8fcf796e3b
Reviewed-on: https://code.wireshark.org/review/7001
Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>
This commit is contained in:
Michal Labedzki 2015-02-07 17:24:48 +01:00
parent 1eec03469d
commit 7610a69251
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
epan/dissectors/packet-btatt.c
View file

@ -1496,7 +1496,7 @@ get_request(tvbuff_t *tvb, gint offset, packet_info *pinfo, guint8 opcode,
if (tvb_captured_length_remaining(tvb, offset) < 1)
return NULL;
opcode = tvb_get_guint8(tvb, 1) + 1;
break;
/* No break */
case 0x03: /* Exchange MTU Response */
case 0x05: /* Find Information Response */
case 0x07: /* Find By Type Value Response */
@ -1530,6 +1530,7 @@ get_request(tvbuff_t *tvb, gint offset, packet_info *pinfo, guint8 opcode,
case 0x16: /* Prepare Write Request */
case 0x18: /* Execute Write Request */
case 0x1D: /* Handle Value Indication */
/* This should never happen */
default:
return NULL;
}