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

Remove useless NULL check

Browse Source 
Coverity points out that we've already dereferenced m by the time we check, so
there's no reason to keep the check. Moreover, it's safe to pass NULL to
m_freem() anyway.

CID:		1019092
This commit is contained in:
Kristof Provost 2019-12-05 16:50:54 +00:00
parent 324cdd9320
commit 200424235e
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
sys/netinet6/udp6_usrreq.c
View File

@ -528,8 +528,7 @@ udp6_input(struct mbuf **mp, int *offp, int proto)
return (IPPROTO_DONE);
badunlocked:
if (m)
m_freem(m);
m_freem(m);
*mp = NULL;
return (IPPROTO_DONE);
}