Commit ee64d05a authored by Eric Dumazet's avatar Eric Dumazet Committed by Yongqiang Liu
Browse files

tcp_metrics: fix data-race in tcpm_suck_dst() vs fastopen

stable inclusion
from stable-v4.19.291
commit 48f0ecf5be777bcd6b83dc0193784406364419c7
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/I7ZELT


CVE: NA

--------------------------------

[ Upstream commit ddf251fa ]

Whenever tcpm_new() reclaims an old entry, tcpm_suck_dst()
would overwrite data that could be read from tcp_fastopen_cache_get()
or tcp_metrics_fill_info().

We need to acquire fastopen_seqlock to maintain consistency.

For newly allocated objects, tcpm_new() can switch to kzalloc()
to avoid an extra fastopen_seqlock acquisition.

Fixes: 1fe4c481 ("net-tcp: Fast Open client - cookie cache")
Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
Cc: Yuchung Cheng <ycheng@google.com>
Reviewed-by: default avatarKuniyuki Iwashima <kuniyu@amazon.com>
Link: https://lore.kernel.org/r/20230802131500.1478140-7-edumazet@google.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
Signed-off-by: default avatarYongqiang Liu <liuyongqiang13@huawei.com>
parent b79f130d
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment