diff options
| author | Peter Wu <peter@lekensteyn.nl> | 2015-11-20 12:15:36 +0100 |
|---|---|---|
| committer | Anders Broman <a.broman58@gmail.com> | 2015-11-21 08:19:31 +0000 |
| commit | fb11b8a8e2a4fa0c358b1b591cc0b3a9b35c806d (patch) | |
| tree | 5940253267fc24047fcede35360ac36f861851a5 /epan/dissectors/packet-usb-video.c | |
| parent | 8762e7e7380d1b5dc1623e7cb83c0d7256fb7cbe (diff) | |
usbvideo: fix Malformed packet error for SET_CUR
The SET_CUR request does not have an extended pseudo-header, the logic
likely refers to the extra bytes in the usmon packet header. Remove it
since the function handles the payload after that header.
Tested with arkmicro_webcam.pcap (from bug 8414) and
usb-malformed-error.pcapng.gz (from bug 11736).
Bug: 11736
Change-Id: I61c71bb06c37a626260447f703a5cc4db2a6fc80
Reviewed-on: https://code.wireshark.org/review/11990
Reviewed-by: Tim Ansell <mithro@mithis.com>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-usb-video.c')
| -rw-r--r-- | epan/dissectors/packet-usb-video.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/epan/dissectors/packet-usb-video.c b/epan/dissectors/packet-usb-video.c index 3de3926cbe..3f22c29e50 100644 --- a/epan/dissectors/packet-usb-video.c +++ b/epan/dissectors/packet-usb-video.c @@ -1837,10 +1837,6 @@ dissect_usb_vid_get_set(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, proto_tree_add_item(tree, hf_usb_vid_length, tvb, offset, 2, ENC_LITTLE_ENDIAN); offset += 2; - - /* If there is an extended pseudo header, skip over it to reach the payload */ - if ((usb_trans_info->setup.request == USB_SETUP_SET_CUR) && (usb_trans_info->header_type == USB_HEADER_LINUX_64_BYTES)) - offset += 16; } else { |