URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=771b937d75c012f7141a50a532b81da5b33fba36
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark
Commits:
771b937 by Guy Harris (guy@xxxxxxxxxxxx):
Fix handling of headers in body part.
Check whether the unfolded-and-compacted header has only printable
characters, not whether the full header does - the full header may
include LWSP, which includes HT, CR, and LF, none of which are
considered "printable", so valid headers were being treated as not being
headers, causing mis-dissection of some packets.
We don't need to split the header name from the value -
is_known_multipart_header() stops comparison at the end of the header
name.
Change-Id: I96e4ac0b69df726b984ee7faeea19eda18be223c
Reviewed-on: https://code.wireshark.org/review/33417
Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
Petri-Dish: Guy Harris <guy@xxxxxxxxxxxx>
Tested-by: Petri Dish Buildbot
(cherry picked from commit 0efc63140ec2fbbabe4eedfd3d3f588fd13ee9cb)
Reviewed-on: https://code.wireshark.org/review/33418
Actions performed:
from 33c28c5 Clean up indentation.
add 771b937 Fix handling of headers in body part.
Summary of changes:
epan/dissectors/packet-multipart.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)