So, I checked and the change in Kame is identical.   Can we get this
committed to our code?  It's a simple, five line, removal.

Thanks,
George


At Tue, 28 Sep 2004 10:39:09 +0900,
George V. Neville-Neil wrote:
> 
> Hi Folks,
> 
>       Robert Watson tried to send email about this but it never got
>       through, and then Sam Leffler got ahold of me and told me he
>       fixed something similar in the FAST_IPSEC code.  So, the
>       following patch fixes, in KAME IPSec.  This patch was
>       generated against 6.0-CURRENT and I included Sam's commit
>       message.
> 
> Later,
> George
> 
> Correct handling of SADB_UPDATE and SADB_ADD requests.  key_align may split
> the mbuf due to use of m_pulldown.  Discarding the result because of this
> does not make sense as no subsequent code depends on the entire msg being
> linearized (only the individual pieces).  It's likely something else is wrong
> here but for now this appears to get things back to a working state.
> 
> Index: sys/netkey/key.c
> ===================================================================
> RCS file: /Volumes/exported/FreeBSD-CVS/src/sys/netkey/key.c,v
> retrieving revision 1.67
> diff -u -r1.67 key.c
> --- sys/netkey/key.c  2 Sep 2004 20:14:03 -0000       1.67
> +++ sys/netkey/key.c  27 Sep 2004 16:08:31 -0000
> @@ -6952,11 +6952,6 @@
>       if (error)
>               return error;
>  
> -     if (m->m_next) {        /*XXX*/
> -             m_freem(m);
> -             return ENOBUFS;
> -     }
> -
>       msg = mh.msg;
>  
>       /* check SA type */
> 
_______________________________________________
[EMAIL PROTECTED] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-net
To unsubscribe, send any mail to "[EMAIL PROTECTED]"

Reply via email to