DGNum/hostapd
6
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

Clean up the closed stdin/stdout/stderr workaround to close sockets

Browse source 
Close the workaround sockets when wpa_supplicant exists to avoid
hitting resource leak warnings.

Signed-hostap: Jouni Malinen <j@w1.fi>
This commit is contained in:
Jouni Malinen 2012-08-19 18:01:04 +03:00
parent 3489cfb09c
commit 392824ef10
1 changed files with 20 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
wpa_supplicant/main.c
View file

@ -101,20 +101,31 @@ static void license(void)
}
static void wpa_supplicant_fd_workaround(void)
static void wpa_supplicant_fd_workaround(int start)
{
#ifdef __linux__
int s, i;
static int fd[3] = { -1, -1, -1 };
int i;
/* When started from pcmcia-cs scripts, wpa_supplicant might start with
* fd 0, 1, and 2 closed. This will cause some issues because many
* places in wpa_supplicant are still printing out to stdout. As a
* workaround, make sure that fd's 0, 1, and 2 are not used for other
* sockets. */
for (i = 0; i < 3; i++) {
s = open("/dev/null", O_RDWR);
if (s > 2) {
close(s);
break;
if (start) {
for (i = 0; i < 3; i++) {
fd[i] = open("/dev/null", O_RDWR);
if (fd[i] > 2) {
close(fd[i]);
fd[i] = -1;
break;
}
}
} else {
for (i = 0; i < 3; i++) {
if (fd[i] >= 0) {
close(fd[i]);
fd[i] = -1;
}
}
}
#endif /* __linux__ */
@ -140,7 +151,7 @@ int main(int argc, char *argv[])
return -1;
iface_count = 1;
wpa_supplicant_fd_workaround();
wpa_supplicant_fd_workaround(1);
for (;;) {
c = getopt(argc, argv,
@ -288,6 +299,7 @@ int main(int argc, char *argv[])
wpa_supplicant_deinit(global);
out:
wpa_supplicant_fd_workaround(0);
os_free(ifaces);
os_free(params.pid_file);