diff options
| author | Michal Labedzki <michal.labedzki@tieto.com> | 2015-02-07 17:24:48 +0100 |
|---|---|---|
| committer | Michal Labedzki <michal.labedzki@tieto.com> | 2015-02-07 16:31:51 +0000 |
| commit | 7610a692518421c93829ae73c02ba89b28518eeb (patch) | |
| tree | d5149ddff5d30439518c666e8536c7d1cf5269a9 /epan | |
| parent | 1eec03469da9a70b079dfe900f3ba7e485a2e667 (diff) | |
Bluetooth: ATT: Fix not missing break
"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>
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-btatt.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/epan/dissectors/packet-btatt.c b/epan/dissectors/packet-btatt.c index cd2635401c..eb29656242 100644 --- a/epan/dissectors/packet-btatt.c +++ b/epan/dissectors/packet-btatt.c @@ -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; } |