From: Jouni Malinen Date: Tue, 15 May 2018 17:40:24 +0000 (+0300) Subject: wext: Avoid gcc warnings on restricted pointer aliasing X-Git-Tag: hostap_2_7~344 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=46053a4c93555a40d49d548a88023097d79d7289;p=thirdparty%2Fhostap.git 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 --- 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, '/');