> On Apr 22, 2016, at 00:51, Garrett Cooper <n...@freebsd.org> wrote: > > Author: ngie > Date: Fri Apr 22 07:51:38 2016 > New Revision: 298462 > URL: https://svnweb.freebsd.org/changeset/base/298462 > > Log: > Return `ret` in op_ifentry(..) to mute a -Wunused-but-set-variable warning > > This will also now detect error conditions with > value->var.subs[sub - 1] == LEAF_ifPhysAddress where `string_get(..)` > could fail if iifp->physaddr and/or iifp->physaddrlen were deemed > invalid. > > MFC after: 2 weeks > CID: 1006551 > Reported by: Coverity, gcc > Sponsored by: EMC / Isilon Storage Division
I pulled up the wrong CID. Coverity didn’t actually complain about this — just gcc. Thanks, -Ngie _______________________________________________ svn-src-all@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"