diff options
| author | Sven Eckelmann <sven@narfation.org> | 2026-06-26 18:10:42 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2026-07-04 13:39:38 +0200 |
| commit | edaadf868d988905a5bc1252ce86b54ffaaa10b6 (patch) | |
| tree | 4b356cdf526ce61ea6bb1b6d7b3924b19e89432c | |
| parent | 06df70f6cbacac0b8acc7f725951c57b395125b4 (diff) | |
| download | linux-stable-edaadf868d988905a5bc1252ce86b54ffaaa10b6.tar.gz linux-stable-edaadf868d988905a5bc1252ce86b54ffaaa10b6.zip | |
batman-adv: tp_meter: initialize dec_cwnd explicitly
commit febfb1b86224489535312296ecfa3d4bf467f339 upstream.
When batadv_tp_update_cwnd() is called, dec_cwnd is increased. But dec_cwnd
is only initialixed (to 0) when a duplicate Ack was received or when cwnd
is below the ss_threshold.
Just initialize the cwnd during the initialization to avoid any potential
access of uninitialized data.
Cc: stable@kernel.org
Fixes: 33a3bb4a3345 ("batman-adv: throughput meter implementation")
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | net/batman-adv/tp_meter.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/batman-adv/tp_meter.c b/net/batman-adv/tp_meter.c index 2b68eacd7bf5..f7b0ce3f8e2a 100644 --- a/net/batman-adv/tp_meter.c +++ b/net/batman-adv/tp_meter.c @@ -1055,6 +1055,8 @@ void batadv_tp_start(struct batadv_priv *bat_priv, const u8 *dst, * soft_interface, hence its MTU */ tp_vars->cwnd = BATADV_TP_PLEN * 3; + tp_vars->dec_cwnd = 0; + /* at the beginning initialise the SS threshold to the biggest possible * window size, hence the AWND size */ |