diff options
author | andre <andre@FreeBSD.org> | 2006-01-18 14:24:39 +0000 |
---|---|---|
committer | andre <andre@FreeBSD.org> | 2006-01-18 14:24:39 +0000 |
commit | a08da7cd98bf2aff1c327d1becbb7614c27a40b4 (patch) | |
tree | 4e69a0c80481917df0c10480c9cecc38671545fd /sys/net/if_ef.c | |
parent | 11141440e8f44bc40ad6efe297d1e4342166f159 (diff) | |
download | FreeBSD-src-a08da7cd98bf2aff1c327d1becbb7614c27a40b4.zip FreeBSD-src-a08da7cd98bf2aff1c327d1becbb7614c27a40b4.tar.gz |
Return mbuf pointer or NULL from ip_fastforward() as the mbuf pointer
may have changed by m_pullup() during fastforward processing.
While this is a bug it is actually never triggered in real world
situations and it is not remotely exploitable.
Found by: Coverity Prevent(tm)
Coverity ID: CID780
Sponsored by: TCP/IP Optimization Fundraise 2005
Diffstat (limited to 'sys/net/if_ef.c')
-rw-r--r-- | sys/net/if_ef.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/net/if_ef.c b/sys/net/if_ef.c index 04262c6..19c1687 100644 --- a/sys/net/if_ef.c +++ b/sys/net/if_ef.c @@ -247,7 +247,7 @@ ef_inputEII(struct mbuf *m, struct ether_header *eh, u_short ether_type) #endif #ifdef INET case ETHERTYPE_IP: - if (ip_fastforward(m)) + if ((m = ip_fastforward(m)) == NULL) return (0); isr = NETISR_IP; break; |