[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wireshark-commits
Subject:    [Wireshark-commits] master-1.12 ce1819b: LTE-RRC: fix dissection of GPRS container
From:       Wireshark code review <code-review-do-not-reply () wireshark ! org>
Date:       2016-05-31 20:30:40
Message-ID: 20160531203027.22113.71712 () thud ! wireshark ! org
[Download RAW message or body]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ce1819bd5997d0f99ebf9fd1f4e7bf347d6c323d
Submitter: Pascal Quantin (pascal.quantin@gmail.com)
Changed: branch: master-1.12
Repository: wireshark

Commits:

ce1819b by Pascal Quantin (pascal.quantin@gmail.com):

    LTE-RRC: fix dissection of GPRS container
    
    Payload embedded in LTE message does not include the first byte identifying the message type.
    Let's prepend a fake one before calling the gsm_rlcmac dissector
    
    Change-Id: Ibcf9b52902474a556b55e9b0a076d09d341f868c
    Reviewed-on: https://code.wireshark.org/review/15642
    Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
    (cherry picked from commit d77b8318e82c113b2f7852f950b9269b57e5da40)
    Conflicts:
    	epan/dissectors/asn1/lte-rrc/lte-rrc.cnf
    	epan/dissectors/packet-lte-rrc.c
    Reviewed-on: https://code.wireshark.org/review/15657
    

Actions performed:

    from  d18f7f2   Fix display of composite tvbs in GUI
    adds  ce1819b   LTE-RRC: fix dissection of GPRS container


Summary of changes:
 asn1/lte-rrc/lte-rrc.cnf         |   16 +++++++++++++++-
 epan/dissectors/packet-lte-rrc.c |   16 +++++++++++++++-
 2 files changed, 30 insertions(+), 2 deletions(-)
___________________________________________________________________________
Sent via:    Wireshark-commits mailing list <wireshark-commits@wireshark.org>
Archives:    https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
             mailto:wireshark-commits-request@wireshark.org?subject=unsubscribe
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic