Wireshark-commits: [Wireshark-commits] rev 20569: /trunk/ /trunk/doc/: README.developer /trunk/docb

Date: Fri, 26 Jan 2007 17:12:01 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=20569

User: jake
Date: 2007/01/26 05:12 PM

Log:
 From Graham Bloice:
 In the Developers Guide, Section 9.3, Example 9.17. Decompressing data
 packets for dissection
 
 The code calls tvb_set_free_cb() for the newly created next_tvb.  This
 is unnecessary as the call to tvb_set_child_real_data() adds next_tvb to
 the chained list of tvb, thus ensuring that next_tvb is correctly
 deleted.  In fact when I had the call in, Visual Studio kept breaking
 deep down in ntdll.dll, probably because of a double free every time the
 main tvb was deleted.
 
 In README.developer, para 2..2.7 The example conversation code doesn't
 assign the result of conversation_new() back into the conversation variable.

Directory: /trunk/doc/
  Changes    Path                Action
  +1 -1      README.developer    Modified

Directory: /trunk/docbook/wsdg_src/
  Changes    Path                           Action
  +0 -1      WSDG_chapter_dissection.xml    Modified