Date: Sat, 9 Dec 2006 18:57:40 +0100 From: Pawel Jakub Dawidek <pjd@FreeBSD.org> To: Craig Rodrigues <rodrigc@FreeBSD.org> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/fs/unionfs union_subr.c Message-ID: <20061209175740.GB12193@garage.freebsd.pl> In-Reply-To: <200612091627.kB9GRoRU010471@repoman.freebsd.org> References: <200612091627.kB9GRoRU010471@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] On Sat, Dec 09, 2006 at 04:27:50PM +0000, Craig Rodrigues wrote: > rodrigc 2006-12-09 16:27:50 UTC > > FreeBSD src repository > > Modified files: > sys/fs/unionfs union_subr.c > Log: > In unionfs_hashrem(), use LIST_FOREACH_SAFE when iterating over > the list of nodes to free them. > > CID: 1668 > Found by: Coverity Prevent (tm) What was before is also safe, but coverity prevent often reports it as a bug, which is actually false positive. -- Pawel Jakub Dawidek http://www.wheel.pl pjd@FreeBSD.org http://www.FreeBSD.org FreeBSD committer Am I Evil? Yes, I Am! [-- Attachment #2 --] -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.4 (FreeBSD) iD8DBQFFevkUForvXbEpPzQRAuEzAKCuvIvcnf4voZ7qMq0CskaD7ZWvFwCguF1S qMmrdtnPy+mFvr+azvsrwTs= =Zt3e -----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20061209175740.GB12193>