From 6cf2e00820aef4e855664499404eb94abbc026cb Mon Sep 17 00:00:00 2001 From: ae Date: Thu, 4 Dec 2014 05:02:12 +0000 Subject: Fix style(9) and remove m_freem(NULL). Add XXX comment, it looks incorrect, because m_pkthdr.len is already incremented by M_PREPEND(). Sponsored by: Yandex LLC --- sys/netipsec/keysock.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'sys/netipsec') diff --git a/sys/netipsec/keysock.c b/sys/netipsec/keysock.c index ed0c384..6deea35 100644 --- a/sys/netipsec/keysock.c +++ b/sys/netipsec/keysock.c @@ -145,12 +145,11 @@ key_sendup0(struct rawcb *rp, struct mbuf *m, int promisc) struct sadb_msg *pmsg; M_PREPEND(m, sizeof(struct sadb_msg), M_NOWAIT); - if (!m) { + if (m == NULL) { PFKEYSTAT_INC(in_nomem); - m_freem(m); - return ENOBUFS; + return (ENOBUFS); } - m->m_pkthdr.len += sizeof(*pmsg); + m->m_pkthdr.len += sizeof(*pmsg); /* XXX: is this correct? */ pmsg = mtod(m, struct sadb_msg *); bzero(pmsg, sizeof(*pmsg)); -- cgit v1.1