]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
wifi: cfg80211: fix grammar in MLO group key error message
authorLouis Kotze <loukot@gmail.com>
Tue, 14 Apr 2026 12:27:28 +0000 (14:27 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Tue, 28 Apr 2026 07:27:08 +0000 (09:27 +0200)
The error message emitted by nl80211_validate_key_link_id() when a group
key install on an MLO wdev is missing the link ID reads "link ID must
for MLO group key", which is missing the words "be set". This makes the
error harder to grep and parse in userspace logs, and is reported
verbatim by wpa_supplicant via its nl80211 extack relay, e.g.:

  wpa_supplicant: nl80211: kernel reports: link ID must for MLO group key

The sibling error strings in the same helper already use grammatical
phrasing ("link ID not allowed for pairwise key", "invalid link ID for
MLO group key", "link ID not allowed for non-MLO group key"). Fix this
one to match.

No functional change.

Fixes: e7a7b84e3317 ("wifi: cfg80211: Add link_id parameter to various key operations for MLO")
Signed-off-by: Louis Kotze <loukot@gmail.com>
Link: https://patch.msgid.link/20260414122728.92234-1-loukot@gmail.com
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/wireless/nl80211.c

index f334cdef895870b5af58d45241dc90dd49fcf314..b0bb94f29e563b3505149a1073ffd0ce4db147dc 100644 (file)
@@ -5100,7 +5100,7 @@ static int nl80211_validate_key_link_id(struct genl_info *info,
        if (wdev->valid_links) {
                if (link_id == -1) {
                        GENL_SET_ERR_MSG(info,
-                                        "link ID must for MLO group key");
+                                        "link ID must be set for MLO group key");
                        return -EINVAL;
                }
                if (!(wdev->valid_links & BIT(link_id))) {