Re: [PATCH 4/4] Fix leak of EIR data if RSSI does not change

Anderson Lizardo
2010-12-27T18:23:26Z

Hi Johan,

On Mon, Dec 27, 2010 at 11:11 AM, Johan Hedberg <johan.hedberg@gmail.com> wrote:
> In general this seems fine, but to avoid future memory leaks in case you
> go ahead and change the eir_data struct wouldn't it be better to have a
> separate free_eir_data() function which does the g_slist_foreach,
> g_slist_free and g_free?

I followed your suggestion and created a free_eir_data(). I also did a
simple code reordering that simplified the complete EIR name storage.
See v2 of this patch. (2/4 should be ignored).

Regards,
--
Anderson Lizardo
OpenBossa Labs - INdT
Manaus - Brazil