Wireshark · Wireshark-commits: [Wireshark-commits] rev 30611: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-per.c

[morriss@xxxxxxxxxxxxx]

http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=30611

User: morriss
Date: 2009/10/18 04:19 PM

Log:
 new_octet_aligned_subset_bits(): call tvb_ensure_bytes_exist() before
 attmepting to malloc memory to ensure we've been asked to decode a reasonable
 number of bits (and thus are asking for a reasonable number of bytes from
 malloc()).
 
 That should fix the fuzz failure in
 https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4138
 
 Add a comment to new_octet_aligned_subset() asking why we're resetting the
 length to tvb_length()--that means we'll never throw an exception.

Directory: /trunk/epan/dissectors/
  Changes    Path            Action
  +69 -60    packet-per.c    Modified