Wireshark-commits: [Wireshark-commits] master 23f0e16: Fix IMF Dissector
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 3 Apr 2015 09:10:46 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=23f0e16d0598d70eb9a2492394fabd5370bf9175
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

23f0e16 by Graham Bloice (graham.bloice@xxxxxxxxxxxxx):

    Fix IMF Dissector
    
    The IMF dissector doesn't properly handle the extra CRLF between
    the headers and the body, leaving it as part of the last header,
    and then fails to obtain the body contents due to incorrectly
    specifying the length of the remaining data.
    
    Change-Id: I8a1dfebd90e4cd5249f2244e9374a8e9ed3a25da
    Reviewed-on: https://code.wireshark.org/review/7884
    Reviewed-by: Graham Bloice <graham.bloice@xxxxxxxxxxxxx>
    Petri-Dish: Graham Bloice <graham.bloice@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  a6459ca   3GPP NAS: upgrade dissector to v12.9.0
    adds  23f0e16   Fix IMF Dissector


Summary of changes:
 epan/dissectors/packet-imf.c |   16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)