Re: (KAME-snap 8809) Patch for fragment problem in key.c
From: George V. Neville-Neil (gnn_at_neville-neil.com)
Date: 09/28/04
- Previous message: ming fu: "Re: reproductible hang with if_em"
- In reply to: George V. Neville-Neil: "Patch for fragment problem in key.c"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Date: Tue, 28 Sep 2004 23:28:50 +0900 To: freebsd-net@freebsd.org, rwatson@freebsd.org
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 */
>
_______________________________________________
freebsd-net@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-net
To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org"
- Previous message: ming fu: "Re: reproductible hang with if_em"
- In reply to: George V. Neville-Neil: "Patch for fragment problem in key.c"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]