Wireshark-commits: [Wireshark-commits] master-2.6 5105e76: IP: Make dissection of ip.frag_offset RF
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, 18 Oct 2019 08:18:32 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=5105e76e9dfc822acd8dc8207b95ed9cd5c41b04
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master-2.6
Repository: wireshark

Commits:

5105e76 by Sake Blok (sake@xxxxxxxxxx):

    IP: Make dissection of ip.frag_offset RFC 791 compliant
    
    The v2.5.0rc0-478-g558fe23226, the dissection of ip.frag_offset changed
    to be listed under "Flags", this is not correct. The Fragmentation
    Offset is a separate field according to the RFC. This change corrects
    that behavior. Also, the raw value from the header was shown instead of
    the real byte offset, this is also corrected.
    
    Change-Id: I1d6dfc4314091eb6f3eef418c5a17ed37f7a1200
    Fixes: v2.5.0rc0-478-g558fe23226 ("[IP] Simplify paring of flags field by using proto_tree_add_bitmask_with_flags().")
    Reviewed-on: https://code.wireshark.org/review/33422
    Petri-Dish: Sake Blok <sake.blok@xxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Reviewed-by: Sake Blok <sake.blok@xxxxxxxxxx>
    (backported from commit 237ec1349f869b4a03282168b5c99d1bcce286ac)
    Reviewed-on: https://code.wireshark.org/review/34803
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  26d9d34   wslua: Improve parameter check in ProtoField.new()
     add  5105e76   IP: Make dissection of ip.frag_offset RFC 791 compliant


Summary of changes:
 epan/dissectors/packet-ip.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)