Wireshark-commits: [Wireshark-commits] rev 49740: / /trunk-1.6/epan/dissectors/: packet-rpc.c /trun

Date: Tue, 04 Jun 2013 00:51:30 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=49740

User: gerald
Date: 2013/06/03 05:51 PM

Log:
 Copy over revisions from the trunk:
 
   ------------------------------------------------------------------------
   r49652 | eapache | 2013-05-31 06:08:03 -0700 (Fri, 31 May 2013) | 5 lines
   Changed paths:
      M /trunk/epan/proto.c
 
   Fix the integer overflow case Jakub caught in
   https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3290#c26
 
   Also use indentation consistent with the rest of the file.
   ------------------------------------------------------------------------
 
 
 Copy over with manual intervention:
 
   ------------------------------------------------------------------------
   r49645 | morriss | 2013-05-30 18:46:40 -0700 (Thu, 30 May 2013) | 6 lines
   Changed paths:
      M /trunk/epan/dissectors/packet-rpc.c
 
   Get rid of most of the if(tree) checks: the proto_tree_add_*() functions
   don't need them any more and, as shown in
   https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3290#c16
   we really want to check if we're looking at something beyond the end of the TVB
   (in order to break out of very long loops).
   ------------------------------------------------------------------------
   r49644 | morriss | 2013-05-30 18:46:28 -0700 (Thu, 30 May 2013) | 14 lines
   Changed paths:
      M /trunk/epan/proto.c
 
   (Finally!) check in part of Didier's patch to fix
   https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3290
   (TRY_TO_FAKE_THIS_ITEM disables bounds errors):
 
   Before calling TRY_TO_FAKE_THIS_ITEM() check if the length given (or, in
   the case of FT_UINT_{STRING,BYTES}, the length we retrieve from the TVB)
   exceeds what's left in the TVB.
 
   Do this only for proto_tree_add_item() for now (it's the most commonly used
   and thus the biggest trouble maker in this area).
 
   Similar changes for other APIs will come later (if nothing blows up).  Despite
   the fuzz failures this bug has caused I'm not sure about back-porting it...
   ------------------------------------------------------------------------
 
 
 Update the release notes.

Directory: /trunk-1.6/epan/dissectors/
  Changes    Path            Action
  +40 -61    packet-rpc.c    Modified

Directory: /trunk-1.6/docbook/
  Changes    Path                 Action
  +6 -0      release-notes.xml    Modified

Directory: /trunk-1.6/epan/
  Changes    Path          Action
  +38 -3     proto.c       Modified