Date: Wed, 18 May 2016 17:33:17 -0700 (PDT) From: Don Lewis <truckman@FreeBSD.org> To: cem@FreeBSD.org Cc: ian.freislich@capeaugusta.com, current@freebsd.org Subject: Re: r299512 breaks dhclient on some networks Message-ID: <201605190033.u4J0XHMe080781@gw.catspoiler.org> In-Reply-To: <CAG6CVpXkEQZij-LbUkhL12jf1cyfUwTFR6VBzosMjo4=WaTL3g@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 18 May, Conrad Meyer wrote: > Hey Ian, > > r299512 incorrectly encoded client identifiers because I misunderstood > the intent of the sizeof()-scaled client_id. I reverted that change > and replaced it with r300174, which I believe fixes the first overrun > more correctly. > > (Coverity may still complain about CID 1305550, but I don't believe > it's valid for 'hlen' to exceed sizeof(hw_addr.haddr).) It's not, but the MIN() doesn't hurt. Coverity may no longer complain though because your change may think that hlen is only 16 at this point since that is what the earlier change tests against. If it is checked in one place, it should probably be checked in both, or you could just add an assert() to check it ...
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201605190033.u4J0XHMe080781>