Recently, I found and fixed some of these problems, but obviously I didn't catch them all. Are there any more thoughts about changing tvb_length_remaining() and tvb_reported_length_remaining() to return 0 instead of -1?
- Chris
Ref: http://www.wireshark.org/lists/wireshark-dev/201207/msg00064.html
________________________________________
From: wireshark-commits-bounces@xxxxxxxxxxxxx [wireshark-commits-bounces@xxxxxxxxxxxxx] On Behalf Of martink@xxxxxxxxxxxxx [martink@xxxxxxxxxxxxx]
Sent: Monday, October 15, 2012 6:34 PM
To: wireshark-commits@xxxxxxxxxxxxx
Subject: [Wireshark-commits] rev 45566: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-rdp.c
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=45566
User: martink
Date: 2012/10/15 03:34 PM
Log:
(try to) fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7862
tvb_length_remaining() may return -1
if that happens in dissect_rdp_fields(), return an error
the caller that calls dissect_rdp_fields() from a for loop detects the error
and exits (others should handle the error as well, this is missing for now)
Directory: /trunk/epan/dissectors/
Changes Path Action
+8 -2 packet-rdp.c Modified
--
CONFIDENTIALITY NOTICE: The information contained in this email message is intended only for use of the intended recipient. If the reader of this message is not the intended recipient, you are hereby notified that any dissemination, distribution or copying of this communication is strictly prohibited. If you have received this communication in error, please immediately delete it from your system and notify the sender by replying to this email. Thank you.