wifi: ray_cs: fix stringop-truncation GCC warning
authorKalle Valo <kvalo@kernel.org>
Tue, 13 Jun 2023 14:09:18 +0000 (17:09 +0300)
committerKalle Valo <kvalo@kernel.org>
Fri, 16 Jun 2023 09:24:16 +0000 (12:24 +0300)
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
drivers/net/wireless/legacy/ray_cs.c

index 4b53a9c70e7e7f392a32f885ff5c09d39ada9cff..8ace797ce951c501621015f0b93a426c0e1dc2f1 100644 (file)
@@ -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 */
 
 /*===========================================================================*/