Wireshark-commits: [Wireshark-commits] rev 49627: / /trunk-1.8/epan/dissectors/: packet-gmr1_bcch.c

Date: Thu, 30 May 2013 01:21:37 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=49627

User: morriss
Date: 2013/05/29 06:21 PM

Log:
 Copy over revision from the trunk in order to fix the fuzz failure reported
 in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8726 :
 
   ------------------------------------------------------------------------
   r44674 | pascal | 2012-08-25 17:31:29 -0400 (Sat, 25 Aug 2012) | 11 lines
   Changed paths:
      M /trunk/epan/dissectors/packet-gmr1_bcch.c
 
   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.
 
   ------------------------------------------------------------------------

Directory: /trunk-1.8/epan/dissectors/
  Changes    Path                  Action
  +2 -0      packet-gmr1_bcch.c    Modified