Netdev List
 help / color / mirror / Atom feed
From: Steffen Klassert <steffen.klassert@secunet.com>
To: "Jérémy Jean" <Jeremy.Jean@oss.cyber.gouv.fr>
Cc: Herbert Xu <herbert@gondor.apana.org.au>,
	"David S . Miller" <davem@davemloft.net>,
	Sabrina Dubroca <sd@queasysnail.net>,
	Saeed Mahameed <saeedm@nvidia.com>,
	Leon Romanovsky <leon@kernel.org>,
	Tariq Toukan <tariqt@nvidia.com>, Mark Bloch <mbloch@nvidia.com>,
	Boris Pismenny <borisp@nvidia.com>, <netdev@vger.kernel.org>
Subject: Re: [PATCH ipsec 0/7] xfrm: fix ESP IV generation and ESN authentication
Date: Thu, 5 Nov 2026 08:21:38 +0100	[thread overview]
Message-ID: <auwuggdz_TCwtlqN@gauss4> (raw)
In-Reply-To: <20260930144523.435271-2-Jeremy.Jean@oss.cyber.gouv.fr>

On Wed, Sep 30, 2026 at 02:45:17PM +0000, Jérémy Jean wrote:
> Hello,
> 
> This series fixes nonce reuse in AES-GCM and ESN authentication errors
> in the IPv4 and IPv6 ESP offload paths:
> * 1/7 and 2/7: Save the current sequence number before incrementing it,
>                so that GCM IV construction prevents nonce reuse,
> * 3/7: Use the current sequence for software AAD and hardware offload,
> * 4/7: Give each early GSO segment a private secpath,
> * 5/7: Use the packet sequence directly for mlx5 IV generation,
> * 6/7: Segment untrusted GSO packets before allocating sequence numbers,
> * 7/7: Keep the sequence counter unchanged after full ESN overflow.
> 
> Note that triggering the bug fixed by 7/7 is impractical: it requires
> processing about 2^64 packets under a single key. Yet, I still believe
> it's cleaner to fix it.
> 
> On the crypto side, as a reminder, reusing a nonce under the same GCM
> key is catastrophic for security: it exposes the XOR of the plaintexts
> and can reveal GCM authentication key, which opens the possibilty for
> an adversary to forge ciphertexts without recovering the AES key.
> 
> This series combines and extends these two individual reports:
> * https://lore.kernel.org/all/20260925095105.446269-2-Jeremy.Jean@oss.cyber.gouv.fr/
> * https://lore.kernel.org/all/20260925095128.446450-2-Jeremy.Jean@oss.cyber.gouv.fr/
> 
> Recent discussion with Sabrina Dubroca may also be of interest:
> https://lore.kernel.org/all/c443bad568f4e03d05b848b1b245707d@oss.cyber.gouv.fr/T/#u
> 
> Regards,
> Jérémy
> 
> ---
> 
> Jérémy Jean (7):
>   xfrm: esp6: use the current sequence number for the IV
>   xfrm: esp4: use the current sequence number for the IV
>   xfrm: esp: use the current sequence number for AAD and offload
>   xfrm: prevent AES-GCM nonce reuse after early GSO
>   net/mlx5e: Use the packet sequence number for the IPsec IV
>   xfrm: segment untrusted GSO packets before sequence allocation
>   xfrm: leave the sequence counter unchanged on ESN overflow

Please rebase your patchset along with the proposed changes to
the current ipsec tree so it can get applied eventually.

Thanks!

  parent reply	other threads:[~2026-11-05  7:21 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-30 14:45 [PATCH ipsec 0/7] xfrm: fix ESP IV generation and ESN authentication Jérémy Jean
2026-09-30 14:45 ` [PATCH ipsec 1/7] xfrm: esp6: use the current sequence number for the IV Jérémy Jean
2026-11-12 10:08   ` Usman Malik
2026-09-30 14:45 ` [PATCH ipsec 2/7] xfrm: esp4: " Jérémy Jean
2026-09-30 14:45 ` [PATCH ipsec 3/7] xfrm: esp: use the current sequence number for AAD and offload Jérémy Jean
2026-09-30 14:45 ` [PATCH ipsec 4/7] xfrm: prevent AES-GCM nonce reuse after early GSO Jérémy Jean
2026-10-01 13:31   ` Sabrina Dubroca
2026-10-01 21:07     ` Jérémy Jean
2026-10-14 14:10       ` Sabrina Dubroca
2026-11-12 10:08   ` Usman Malik
2026-09-30 14:45 ` [PATCH ipsec 5/7] net/mlx5e: Use the packet sequence number for the IPsec IV Jérémy Jean
2026-10-01 11:42   ` Sabrina Dubroca
2026-10-01 19:33     ` Jérémy Jean
2026-10-05 13:06   ` Tariq Toukan
2026-10-05 13:50     ` Jérémy Jean
2026-10-05 18:28       ` Jérémy Jean
2026-10-06  6:49         ` Tariq Toukan
2026-10-06  8:51           ` Jérémy Jean
2026-09-30 14:45 ` [PATCH ipsec 6/7] xfrm: segment untrusted GSO packets before sequence allocation Jérémy Jean
2026-09-30 14:45 ` [PATCH ipsec 7/7] xfrm: leave the sequence counter unchanged on ESN overflow Jérémy Jean
2026-10-01 11:24   ` Sabrina Dubroca
2026-10-01 11:37     ` Jérémy Jean
2026-10-01 11:45       ` Sabrina Dubroca
2026-10-01 11:48         ` Jérémy Jean
2026-09-30 14:48 ` [PATCH ipsec 0/7] xfrm: fix ESP IV generation and ESN authentication netdev-bot+sinfo
2026-11-05  7:21 ` Steffen Klassert [this message]
2026-11-08 14:29   ` Jérémy Jean

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=auwuggdz_TCwtlqN@gauss4 \
    --to=steffen.klassert@secunet.com \
    --cc=Jeremy.Jean@oss.cyber.gouv.fr \
    --cc=borisp@nvidia.com \
    --cc=davem@davemloft.net \
    --cc=herbert@gondor.apana.org.au \
    --cc=leon@kernel.org \
    --cc=mbloch@nvidia.com \
    --cc=netdev@vger.kernel.org \
    --cc=saeedm@nvidia.com \
    --cc=sd@queasysnail.net \
    --cc=tariqt@nvidia.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox