Wireshark-dev: Re: [Wireshark-dev] g_mem-chunk_destroy does not free memory for GLib > 2.8
From: Jaap Keuter <[email protected]>
Date: Tue, 13 Oct 2009 07:12:07 +0200
Hi,

Memory leaks are real bugs that need to be fixed in the release branch. So yeah, careful backporting is required.
Thanx,
Jaap

Send from my iPhone

On 13 okt 2009, at 03:55, "Anders Broman" <[email protected]> wrote:
Hi,
Should fixes to these probleems be backported to 1.2.3?
Regards
Anders

-----Original Message-----
From: [email protected]
[mailto:[email protected]] On Behalf Of Bill Meier
Sent: den 11 oktober 2009 15:32
To: Developer support list for Wireshark
Subject: Re: [Wireshark-dev] g_mem-chunk_destroy does not free memory
for GLib > 2.8

Kovarththanan Rajaratnam wrote:
Hey Bill,


Is there any reason why we couldn't use se_alloc and friends? This
should be faster now that we're defaulting to _not_ use canaries for
seasonal memory.

Using se_alloc is the conclusion I came to.... (when the code frees
GMemChunk memory per-capture).

However: packet-isakmp destroys GMemChunks more often. I haven't yet
looked at the other dissectors (and other code) using GMemChunks to see
if there are other cases wherein GMemChunks are destroyed more often
than every capture.

If so, it occurs to me that it might be quite simple to just allow
dissectors & etc  to init/alloc/free their own emem_chunks using
slightly generalized emem code.....

Bill


________________________________________________________________________

___
Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev

mailto:[email protected]?subject=unsubscribe
___________________________________________________________________________

Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
            mailto:[email protected]?subject=unsubscribe