Wireshark-commits: [Wireshark-commits] rev 50450: /trunk/epan/dissectors/ /trunk/epan/dissectors/:

Date: Mon, 08 Jul 2013 21:12:06 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=50450

User: morriss
Date: 2013/07/08 02:12 PM

Log:
 Fix the very long loop reported in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8911 :
 
 In parseFields() ensure that we have at least one byte so that callers
 don't have to protect against it returning an offset which hasn't incremented.
 
 Remove a couple of now-unnecessary length-remaining (aka "will the offset
 move?") checks.
 
 In some other checks, use tvb_ensure_length_remaining() rather than calling
 tvb_length_remaining() and (potentially) THROWing an exception.  I'm not sure
 if these are really necessary now or not...

Directory: /trunk/epan/dissectors/
  Changes    Path                 Action
  +31 -48    packet-dis-pdus.c    Modified