Patch for winbindd memory leaks

jra at dp.samba.org jra at dp.samba.org
Mon Sep 23 19:06:00 GMT 2002


On Mon, Sep 23, 2002 at 03:14:34PM +0200, Kim R. Pedersen wrote:
> Hello Winbindd-Fans,
> 
> During my port of winbindd to my company's new platform, I have come across 
> three memory leaks in winbindd.
> 
> They are all related to misconfigured "password server" parameter, and 
> therefore not critical for "normal" environments.
> 
> As I'm working on an Internet Appliance, they are however critical for me, and 
> perhaps for others doing "ever-running appliance" devices.
> 
> Here we go:
> 
> 1) A simple missing free(ip_list) in cm_get_dc_name() file winbindd_cm.c.
> 
> 2) A wrong sequence of caching additions in cm_open_connection() file 
> winbindd_cm.c
> 
> 3) A potential endless loop in init_domain_list() file winbindd_util.c that 
> never calls lp_talloc_free().
> 
> I have attached the patches to winbindd_cm.c as the fixes are fairly simple 
> and obvious, but bug #3 is more problematic. One could simply add the missing 
> call to lp_talloc_free(), but I feel that there is something fundamentally 
> wrong with having this endless loop that doesn't take SIGHUP handling into 
> consideration or the fact that other non-domain related requests should be 
> serviced by winbindd.
> 
> In my own version of winbindd I simply return an error from init_domain_list() 
> and return back to the main loop, so reconfiguration can take place.

I've applied the first 2 fixes to 2.2 and appliance-head - thanks a *lot* !

The third I don't think is a problem with the way winbindd is currently
designed.

Thanks,

	Jeremy.



More information about the samba-technical mailing list