Hi Paul,

The fix I sent to list was just to make KLIPS stop crashing my kernel, it was 
not a final fix since it overwrote sock parameter skc_daddr: Foreign IPv4 addr, 
so I changed my fix to save the key_pid into the void * sk_user_data, seems 
like a better alternative.

/Ofer

-----Original Message-----
From: Paul Wouters [mailto:[email protected]] 
Sent: Monday, May 30, 2016 6:36 PM
To: Ofer Heifetz <[email protected]>
Cc: [email protected]
Subject: Re: [Swan] KLIPS pfkey_create kernel oops v3.17

On Fri, 27 May 2016, Ofer Heifetz wrote:

> To overcome this I simply changed the :
> # define key_pid(sk) ((struct key_opt*)&(sk))->key_pid
> To:
> # define key_pid(sk) ((struct key_opt*)(sk))->key_pid
>
> key_pid should be saved in sock struct w/o overwriting a valuable field.

Thanks for the patch, will be in 3.18.

Paul

Attachment: 0001-KLIPS-fix-sk-pointer-usage-for-key_pid.patch
Description: 0001-KLIPS-fix-sk-pointer-usage-for-key_pid.patch

_______________________________________________
Swan mailing list
[email protected]
https://lists.libreswan.org/mailman/listinfo/swan

Reply via email to