]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
staging: rtl8723bs: use standard offsetof in cfg80211 operations
authorNavaneeth K <knavaneeth786@gmail.com>
Tue, 25 Nov 2025 11:20:56 +0000 (11:20 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 26 Nov 2025 13:21:02 +0000 (14:21 +0100)
Replace usage of the custom FIELD_OFFSET macro with the standard
offsetof() macro in ioctl_cfg80211.c. This improves code readability
and uses the kernel's standard mechanism.

Also include <linux/stddef.h> in basic_types.h to ensure offsetof()
is available for this and future conversions.

Signed-off-by: Navaneeth K <knavaneeth786@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://patch.msgid.link/20251125112059.16913-3-knavaneeth786@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/include/basic_types.h
drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c

index 1c2da18e62100d56a3eab367b6be448985295ac2..16b270fe0203db60cbeed1f2f283747a98867e35 100644 (file)
@@ -12,6 +12,7 @@
 #define FAIL   (-1)
 
 #include <linux/types.h>
+#include <linux/stddef.h>
 
 #define FIELD_OFFSET(s, field) ((__kernel_ssize_t)&((s *)(0))->field)
 
index 315bab3737294bb06df6da8435db881a3e8ea03a..60edeae1cffe7137129c9d91451bb8cfb5655df5 100644 (file)
@@ -1712,7 +1712,8 @@ static int cfg80211_rtw_connect(struct wiphy *wiphy, struct net_device *ndev,
 
                if (wep_key_len > 0) {
                        wep_key_len = wep_key_len <= 5 ? 5 : 13;
-                       wep_total_len = wep_key_len + FIELD_OFFSET(struct ndis_802_11_wep, key_material);
+                       wep_total_len = wep_key_len +
+                               offsetof(struct ndis_802_11_wep, key_material);
                        pwep = rtw_malloc(wep_total_len);
                        if (!pwep) {
                                ret = -ENOMEM;