From 3244e9a493531c24b414b6e64d3396f466e357aa Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Fri, 11 Aug 2006 20:28:42 +0000 Subject: * pipe-safer.c (pipe_safer): Fix misspelling: HAVE_FUNC_PIPE -> HAVE_PIPE. Fix a file descriptor leak when fd_safer fails. --- lib/ChangeLog | 3 +++ lib/pipe-safer.c | 40 +++++++++++++++++++++------------------- 2 files changed, 24 insertions(+), 19 deletions(-) (limited to 'lib') diff --git a/lib/ChangeLog b/lib/ChangeLog index 78d9b9090..9fa61ac7b 100644 --- a/lib/ChangeLog +++ b/lib/ChangeLog @@ -1,5 +1,8 @@ 2006-08-11 Paul Eggert + * pipe-safer.c (pipe_safer): Fix misspelling: HAVE_FUNC_PIPE -> + HAVE_PIPE. Fix a file descriptor leak when fd_safer fails. + * regex_internal.c (re_string_skip_chars): Don't assume WEOF fits in wchar_t. Problem reported by Eric Blake. diff --git a/lib/pipe-safer.c b/lib/pipe-safer.c index 646cd5dab..81df99454 100644 --- a/lib/pipe-safer.c +++ b/lib/pipe-safer.c @@ -33,25 +33,27 @@ int pipe_safer (int fd[2]) { -#if HAVE_FUNC_PIPE - int fail = pipe (fd); - if (fail) - return fail; - - { - int i; - for (i = 0; i < 2; i++) - { - int f = fd_safer (fd[i]); - if (f < 0) - return -1; - fd[i] = f; - } - } - - return 0; -#else /* ! HAVE_FUNC_PIPE */ +#if HAVE_PIPE + if (pipe (fd) == 0) + { + int i; + for (i = 0; i < 2; i++) + { + fd[i] = fd_safer (fd[i]); + if (fd[i] < 0) + { + int e = errno; + close (fd[1 - i]); + errno = e; + return -1; + } + } + + return 0; + } +#else errno = ENOSYS; - return -1; #endif + + return -1; } -- cgit v1.2.3-54-g00ecf