Date: Tue, 05 Jun 2001 02:33:53 +0100 From: Paul Richards <paul@freebsd-services.co.uk> To: Dag-Erling Smorgrav <des@ofug.org>, Paul Richards <paul@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/usr.sbin/mailwrapper mailwrapper.c Message-ID: <57630000.991704833@lobster.originative.co.uk> In-Reply-To: <xzpg0dfbxos.fsf@flood.ping.uio.no>
next in thread | previous in thread | raw e-mail | index | archive | help
--On Tuesday, June 05, 2001 02:02:59 +0200 Dag-Erling Smorgrav <des@ofug.org> wrote: > Paul Richards <paul@FreeBSD.org> writes: >> Log: >> 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. > > If you're going to exit() anyway, you don't need to bother freeing the > string. That would be a pretty sloppy coding habit in general which is why I decided against it. I did think about it in this case though because errx was the last consumer of the string. Paul. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?57630000.991704833>