Date: Sat, 23 Feb 2019 02:09:07 +1100 (EST) From: Bruce Evans <brde@optusnet.com.au> To: David Bright <dab@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r344468 - head/usr.sbin/newsyslog Message-ID: <20190223020739.H33323@besplex.bde.org> In-Reply-To: <201902221459.x1MExegD026299@repo.freebsd.org> References: <201902221459.x1MExegD026299@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 22 Feb 2019, David Bright wrote: > Log: > Revert r344389 & r343906 - Fix Coverity errors in newsyslog > > It was pointed out that a couple of the "memory leak" CIDs that I > fixed were arguably Coverity errors rather than errors in the > newsyslog code and the cure was worse than the disease. Revert both > changes. The first change, which included fixes for other Coverity > errors, will be re-worked to omit the troublesome changes and then > re-committed with the remaining fixes. > > Reported by: bde > Sponsored by: Dell EMC Isilon Thanks. The other changes look OK. Bruce
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20190223020739.H33323>