Re: [Wireshark-commits] rev 46320: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-bthci_acl.c

[Jakub Zawadzki <darkjames-ws () darkjames pl>]

On Sun, Dec 02, 2012 at 09:28:24AM -0500, Evan Huus wrote:
> I'm starting to think we tightened the scope of ep memory too much. I do
> think that freeing it at the beginning of epan_dissect_run was wrong, but
> I'm starting to wonder if the right place for it isn't in epan_dissect_run
> at all (beginning or end) but is in epan_dissect_cleanup instead. As a
> bonus I believe this may let us get rid of the epan_dissect_run_with_taps
> function?

You've already proposed it once, in my opinion in some scenarios it won't work,
please read my reply: http://www.wireshark.org/lists/wireshark-dev/201210/msg00198.html

Anyway, some memory pool only for single edt initalized in epan_dissect_init(),
and later freed in epan_dissect_cleanup() will work.

Also I proposed other (simpler?) idea: http://www.wireshark.org/lists/wireshark-dev/201210/msg00204.html
But we right now have wmem_ so it'd be better to use wmem interface.

Cheers,
 Jakub.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe