osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

GTP: Use alternative GSN address decoders for Update PDP Resp 

The GGSN addresses for control plane and user traffic are both
included or both not included in the Update PDP Context Response
message (included if the Cause is Request Accepted), so we know
which one is the control plane and which one is the user plane.
Also fix the coment about the IEs for the alternative address, and
that they are Conditional, not Optional.
This commit is contained in:
John Thacker 2022-02-01 20:05:51 -05:00 committed by A Wireshark GitLab Utility
parent 09ecc11be5
commit f201f23299
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
epan/dissectors/packet-gtp.c
View File

@ -3494,10 +3494,10 @@ static _gtp_mess_items umts_mess_items[] = {
{GTP_EXT_TEID_CP, GTP_CONDITIONAL, NULL},
{GTP_EXT_CHRG_ID, GTP_CONDITIONAL, NULL},
{GTP_EXT_PROTO_CONF, GTP_OPTIONAL, NULL}, /* Protocol Configuration Options Optional 7.7.31 */
{GTP_EXT_GSN_ADDR, GTP_CONDITIONAL, NULL},
{GTP_EXT_GSN_ADDR, GTP_CONDITIONAL, NULL},
{GTP_EXT_GSN_ADDR, GTP_OPTIONAL, NULL}, /* Alternative SGSN Address for Control Plane Conditional GSN Address 7.7.32 */
{GTP_EXT_GSN_ADDR, GTP_OPTIONAL, NULL}, /* Alternative SGSN Address for User Traffic Conditional GSN Address 7.7.32 */
{GTP_EXT_GSN_ADDR, GTP_CONDITIONAL, decode_gtp_ggsn_addr_for_control_plane},
{GTP_EXT_GSN_ADDR, GTP_CONDITIONAL, decode_gtp_ggsn_addr_for_user_plane},
{GTP_EXT_GSN_ADDR, GTP_CONDITIONAL, NULL}, /* Alternative GGSN Address for Control Plane Conditional GSN Address 7.7.32 */
{GTP_EXT_GSN_ADDR, GTP_CONDITIONAL, NULL}, /* Alternative GGSN Address for User Traffic Conditional GSN Address 7.7.32 */
{GTP_EXT_QOS_UMTS, GTP_CONDITIONAL, NULL},
{GTP_EXT_CHRG_ADDR, GTP_OPTIONAL, NULL},
{GTP_EXT_CHRG_ADDR, GTP_OPTIONAL, NULL}, /* Alternative Charging Gateway Address Optional 7.7.44 */