Wireshark-bugs: [Wireshark-bugs] [Bug 8787] 9P dissector - compute fids on first visit
Comment # 21
on bug 8787
from Dominique Martinet
(In reply to comment #20)
> > Another thing which has changed since this started: the ep_ and se_
> > allocators have been deprecated. Ideally those allocations would be
> > replaced with wmem (see doc/README.wmem).
>
> wmem looks nice! It doesn't seem such a hard change, but I'd rather have it
> in a different patch as this one already is quite big, and I'm more of the
> "incremental" type :P (It'd go with the tcp_dissect_pdus one that should be
> quite small)
> Also, I suppose this means I probably should replace g_malloc with wmem with
> the epan scope? does tvb allocated data gets allocated with this as well,
> and should use wmem_free instead of g_free? (e.g. tvb_get_string returns a
> g_malloc'd string at the moment, I suppose there would be two functions
> laying around while the transition is being done?)
Sorry for the double comment, but I just noticed wmem doesn't provide trees yet
- so I guess this much will have to wait at least :)
(I'd assume tvb isn't ready just yet either :P And I guess that if
tvb_get_string's return value is allocated in the packet scope, there's no need
to free actually)
You are receiving this mail because:
- You are watching all bug changes.
