Wireshark · Wireshark-commits: [Wireshark-commits] rev 44149: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-rlogin.c

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

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

Prev by Date: [Wireshark-commits] rev 44148: /trunk/ui/qt/ /trunk/ui/qt/: monospace_font.cpp
Next by Date: [Wireshark-commits] rev 44150: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-dvb-s2-bb.c
Previous by thread: [Wireshark-commits] rev 44148: /trunk/ui/qt/ /trunk/ui/qt/: monospace_font.cpp
Next by thread: [Wireshark-commits] rev 44150: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-dvb-s2-bb.c
Index(es):
- Date
- Thread