[PATCH] Address another obvious memory leak in loadparm.

Andreas Schneider asn at samba.org
Wed Feb 17 10:22:09 UTC 2016

On Wednesday 17 February 2016 08:47:43 Volker Lendecke wrote:
> On Wed, Feb 17, 2016 at 12:18:09AM +0000, Hemanth Thummala wrote:
> > I have attached the patch which has the fix for this
> > problem. Please let me know if the patch looks good.
> Yes, the patch looks very good, thanks! I haven't run a private autobuild
> with that yet. Does that survive?
> Attached find two cosmetic changes. Feel free to squash them.
> Also, your change triggers the question -- do we have a proper talloc
> hierarchy for "struct loadparm_service"? If so, couldn't we avoid the
> special handling in free_service()?

We also need to backport this, so please open a bug report and add it to the 
commit message. Check "git log v4-4-test" for the format ...


	-- andreas

Andreas Schneider                   GPG-ID: CC014E3D
Samba Team                             asn at samba.org

More information about the samba-technical mailing list