qiurui
/
Centos-kernel-stream-9
mirror of https://gitlab.com/redhat/centos-stream/src/kernel/centos-stream-9.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

wifi: cfg80211: sme: init n_channels before channels[] access 

JIRA: https://issues.redhat.com/browse/RHEL-73817

commit f1d3334d604cc32db63f6e2b3283011e02294e54
Author: Haoyu Li <lihaoyu499@gmail.com>
Date:   Tue Dec 3 23:20:49 2024 +0800

    wifi: cfg80211: sme: init n_channels before channels[] access
    
    With the __counted_by annocation in cfg80211_scan_request struct,
    the "n_channels" struct member must be set before accessing the
    "channels" array. Failing to do so will trigger a runtime warning
    when enabling CONFIG_UBSAN_BOUNDS and CONFIG_FORTIFY_SOURCE.
    
    Fixes: e3eac9f32ec0 ("wifi: cfg80211: Annotate struct cfg80211_scan_request with __counted_by")
    Signed-off-by: Haoyu Li <lihaoyu499@gmail.com>
    Link: https://patch.msgid.link/20241203152049.348806-1-lihaoyu499@gmail.com
    Signed-off-by: Johannes Berg <johannes.berg@intel.com>

Signed-off-by: Jose Ignacio Tornos Martinez <jtornosm@redhat.com>
This commit is contained in:
Jose Ignacio Tornos Martinez 2025-01-15 17:26:24 +01:00
parent 2b1b06b690
commit 1d6eefe326
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
net/wireless/sme.c
View File

@ -83,6 +83,7 @@ static int cfg80211_conn_scan(struct wireless_dev *wdev)
if (!request)
return -ENOMEM;
request->n_channels = n_channels;
if (wdev->conn->params.channel) {
enum nl80211_band band = wdev->conn->params.channel->band;
struct ieee80211_supported_band *sband =