[prev in list] [next in list] [prev in thread] [next in thread]
List: wireshark-commits
Subject: [Wireshark-commits] master-2.0 8ada61c: LTE-RRC: fix dissection of GPRS container
From: Wireshark code review <code-review-do-not-reply () wireshark ! org>
Date: 2016-05-31 21:11:50
Message-ID: 20160531211137.28003.49342 () thud ! wireshark ! org
[Download RAW message or body]
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=8ada61c1ee5b01c56449004c355f59b1f34fddc2
Submitter: Pascal Quantin (pascal.quantin@gmail.com)
Changed: branch: master-2.0
Repository: wireshark
Commits:
8ada61c 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/15655
Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Actions performed:
from b1358c0 Fix display of composite tvbs in GUI
adds 8ada61c 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