Wireshark-commits: [Wireshark-commits] master 88364fb: GSM A-bis/OML: fix: do not use tvb_get_ntohl

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 20 Dec 2019 05:21:44 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=88364fb2965e05c3eadb08a9b6a667c0a87313e3
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

88364fb by Vadim Yanitskiy (axilirator@xxxxxxxxx):

    GSM A-bis/OML: fix: do not use tvb_get_ntohl() to fetch IPv4 address
    
    As per doc/README.dissector, the IPv4 addresses shall not be converted
    to host byte order before being passed to proto_tree_add_ipv4(). Due
    to this bug, the IPA Destination IP Address was displayed wrong.
    
    Change-Id: I2dd96590bbd736a8bfe05502d6803813838c6c59
    Bug: 16282
    Reviewed-on: https://code.wireshark.org/review/35501
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  23b1100   h264: fix label for frame_crop_right_offset
     add  88364fb   GSM A-bis/OML: fix: do not use tvb_get_ntohl() to fetch IPv4 address


Summary of changes:
 epan/dissectors/packet-gsm_abis_oml.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)