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

When storing the results of malloc() in a pointer to a pointer, check

Browse Source 
the pointer to a pointer for NULL, not the pointer for NULL.

Noticed by:	Coverity Prevent analysis tool
MFC after:	3 days
This commit is contained in:
Robert Watson 2006-01-14 00:09:41 +00:00
parent 71c47d1480
commit fc4c825847
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sys/netinet6/ip6_output.c
View File

@ -3299,7 +3299,7 @@ ip6_setpktopt(optname, buf, len, opt, priv, sticky, cmsg, uproto)
/* turn off the previous option, then set the new option. */
ip6_clearpktopts(opt, optname);
*newdest = malloc(destlen, M_IP6OPT, M_NOWAIT);
if (newdest == NULL)
if (*newdest == NULL)
return (ENOBUFS);
bcopy(dest, *newdest, destlen);