90 lines
1.9 KiB
Diff
90 lines
1.9 KiB
Diff
diff -ur ppp-2.4.3.old/pppd/main.c ppp-2.4.3/pppd/main.c
|
|
--- ppp-2.4.3.old/pppd/main.c 2005-06-18 17:48:50.000000000 +0200
|
|
+++ ppp-2.4.3/pppd/main.c 2005-06-20 11:03:43.000000000 +0200
|
|
@@ -1457,14 +1457,6 @@
|
|
int fd, pipefd[2];
|
|
char buf[1];
|
|
|
|
- /* make sure fds 0, 1, 2 are occupied (probably not necessary) */
|
|
- while ((fd = dup(fd_devnull)) >= 0) {
|
|
- if (fd > 2) {
|
|
- close(fd);
|
|
- break;
|
|
- }
|
|
- }
|
|
-
|
|
if (pipe(pipefd) == -1)
|
|
pipefd[0] = pipefd[1] = -1;
|
|
pid = fork();
|
|
@@ -1487,24 +1479,30 @@
|
|
tdb_close(pppdb);
|
|
#endif
|
|
|
|
- /* make sure infd, outfd and errfd won't get tromped on below */
|
|
- if (infd == 1 || infd == 2)
|
|
- infd = dup(infd);
|
|
- if (outfd == 0 || outfd == 2)
|
|
- outfd = dup(outfd);
|
|
- if (errfd == 0 || errfd == 1)
|
|
- errfd = dup(errfd);
|
|
+ /* make sure fds 0, 1, 2 are occupied, so the duplicated fds always > 2 */
|
|
+ while ((fd = dup(fd_devnull)) >= 0) {
|
|
+ if (fd > 2) {
|
|
+ close(fd);
|
|
+ break;
|
|
+ }
|
|
+ }
|
|
|
|
- /* dup the in, out, err fds to 0, 1, 2 */
|
|
- if (infd != 0)
|
|
- dup2(infd, 0);
|
|
- if (outfd != 1)
|
|
- dup2(outfd, 1);
|
|
- if (errfd != 2)
|
|
- dup2(errfd, 2);
|
|
+ /* always copy fd's to avoid to use a already closed fd later */
|
|
+ {
|
|
+ int fdi = infd, fdo = outfd;
|
|
|
|
+ infd = dup(infd);
|
|
+ outfd = dup(outfd);
|
|
+ if (errfd >= 0) {
|
|
+ fd = errfd;
|
|
+ errfd = dup(errfd);
|
|
+ close(fd);
|
|
+ }
|
|
+ close(fdi);
|
|
+ close(fdo);
|
|
+ }
|
|
closelog();
|
|
- if (log_to_fd > 2)
|
|
+ if (log_to_fd >= 0)
|
|
close(log_to_fd);
|
|
if (the_channel->close)
|
|
(*the_channel->close)();
|
|
@@ -1512,12 +1510,18 @@
|
|
close(devfd); /* some plugins don't have a close function */
|
|
close(fd_ppp);
|
|
close(fd_devnull);
|
|
- if (infd != 0)
|
|
- close(infd);
|
|
- if (outfd != 1)
|
|
- close(outfd);
|
|
- if (errfd != 2)
|
|
+
|
|
+ close(0);
|
|
+ dup2(infd, 0);
|
|
+ close(infd);
|
|
+ close(1);
|
|
+ dup2(outfd, 1);
|
|
+ close(outfd);
|
|
+ if (errfd >= 0) {
|
|
+ close(2);
|
|
+ dup2(errfd, 2);
|
|
close(errfd);
|
|
+ }
|
|
|
|
notify(fork_notifier, 0);
|
|
close(pipefd[0]);
|
|
|