>you are absolutely right that the code was correct at the time
>it was committed -- i think i tried to mention this.
>
>The question is where to move it now. Because it seems to be only
>necessary when we have fragmentation, perhaps the correct place is
>somewhere near the place where fragments are created (around line
>830 in 4.2):
>
>        len = (ifp->if_mtu - hlen) &~ 7;
>        if (len < 8) {
>                error = EMSGSIZE;
>                goto bad;
>        }
>---> move check here <----
>        /*
>         * if the interface will not calculate checksums on
>         * fragmented packets, then do it here.
>         */
>
>so we get as much error reports as possible -- what do you think ?

  If it works there, then that's fine by me.

-DG

David Greenman
Co-founder, The FreeBSD Project - http://www.freebsd.org
President, TeraSolutions, Inc. - http://www.terasolutions.com
Pave the road of life with opportunities.


To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-net" in the body of the message

Reply via email to