Wireshark-bugs: [Wireshark-bugs] [Bug 9296] Wireshark lua dissector unable to load for media_typ
Comment # 6
on bug 9296
from Michael Mann
(In reply to comment #5)
> The simplest fix, I think, is to have dissector_add_string do a g_strdup of
> the key, and subsequently free it either in dissector_delete_string or when
> the table is destroyed at the end of the program (possibly by specifying a
> key destructor function to g_hash_table_new_full). The annoying part is that
> this wastes a little memory for the 99.9% of strings that are static on the
> stack, but I can't think of anything else off the top of my head.
>
> Other ideas are welcome :)
I still think that using capture scope memory in the lua wrapper is "simpler".
While I have seen dissectors added/deleted during dissection (closest I can
come to a real world example of your while loop), it's usually done in the
"handoff". And the memory footprint is small enough that I can live with it
over punishing all other dissectors (and overall Wireshark memory) with the
extra allocations.
I assume using se_ API would be prefered as it needs to be backported? This
also seems like a "messy" area for wmem.
You are receiving this mail because:
- You are watching all bug changes.
