Date: Fri, 15 Jun 2007 23:45:39 +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: <200706152345.l5FNje1U035601@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
bz 2007-06-15 23:45:39 UTC
FreeBSD src repository
Modified files:
sys/netipsec key.c
Log:
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.25 +2 -3 src/sys/netipsec/key.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200706152345.l5FNje1U035601>