diff options
| author | Benjamin Berg <benjamin.berg@intel.com> | 2023-09-25 18:18:56 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-10-25 13:03:11 +0300 |
| commit | 2353f64762330cef00636b3584f413233a77c5a1 (patch) | |
| tree | ac938e8e0197fcaf8abe3b41b4f0972178949859 | |
| parent | 2fd88f5632e193b7bba97e50ea746bfe15b28940 (diff) | |
| download | linux-2353f64762330cef00636b3584f413233a77c5a1.tar.xz | |
wifi: cfg80211: avoid leaking stack data into trace
[ Upstream commit 334bf33eec5701a1e4e967bcb7cc8611a998334b ]
If the structure is not initialized then boolean types might be copied
into the tracing data without being initialised. This causes data from
the stack to leak into the trace and also triggers a UBSAN failure which
can easily be avoided here.
Signed-off-by: Benjamin Berg <benjamin.berg@intel.com>
Link: https://lore.kernel.org/r/20230925171855.a9271ef53b05.I8180bae663984c91a3e036b87f36a640ba409817@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | net/wireless/nl80211.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c index 1d993a490ac4..b19b5acfaf3a 100644 --- a/net/wireless/nl80211.c +++ b/net/wireless/nl80211.c @@ -8289,7 +8289,7 @@ static int nl80211_update_mesh_config(struct sk_buff *skb, struct cfg80211_registered_device *rdev = info->user_ptr[0]; struct net_device *dev = info->user_ptr[1]; struct wireless_dev *wdev = dev->ieee80211_ptr; - struct mesh_config cfg; + struct mesh_config cfg = {}; u32 mask; int err; |