From 46053a4c93555a40d49d548a88023097d79d7289 Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Tue, 15 May 2018 20:40:24 +0300 Subject: [PATCH] wext: Avoid gcc warnings on restricted pointer aliasing The first two parameters to readlink() are marked restricted and at least gcc 8.2 warns about used the same pointer for then, so avoid this by using separate buffers for the pathname and response buffer. Signed-off-by: Jouni Malinen --- src/drivers/driver_wext.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/drivers/driver_wext.c b/src/drivers/driver_wext.c index 659eda2f1..933b8d986 100644 --- a/src/drivers/driver_wext.c +++ b/src/drivers/driver_wext.c @@ -922,7 +922,7 @@ static int wext_add_hostap(struct wpa_driver_wext_data *drv) static void wext_check_hostap(struct wpa_driver_wext_data *drv) { - char buf[200], *pos; + char path[200], buf[200], *pos; ssize_t res; /* @@ -937,9 +937,9 @@ static void wext_check_hostap(struct wpa_driver_wext_data *drv) */ /* First, try to see if driver information is available from sysfs */ - snprintf(buf, sizeof(buf), "/sys/class/net/%s/device/driver", + snprintf(path, sizeof(path), "/sys/class/net/%s/device/driver", drv->ifname); - res = readlink(buf, buf, sizeof(buf) - 1); + res = readlink(path, buf, sizeof(buf) - 1); if (res > 0) { buf[res] = '\0'; pos = strrchr(buf, '/');