>From e6db5cbcc2edcc514aef10eaf63d108d1448f5df Mon Sep 17 00:00:00 2001 From: KO Myung-Hun Date: Wed, 3 Sep 2014 20:47:02 +0900 Subject: [PATCH] freopen: workaround freopen() on OS/2 kLIBC On OS/2 kLIBC, freopen() returns NULL even if it is successful if filename is NULL. * lib/freopen-safer.c (klibc_freopen): New. * lib/freopen.c (orig_freopen): Workaround. --- lib/freopen-safer.c | 24 ++++++++++++++++++++++++ lib/freopen.c | 16 +++++++++++++++- 2 files changed, 39 insertions(+), 1 deletion(-) diff --git a/lib/freopen-safer.c b/lib/freopen-safer.c index 714dc1d..b889123 100644 --- a/lib/freopen-safer.c +++ b/lib/freopen-safer.c @@ -44,6 +44,30 @@ protect_fd (int fd) return true; } +#ifdef __KLIBC__ +/* Workaround for freopen() on OS/2 kLIBC */ +static FILE * +klibc_freopen (const char *filename, const char *mode, FILE *stream) +{ + FILE *result; + + /* Clear errno to check the success of freopen() with it */ + errno = 0; + + result = freopen (filename, mode, stream); + + /* On OS/2 kLIBC, freopen() returns NULL even if it is successful + if filename is NULL. */ + if (!filename && !result && !errno) + result = stream; + + return result; +} + +# undef freopen +# define freopen klibc_freopen +#endif + /* Like freopen, but guarantee that reopening stdin, stdout, or stderr preserves the invariant that STDxxx_FILENO==fileno(stdxxx), and that no other stream will interfere with the standard streams. diff --git a/lib/freopen.c b/lib/freopen.c index 384eba6..4669773 100644 --- a/lib/freopen.c +++ b/lib/freopen.c @@ -29,7 +29,21 @@ static FILE * orig_freopen (const char *filename, const char *mode, FILE *stream) { - return freopen (filename, mode, stream); + FILE *result; + + /* Clear errno to check the success of freopen() with it */ + errno = 0; + + result = freopen (filename, mode, stream); + +#ifdef __KLIBC__ + /* On OS/2 kLIBC, freopen() returns NULL even if it is successful + if filename is NULL. */ + if (!filename && !result && !errno) + result = stream; +#endif + + return result; } /* Specification. */ -- 1.8.5.2