From owner-freebsd-hackers@FreeBSD.ORG Mon Nov 16 20:42:06 2009 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 184FE1065676 for ; Mon, 16 Nov 2009 20:42:06 +0000 (UTC) (envelope-from gary.jennejohn@freenet.de) Received: from mout5.freenet.de (mout5.freenet.de [IPv6:2001:748:100:40::2:7]) by mx1.freebsd.org (Postfix) with ESMTP id CBA708FC14 for ; Mon, 16 Nov 2009 20:42:05 +0000 (UTC) Received: from [195.4.92.17] (helo=7.mx.freenet.de) by mout5.freenet.de with esmtpa (ID gary.jennejohn@freenet.de) (port 25) (Exim 4.69 #92) id 1NA8Oi-00073B-9u; Mon, 16 Nov 2009 21:42:04 +0100 Received: from te89d.t.pppool.de ([89.55.232.157]:30522 helo=ernst.jennejohn.org) by 7.mx.freenet.de with esmtpa (ID gary.jennejohn@freenet.de) (port 25) (Exim 4.69 #94) id 1NA8Oi-0001Hz-1l; Mon, 16 Nov 2009 21:42:04 +0100 Date: Mon, 16 Nov 2009 21:42:02 +0100 From: Gary Jennejohn To: volker@vwsoft.com Message-ID: <20091116214202.480a0aa7@ernst.jennejohn.org> In-Reply-To: <4B01B23F.8040002@vwsoft.com> References: <4B01B23F.8040002@vwsoft.com> X-Mailer: Claws Mail 3.7.2 (GTK+ 2.16.2; amd64-portbld-freebsd8.0) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: freebsd-hackers@freebsd.org, Jim Wilcoxson Subject: Re: acl_from_text leaking memory X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: gary.jennejohn@freenet.de List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 Nov 2009 20:42:06 -0000 On Mon, 16 Nov 2009 21:12:47 +0100 volker@vwsoft.com wrote: > you may want to have a look at the manpage acl_from_text(3): > > "...This function may cause memory to be allocated. The caller should > free any releasable memory, when the new ACL is no longer required, by > calling acl_free(3) with the (void *)acl_t as an argument." > > Please use an acl_free(void *obj_p) call afterwards to avoid leaking memory. > The suggested fix was appplied to HEAD today. Apparently, the man page should now be updated. --- Gary Jennejohn