Date: Sat, 8 Mar 2008 18:34:17 +0000 (UTC) From: "Bjoern A. Zeeb" <bz@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/netipsec key.c Message-ID: <200803081834.m28IYHB6035877@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
bz 2008-03-08 18:34:17 UTC FreeBSD src repository Modified files: (Branch: RELENG_6) sys/netipsec key.c Log: MFC rev. 1.25 Add a missing return so that we drop out in case of an error and do not continue with a NULL pointer. [1] While here change the return of the error handling code path above. I cannot see why we should always return 0 there. Neither does KAME nor do we in here for the similar check in all the other functions. Found with: Coverity Prevent(tm) [1] CID: 2521 Revision Changes Path 1.20.2.3 +2 -3 src/sys/netipsec/key.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200803081834.m28IYHB6035877>