Two FEATURE_CLEAN_UP patches from Shaun Jackman, closing filehandles.
This commit is contained in:
parent
d00b3a5e94
commit
6417564eeb
@ -64,6 +64,7 @@ int xconnect(struct sockaddr_in *s_addr)
|
|||||||
int s = socket(AF_INET, SOCK_STREAM, 0);
|
int s = socket(AF_INET, SOCK_STREAM, 0);
|
||||||
if (connect(s, (struct sockaddr_in *)s_addr, sizeof(struct sockaddr_in)) < 0)
|
if (connect(s, (struct sockaddr_in *)s_addr, sizeof(struct sockaddr_in)) < 0)
|
||||||
{
|
{
|
||||||
|
if (ENABLE_FEATURE_CLEAN_UP) close(s);
|
||||||
bb_perror_msg_and_die("Unable to connect to remote host (%s)",
|
bb_perror_msg_and_die("Unable to connect to remote host (%s)",
|
||||||
inet_ntoa(s_addr->sin_addr));
|
inet_ntoa(s_addr->sin_addr));
|
||||||
}
|
}
|
||||||
|
@ -329,8 +329,7 @@ static void INET_setroute(int action, char **args)
|
|||||||
bb_perror_msg_and_die("SIOC[ADD|DEL]RT");
|
bb_perror_msg_and_die("SIOC[ADD|DEL]RT");
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Don't bother closing, as we're exiting after we return anyway. */
|
if (ENABLE_FEATURE_CLEAN_UP) close(skfd);
|
||||||
/* close(skfd); */
|
|
||||||
}
|
}
|
||||||
|
|
||||||
#ifdef CONFIG_FEATURE_IPV6
|
#ifdef CONFIG_FEATURE_IPV6
|
||||||
@ -440,8 +439,7 @@ static void INET6_setroute(int action, char **args)
|
|||||||
bb_perror_msg_and_die("SIOC[ADD|DEL]RT");
|
bb_perror_msg_and_die("SIOC[ADD|DEL]RT");
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Don't bother closing, as we're exiting after we return anyway. */
|
if (ENABLE_FEATURE_CLEAN_UP) close(skfd);
|
||||||
/* close(skfd); */
|
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user