Re: [PATCH] IPSec fixes

From: Hajimu UMEMOTO <ume_at_FreeBSD.org>
Date: Fri, 16 Jan 2004 19:09:03 +0900
Hi,

>>>>> On Fri, 16 Jan 2004 09:31:25 +0000 (UTC)
>>>>> bzeeb-lists_at_lists.zabbadoz.net ("Bjoern A. Zeeb") said:

bzeeb-lists> I never applied it as going through the code showed that it will not
bzeeb-lists> do the right thing(TM). See older mail.

You wrote that the check is entirely useless.  However, the patch
actually changed the behavior and fixed the problem on my box.
Without the patch, I could reproduce a panic easily by setkey.
So, I wish to hear an actuall result on your box.

bzeeb-lists> ahh - this isn't exactly the diff itojun offered.
bzeeb-lists> the newsp->refcnt++; is missing. Is it also missing in your kernel ?

Yes.  Since you pointed that this is wrong, and itojun agreed with it.
So, I nuked this line.

bzeeb-lists> PS: do you have other patches in your local tree that are not yet
bzeeb-lists> in HEAD ? I got another patch from you once in Nov or Dec but I think
bzeeb-lists> it was renaming functions and passing pcb instead of so only (removing
bzeeb-lists> some unnecessary function calls) [more like FATS_IPSEC]. I have a more
bzeeb-lists> improved version of this in my private patchset but currently
bzeeb-lists> suspended while debugging.

No, I'm not applying any other patches.
Originally, it was jlemon's work.  It was withdrawed during KAME
merge.  When this issue is fixed, I'll rework about this.

Sincerely,

--
Hajimu UMEMOTO _at_ Internet Mutual Aid Society Yokohama, Japan
ume_at_mahoroba.org  ume_at_bisd.hitachi.co.jp  ume_at_{,jp.}FreeBSD.org
http://www.imasy.org/~ume/
Received on Fri Jan 16 2004 - 01:09:13 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:37:38 UTC