diff options
author | D. Ulis <daulis0@gmail.com> | 2016-11-04 15:50:25 -0400 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2016-11-15 17:28:50 +0000 |
commit | d84e46ff1004018f88be6011f11f70dce4e3227b (patch) | |
tree | 032674bf48762166bdf03e02e9b4565f91b2a9a0 /wsutil | |
parent | 062378920a2e5f49c1cc43135f13740073d56b5b (diff) |
CIP: Properly handle EPATH attributes that do not specify path size
Most attributes specify the path length before the EPATH data, but some do not. The previous code for parsing EPATHs just looped until there was no more data. This is a problem for EPATH that do not specify a length, because it will eat up too many bytes. This mainly affects Get Attribute List Responses and Set Attribute List Requests. For the small number of attributes like this, the Spec says exactly what kind of segment should be in the EPATH, so just parse a single segment.
This fixes: Port attributes: 'Port Number and Node Address' and 'Chassis Identity'
There is still one that is a problem in packet-cipsafety.c, but we will have to deal with it later. See the TODO in that file.
I use the .pcap attached to https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=12049, to make sure I didn't break any existing path/segment handling.
Ping-Bug: 12049
Change-Id: Id035f9809f6cc747ea7b6327d94dd26dc66cb466
Reviewed-on: https://code.wireshark.org/review/18675
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'wsutil')
0 files changed, 0 insertions, 0 deletions