summaryrefslogtreecommitdiff
path: root/wireless.c
diff options
context:
space:
mode:
authorHauke Mehrtens <hauke@hauke-m.de>2023-02-12 21:07:31 +0100
committerHauke Mehrtens <hauke@hauke-m.de>2023-02-19 16:08:28 +0100
commit19372d8528d25d9774c1c6ede2642bdf57ce3cc3 (patch)
treed7bdc939ff703af66c9b884d9f28dab1ce0bc2df /wireless.c
parented65a00adc751d102c2041a4a9a32df9dad52b9e (diff)
downloadnetifd-19372d8528d25d9774c1c6ede2642bdf57ce3cc3.tar.gz
netifd: Fix multiple -Wsign-compare warnings
This fixes warnings like this: warning: comparison of integer expressions of different signedness: 'int' and 'long unsigned int' [-Wsign-compare] Mostly this was an int compared to a size_t returned by ARRAY_SIZE(). The easiest fix is to count on the size_t type. The ifindex is sometimes an unsigned int and sometimes a signed int in the kernel interfaces. I think it normally fits into an unsigned 16 bit value, so this should be fine. Do the one comparison where the compiler complains as a long. Casting the result of sizeof() to int should be safe. These values are never out of range of int. Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
Diffstat (limited to 'wireless.c')
-rw-r--r--wireless.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/wireless.c b/wireless.c
index 705d6dd..1f54bfb 100644
--- a/wireless.c
+++ b/wireless.c
@@ -1542,7 +1542,7 @@ void wireless_device_hotplug_event(const char *name, bool add)
struct wireless_interface *vif;
struct wireless_device *wdev;
const char *s;
- int len;
+ size_t len;
s = strstr(name, ".sta");
if (s) {