On Wed, 28 Nov 2007, Nick Hilliard wrote:

Hi,

Bjoern A. Zeeb wrote:
I'll try to find your bug the next days (in case you find anything let
me know).

At the very least, this will be necessary:

--- tcp_subr.c~        2007-11-28 01:14:46.000000000 +0000
+++ tcp_subr.c  2007-11-28 01:14:46.000000000 +0000
@@ -1948,7 +1948,7 @@
       /*
        * Step 4: Update MD5 hash with shared secret.
        */
-       MD5Update(&ctx, _KEYBUF(sav->key_auth), _KEYLEN(sav->key_auth));
+       MD5Update(&ctx, sav->key_auth->key_data, _KEYLEN(sav->key_auth));
       MD5Final(buf, &ctx);

       key_sa_recordxfer(sav, m);

But it doesn't fix the problem.

I stopped at the point validating the tcp header was correct last
night somewhen after midnight. With your + my other patches (not
posted yet) I have valid md5 sums again for all but the S/A case.
Expect a patch soonish.

--
Bjoern A. Zeeb                                 bzeeb at Zabbadoz dot NeT
Software is harder than hardware  so better get it right the first time.
_______________________________________________
freebsd-net@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-net
To unsubscribe, send any mail to "[EMAIL PROTECTED]"

Reply via email to