URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=20b214f3ca085714dd35dd9aa0779ee44a20f367
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master
Repository: wireshark
Commits:
20b214f by Vadim Yanitskiy (axilirator@xxxxxxxxx):
[GSM A RR]: fix: properly dissect APDU Flags for Application Info
According to 3GPP TS 04.08, section 10.5.2.49, the APDU Flags field
is coded together with APDU ID, and occupies bits 1-3 (mask 0x70):
0 1 2 3
S T F L
. . . * Last Segment
. . * . First Segment
. * . . C/R, if L=0 only, otherwise spare and set to 0
* . . . Spare (0)
Instead of parsing all bits together as a set of integer values,
let's parse each flag individually. Moreover, the previous
definition was missing some possible bit combinations, so
this change also fixes that problem.
Change-Id: Id71fae9ef06572c1ad17aafe0be3dfb66e081b7d
Reviewed-on: https://code.wireshark.org/review/28948
Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
Tested-by: Petri Dish Buildbot
Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
Actions performed:
from 98484f6 AT: Allow test command for AT+CGMM
add 20b214f [GSM A RR]: fix: properly dissect APDU Flags for Application Info
Summary of changes:
epan/dissectors/packet-gsm_a_rr.c | 41 ++++++++++++++++++++++++++++++---------
1 file changed, 32 insertions(+), 9 deletions(-)