jeltz
/
hostap
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

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 <jouni@codeaurora.org>
This commit is contained in:
Jouni Malinen 2018-05-15 20:40:24 +03:00 committed by Jouni Malinen
parent ce2a9644bd
commit 46053a4c93
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/drivers/driver_wext.c
View File

@ -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, '/');