Date: Sat, 3 Jun 2017 20:16:32 -0500 From: Pedro Giffuni <pfg@FreeBSD.org> To: "Ngie Cooper (yaneurabeya)" <yaneurabeya@gmail.com>, Conrad Meyer <cem@FreeBSD.org> Cc: src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r319557 - head/sys/fs/ext2fs Message-ID: <fdeec82d-0813-7f00-281b-896056551abd@FreeBSD.org> In-Reply-To: <25EB123A-EAF7-4ADD-8834-93C2618315B6@gmail.com> References: <201706032230.v53MUUAB056536@repo.freebsd.org> <25EB123A-EAF7-4ADD-8834-93C2618315B6@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 03/06/2017 17:54, Ngie Cooper (yaneurabeya) wrote: >> On Jun 3, 2017, at 15:30, Conrad Meyer <cem@FreeBSD.org> wrote: >> >> Author: cem >> Date: Sat Jun 3 22:30:30 2017 >> New Revision: 319557 >> URL: https://svnweb.freebsd.org/changeset/base/319557 >> >> Log: >> ext2fs(4): Fix a number of broken flag checks >> >> Introduced in r319071. >> >> Reported by: Coverity >> CIDs: 1375847, 1375848, 1375849 >> Sponsored by: Dell EMC Isilon > Did pjg review this? I don't know about pjg, but I wasn't even aware about the Coverity issues. I never added myself to the MAINTAINERS list so it's OK, I will just have the code submitter confirm the fixes. Luckily this is code that has not been MFC'd because I don't consider it bullet-proof and more eyes are always welcome. Pedro.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?fdeec82d-0813-7f00-281b-896056551abd>