Date: Sun, 31 Oct 2004 09:45:25 +0100 From: "Jacques A. Vidrine" <nectar@FreeBSD.org> To: Danny Braniss <danny@cs.huji.ac.il> Cc: bushman@rsu.ru Subject: Re: nsdispatch services patch + lookupd Message-ID: <20041031084525.GF2099@lum.celabo.org> In-Reply-To: <20041027073335.8B74343D55@mx1.FreeBSD.org> References: <1098785868.61417.39.camel@localhost> <20041027073335.8B74343D55@mx1.FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Oct 27, 2004 at 09:33:33AM +0200, Danny Braniss wrote: > while trying to add hesiod/dns support, i've noticed, what looks as a problem: > > in nss_tls.h, the function name##_getstate(...) can return a static pointer, > which gets freed in name##_endstate(...), as far as i know, freeing a non > malloced memory is asking for trouble. > proposed fix, instead of static, also do a calloc(...). Yes, looks like you are correct. I will fix this month. Thanks for the report! (Is there already a PR?) Cheers, -- Jacques A Vidrine / NTT/Verio nectar@celabo.org / jvidrine@verio.net / nectar@FreeBSD.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20041031084525.GF2099>