[Repository Listing] / [Wireshark]
 
Wireshark development

We've moved to Git

On January 28, 2014 we migrated from Subversion to Git. Please go to our code review site for current development activity.

Revision 44674


Jump to revision: Previous Next
Author: pascal
Date: Sat Aug 25 21:31:29 2012 UTC (2 years, 2 months ago)
Log Message:
From Sylvain Munaut via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7664 :
packet-gmr1_bcch: Add guards in the SI1/2 choice of segment

Although the CSN1 dissector itself will just stop if there is
no matching segment, it will leave the choice field uninitizalized
and so when we use it to fill some other text, it crashes ...

To protect against that, we put a last choice entry that will always
match. As a bonus, it triggers an explicit error in CSN so you
know something is wrong.


Changed paths

Path Action
trunk/epan/dissectors/packet-gmr1_bcch.c trunk/epan/dissectors/packet-gmr1_bcch.c modified Revision Log
Diff to Previous

webmaster [at] wireshark.org
Tango Icons © Tango Desktop Project. Subversion  TortoiseSVN  ViewVC