Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/netinet Only check that the offset < sizeof(struct ip) i...



details:   https://anonhg.NetBSD.org/src/rev/636c55a556a7
branches:  trunk
changeset: 804807:636c55a556a7
user:      christos <christos%NetBSD.org@localhost>
date:      Sun Nov 30 18:15:41 2014 +0000

description:
Only check that the offset < sizeof(struct ip) if nxt != 0, i.e. in the
tcp and udp cases. From kre.
XXX: pullup 7

diffstat:

 sys/netinet/in4_cksum.c |  11 ++++++-----
 1 files changed, 6 insertions(+), 5 deletions(-)

diffs (39 lines):

diff -r 234b11f9734a -r 636c55a556a7 sys/netinet/in4_cksum.c
--- a/sys/netinet/in4_cksum.c   Sun Nov 30 15:54:34 2014 +0000
+++ b/sys/netinet/in4_cksum.c   Sun Nov 30 18:15:41 2014 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: in4_cksum.c,v 1.19 2013/03/12 21:54:36 christos Exp $  */
+/*     $NetBSD: in4_cksum.c,v 1.20 2014/11/30 18:15:41 christos Exp $  */
 
 /*-
  * Copyright (c) 2008 Joerg Sonnenberger <joerg%NetBSD.org@localhost>.
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: in4_cksum.c,v 1.19 2013/03/12 21:54:36 christos Exp $");
+__KERNEL_RCSID(0, "$NetBSD: in4_cksum.c,v 1.20 2014/11/30 18:15:41 christos Exp $");
 
 #include <sys/param.h>
 #include <sys/mbuf.h>
@@ -58,9 +58,6 @@
        uint32_t sum;
        uint16_t *w;
 
-       if (__predict_false(off < sizeof(struct ip)))
-               PANIC("%s: offset %d too short for IP header %zu", __func__,
-                   off, sizeof(struct ip));
        if (__predict_false(m->m_len < sizeof(struct ip)))
                PANIC("%s: mbuf %d too short for IP header %zu", __func__,
                    m->m_len, sizeof(struct ip));
@@ -68,6 +65,10 @@
        if (nxt == 0)
                return cpu_in_cksum(m, len, off, 0);
 
+       if (__predict_false(off < sizeof(struct ip)))
+               PANIC("%s: offset %d too short for IP header %zu", __func__,
+                   off, sizeof(struct ip));
+
        /*
         * Compute the equivalent of:
         * struct ipovly ip;



Home | Main Index | Thread Index | Old Index