From owner-cvs-all Thu Jul 19 9:28:37 2001 Delivered-To: cvs-all@freebsd.org Received: from aldan.algebra.com (aldan.algebra.com [216.254.65.224]) by hub.freebsd.org (Postfix) with ESMTP id 9CC3737B403; Thu, 19 Jul 2001 09:28:30 -0700 (PDT) (envelope-from mi@aldan.algebra.com) Received: from aldan.algebra.com (localhost [127.0.0.1]) by aldan.algebra.com (8.11.4/8.11.4) with ESMTP id f6JGSIO33402; Thu, 19 Jul 2001 12:28:23 -0400 (EDT) (envelope-from mi@aldan.algebra.com) Message-Id: <200107191628.f6JGSIO33402@aldan.algebra.com> Date: Thu, 19 Jul 2001 12:28:15 -0400 (EDT) From: Mikhail Teterin Subject: Re: cvs commit: src/usr.sbin/mailwrapper mailwrapper.c To: dima@unixfreak.org Cc: kris@FreeBSD.org, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org In-Reply-To: <20010719144945.0B1AA3E2F@bazooka.unixfreak.org> MIME-Version: 1.0 Content-Type: TEXT/plain; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On 19 Jul, Dima Dorfman wrote: > Mikhail Teterin writes: >> On 18 Jul, Kris Kennaway wrote: >> > kris 2001/07/18 22:04:46 PDT >> > >> > Modified files: (Branch: RELENG_4) >> > usr.sbin/mailwrapper mailwrapper.c >> > Log: >> > MFC: r1.6: You can't free a string and then use it as the error >> > message to errx. Instead use warn to print the string, then >> > cleanup and exit normally. [...] >> It was not there before, I guess, because the "founding fathers" did >> not concern themselves with the fate of the allocated memory after >> exit :-) > > Maybe I'm missing something, but who cares about allocated memory > *after* *exit*? Well, may be *I* am missing something, but this seems to be the point of the way Kris fixed the problem -- the string was beeing freed and then passed to errx. Obviously wrong. Instead of just removing the free(), he replaced the errx with warn (before the free()) and exit (after free()). (Personally, I wouldn't care either. I don't think, it is a sign of a good programmer to care about this.) -mi To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message