Wireshark-commits: [Wireshark-commits] rev 51342: /trunk/ /trunk/epan/: epan.c /trunk/: tshark.c /t

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: eapache@xxxxxxxxxxxxx
Date: Tue, 13 Aug 2013 22:41:35 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=51342

User: eapache
Date: 2013/08/13 03:41 PM

Log:
 As suggested in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9033
 
 Make epan_free a no-op if the pointer is NULL. This fixes 99% of the cases
 causing problems for wmem_leave_file_scope() - remove that XXX comment and add
 back the assertion.
 
 Remove the cleanup_dissection call from epan_cleanup, it doesn't make sense
 there. init_dissection is only called from epan_new, so cleanup_dissection
 should only be called from epan_free.
 
 Add one missing epan_free call to tshark revealed by the above changes.

Directory: /trunk/epan/
  Changes    Path          Action
  +5 -4      epan.c        Modified

Directory: /trunk/
  Changes    Path          Action
  +1 -0      tshark.c      Modified

Directory: /trunk/epan/wmem/
  Changes    Path             Action
  +1 -11     wmem_scopes.c    Modified