diff options
| author | Kalle Valo <kvalo@kernel.org> | 2023-06-13 17:09:18 +0300 |
|---|---|---|
| committer | Kalle Valo <kvalo@kernel.org> | 2023-06-16 12:24:16 +0300 |
| commit | 707a13c7e488785170a5e7f2467f2823824651e2 (patch) | |
| tree | ff86f6b972dee6bc9cf20ad229340e5ca73aa1ea /drivers/net/wireless/legacy | |
| parent | 5ea0c313090046d84064dc556983d8ed7e6065dd (diff) | |
| download | linux-707a13c7e488785170a5e7f2467f2823824651e2.tar.gz linux-707a13c7e488785170a5e7f2467f2823824651e2.tar.bz2 linux-707a13c7e488785170a5e7f2467f2823824651e2.zip | |
wifi: ray_cs: fix stringop-truncation GCC warning
GCC 12.2 with W=1 warns:
drivers/net/wireless/legacy/ray_cs.c:630:17: warning: 'strncpy' specified bound 32 equals destination size [-Wstringop-truncation]
The driver uses SSID as a string which is just wrong, it should be treated as a
byte array instead. But as the driver is ancient and most likely there are no
users so convert it to use strscpy(). This makes sure that the string is
NUL-terminated and also the warning is fixed.
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20230613140918.389690-5-kvalo@kernel.org
Diffstat (limited to 'drivers/net/wireless/legacy')
| -rw-r--r-- | drivers/net/wireless/legacy/ray_cs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/legacy/ray_cs.c b/drivers/net/wireless/legacy/ray_cs.c index 4b53a9c70e7e..8ace797ce951 100644 --- a/drivers/net/wireless/legacy/ray_cs.c +++ b/drivers/net/wireless/legacy/ray_cs.c @@ -627,7 +627,7 @@ static void init_startup_params(ray_dev_t *local) local->sparm.b4.a_acting_as_ap_status = TYPE_STA; if (essid != NULL) - strncpy(local->sparm.b4.a_current_ess_id, essid, ESSID_SIZE); + strscpy(local->sparm.b4.a_current_ess_id, essid, ESSID_SIZE); } /* init_startup_params */ /*===========================================================================*/ |