gtp.sai_sac field size limitation

edit

There seems to be a discrepancy between the use of the "sai_sac" field and the registration of this field. The use is based on a 2 octet value (proto_tree_add_item(tree, hf_gtp_sai_sac, tvb, offset, 2, ENC_BIG_ENDIAN);) while the registration is for a 1 octet value (..."Service Area Code (SAC)", "gtp.sai_sac", FT_UINT8, BASE_DEC,...). "gtp.rai_rac" suffers this same fate.

Please file a bug report, attaching a sample capture file showing this issue, so that it can be fixed in a later release.

Looks like a bug, the field definition in the gtp dissector source has it as a FT_UINT8, but the dissection reads 2 bytes from the packet:

{&hf_gtp_cgi_ci,
 { "Cell ID (CI)", "gtp.cgi_ci",
   FT_UINT8, BASE_DEC, NULL, 0,
   NULL, HFILL}
},
{&hf_gtp_sai_sac,
 { "Service Area Code (SAC)", "gtp.sai_sac",
   FT_UINT8, BASE_DEC, NULL, 0,
   NULL, HFILL}
},
{&hf_gtp_rai_rac,
 { "Routing Area Code (RAC)", "gtp.rai_rac",
   FT_UINT8, BASE_DEC, NULL, 0,
   NULL, HFILL}
},

gtp.cgi_ci and gtp.rai_rac seem to be similarly afflicted. Please raise a bug entry at our bugzilla.