GTPv1-C / MM Context parameter / Authentication quintuplet / RAND is not correct

[GoogleCodeExporter]

************************************************************
***** NOTE: THIS BUG TRACKER IS FOR GERRIT CODE REVIEW *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, INTERNAL *****
***** ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.    *****
***** THOSE ISSUE BELONG IN DIFFERENT ISSUE TRACKERS!  *****
************************************************************

Affected Version:

What steps will reproduce the problem?
1. Load the attached file
2. Look at the GTP RAND value in the second or third Quintuplet.

What is the expected output? What do you see instead?
The RAND value in the packet details is different from that in the packet bytes.

Please provide any additional information below.
The bug is in the packet-gtp.c, decode_quintuplet() function, in the line:
  proto_tree_add_text(ext_tree_quint, tvb, offset + q_offset, 16, "RAND: %s", tvb_bytes_to_str(tvb, offset, 16));

The correct code would be:
  proto_tree_add_text(ext_tree_quint, tvb, offset + q_offset, 16, "RAND: %s", tvb_bytes_to_str(tvb, offset + q_offset, 16));

Original issue reported on code.google.com by lukobal...@gmail.com on 4 Feb 2014 at 12:19

Attachments:

• GTPv1-C_bad_RAND.pcap

[GoogleCodeExporter]

The problem arises in Wireshark v1.10.5.

Original comment by lukobal...@gmail.com on 4 Feb 2014 at 12:23

[GoogleCodeExporter]

Wrong issue tracker. This issue tracker is about the Gerrit Code Review 
Project, but what you describe seems to have nothing to do with Gerrit.

Original comment by edwin.ke...@gmail.com on 4 Feb 2014 at 12:24

• Changed state: Invalid

[GoogleCodeExporter]

You are right, I put it to the wrong place, sorry!

Original comment by lukobal...@gmail.com on 4 Feb 2014 at 1:44