public-mirrors/linux
1
0
Fork 0
mirror of git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git synced 2025-09-18 22:14:16 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

xfrm: validate assignment of maximal possible SEQ number

Browse source 
Users can set any seq/seq_hi/oseq/oseq_hi values. The XFRM core code
doesn't prevent from them to set even 0xFFFFFFFF, however this value
will cause for traffic drop.

Is is happening because SEQ numbers here mean that packet with such
number was processed and next number should be sent on the wire. In this
case, the next number will be 0, and it means overflow which causes to
(expected) packet drops.

While it can be considered as misconfiguration and handled by XFRM
datapath in the same manner as any other SEQ number, let's add
validation to easy for packet offloads implementations which need to
configure HW with next SEQ to send and not with current SEQ like it is
done in core code.

Signed-off-by: Leon Romanovsky <leonro@nvidia.com>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
This commit is contained in:
Leon Romanovsky 2025-05-13 13:56:22 +03:00 • committed by Steffen Klassert
parent 86e530c84c
commit e86212b6b1
1 changed files with 42 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

52
net/xfrm/xfrm_user.c
View file

@ -178,11 +178,27 @@ static inline int verify_replay(struct xfrm_usersa_info *p,
"Replay seq and seq_hi should be 0 for output SA");
return -EINVAL;
}
if (rs->oseq_hi && !(p->flags & XFRM_STATE_ESN)) {
NL_SET_ERR_MSG(
extack,
"Replay oseq_hi should be 0 in non-ESN mode for output SA");
return -EINVAL;
if (!(p->flags & XFRM_STATE_ESN)) {
if (rs->oseq_hi) {
NL_SET_ERR_MSG(
extack,
"Replay oseq_hi should be 0 in non-ESN mode for output SA");
return -EINVAL;
}
if (rs->oseq == U32_MAX) {
NL_SET_ERR_MSG(
extack,
"Replay oseq should be less than 0xFFFFFFFF in non-ESN mode for output SA");
return -EINVAL;
}
} else {
if (rs->oseq == U32_MAX && rs->oseq_hi == U32_MAX) {
NL_SET_ERR_MSG(
extack,
"Replay oseq and oseq_hi should be less than 0xFFFFFFFF for output SA");
return -EINVAL;
}
}
if (rs->bmp_len) {
NL_SET_ERR_MSG(extack, "Replay bmp_len should 0 for output SA");
@ -196,11 +212,27 @@ static inline int verify_replay(struct xfrm_usersa_info *p,
"Replay oseq and oseq_hi should be 0 for input SA");
return -EINVAL;
}
if (rs->seq_hi && !(p->flags & XFRM_STATE_ESN)) {
NL_SET_ERR_MSG(
extack,
"Replay seq_hi should be 0 in non-ESN mode for input SA");
return -EINVAL;
if (!(p->flags & XFRM_STATE_ESN)) {
if (rs->seq_hi) {
NL_SET_ERR_MSG(
extack,
"Replay seq_hi should be 0 in non-ESN mode for input SA");
return -EINVAL;
}
if (rs->seq == U32_MAX) {
NL_SET_ERR_MSG(
extack,
"Replay seq should be less than 0xFFFFFFFF in non-ESN mode for input SA");
return -EINVAL;
}
} else {
if (rs->seq == U32_MAX && rs->seq_hi == U32_MAX) {
NL_SET_ERR_MSG(
extack,
"Replay seq and seq_hi should be less than 0xFFFFFFFF for input SA");
return -EINVAL;
}
}
}