d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

With the current code 'src' is never NULL. Nevertheless move the check for

Browse Source 
NULL before dereferencing the pointer.

Found with:	Coverity Prevent(tm)
CID:		2528
This commit is contained in:
Bjoern A. Zeeb 2007-06-15 22:35:59 +00:00
parent 0061e03d7f
commit 91c7ac6740
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
sys/netipsec/key.c
View File

@ -7296,11 +7296,13 @@ key_setkey(struct seckey *src, u_int16_t exttype)
{ {
struct mbuf *m; struct mbuf *m;
struct sadb_key *p; struct sadb_key *p;
int len = PFKEY_ALIGN8(sizeof(struct sadb_key) + _KEYLEN(src)); int len;
if (src == NULL) if (src == NULL)
return NULL; return NULL;
len = PFKEY_ALIGN8(sizeof(struct sadb_key) + _KEYLEN(src));
m = key_alloc_mbuf(len); m = key_alloc_mbuf(len);
if (m == NULL) if (m == NULL)
return NULL; return NULL;