Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 3 Feb 2014 14:55:55 -0500
From:      Eitan Adler <lists@eitanadler.com>
To:        "freebsd-net@freebsd.org" <freebsd-net@freebsd.org>, swildner@dragonflybsd.org
Subject:   ip6opt.c
Message-ID:  <CAF6rxgmQMXKDMETRPRE-QkQwGS3vgZxQRFKoe6d3_BycQOouYw@mail.gmail.com>

next in thread | raw e-mail | index | archive | help
Hi all,

DragonFly recently committed the following change and it seems that it
applies to us as well.

http://gitweb.dragonflybsd.org/dragonfly.git/blobdiff/5764e12516158974fac10d50dbd2df76ce1ab007..98651c6e0e1c3b7a6b8650b55b473fcc745a22b7:/lib/libc/net/ip6opt.c

Should I commit it?

Index: ip6opt.c
===================================================================
--- ip6opt.c    (revision 261405)
+++ ip6opt.c    (working copy)
@@ -381,11 +381,8 @@ inet6_opt_init(void *extbuf, socklen_t extlen)
 {
        struct ip6_ext *ext = (struct ip6_ext *)extbuf;

-       if (extlen < 0 || (extlen % 8))
-               return(-1);
-
        if (ext) {
-               if (extlen == 0)
+               if (extlen == 0 || (extlen % 8))
                        return(-1);
                ext->ip6e_len = (extlen >> 3) - 1;
        }


-- 
Eitan Adler



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAF6rxgmQMXKDMETRPRE-QkQwGS3vgZxQRFKoe6d3_BycQOouYw>