netdev - Re: [PATCH 1/4] xfrm: Add support for per cpu xfrm state handling.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <d514ef1a-bf56-4949-91f1-c64e3f599922@ijzerbout.nl>
Date: Tue, 12 Nov 2024 20:21:07 +0100
From: Kees Bakker <kees@...erbout.nl>
To: Steffen Klassert <steffen.klassert@...unet.com>
Cc: netdev@...r.kernel.org
Subject: Re: [PATCH 1/4] xfrm: Add support for per cpu xfrm state handling.

Op 12-11-2024 om 12:03 schreef Steffen Klassert:
> On Mon, Nov 11, 2024 at 09:42:02PM +0100, Kees Bakker wrote:
>> Hi Steffen,
>>
>> Sorry for the direct email. Did you perhaps forgot a "goto out_cancel" here?
> Yes, looks like that. Do you want to send a patch?
I prefer that you create the patch.
>
>> diff --git a/net/xfrm/xfrm_user.c b/net/xfrm/xfrm_user.c
>> [...]
>> @@ -2576,6 +2603,8 @@ static int build_aevent(struct sk_buff *skb, struct
>> xfrm_state *x, const struct
>>       err = xfrm_if_id_put(skb, x->if_id);
>>       if (err)
>>           goto out_cancel;
>> +    if (x->pcpu_num != UINT_MAX)
>> +        err = nla_put_u32(skb, XFRMA_SA_PCPU, x->pcpu_num);
>>
>>       if (x->dir) {
>>           err = nla_put_u8(skb, XFRMA_SA_DIR, x->dir);
>>
>> -- 
>> Kees Bakker