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

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: cmaynard@xxxxxxxxxxxxx
Date: Tue, 31 Jul 2012 00:35:38 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=44149

User: cmaynard
Date: 2012/07/30 05:35 PM

Log:
 tvb_length_remaining() can return -1 if the offset is out of bounds, so return if it does.  In this case, it won't since tvb_offset_exists() is previously called, but it's not a bad idea to do it anyway in case the call to tvb_offset_exists() is ever removed, which we might just want to do?  Anyway, this is a simple enough change and fixes Coverity CID 280522: Improper use of negative value.

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