diff options
Diffstat (limited to 'file_io')
-rw-r--r-- | file_io/os2/filedup.c | 8 | ||||
-rw-r--r-- | file_io/unix/filedup.c | 6 | ||||
-rw-r--r-- | file_io/win32/filedup.c | 16 |
3 files changed, 13 insertions, 17 deletions
diff --git a/file_io/os2/filedup.c b/file_io/os2/filedup.c index eec3229e6..940657c23 100644 --- a/file_io/os2/filedup.c +++ b/file_io/os2/filedup.c @@ -113,11 +113,7 @@ APR_DECLARE(apr_status_t) apr_file_dup(apr_file_t **new_file, apr_file_t *old_fi -APR_DECLARE(apr_status_t) apr_file_dup2(apr_file_t **new_file, apr_file_t *old_file, apr_pool_t *p) +APR_DECLARE(apr_status_t) apr_file_dup2(apr_file_t *new_file, apr_file_t *old_file, apr_pool_t *p) { - if (*new_file == NULL) { - return APR_EINVAL; - } - - return file_dup(new_file, old_file, p); + return file_dup(&new_file, old_file, p); } diff --git a/file_io/unix/filedup.c b/file_io/unix/filedup.c index 7cda84fd3..4b7fb76da 100644 --- a/file_io/unix/filedup.c +++ b/file_io/unix/filedup.c @@ -138,13 +138,13 @@ APR_DECLARE(apr_status_t) apr_file_dup(apr_file_t **new_file, } -APR_DECLARE(apr_status_t) apr_file_dup2(apr_file_t **new_file, +APR_DECLARE(apr_status_t) apr_file_dup2(apr_file_t *new_file, apr_file_t *old_file, apr_pool_t *p) { #ifdef NETWARE - return _file_dup(new_file, old_file, p, 1); + return _file_dup(&new_file, old_file, p, 1); #else - return _file_dup(new_file, old_file, p, 2); + return _file_dup(&new_file, old_file, p, 2); #endif } diff --git a/file_io/win32/filedup.c b/file_io/win32/filedup.c index 22a728d8f..c4717d8cc 100644 --- a/file_io/win32/filedup.c +++ b/file_io/win32/filedup.c @@ -94,7 +94,7 @@ APR_DECLARE(apr_status_t) apr_file_dup(apr_file_t **new_file, } -APR_DECLARE(apr_status_t) apr_file_dup2(apr_file_t **new_file, +APR_DECLARE(apr_status_t) apr_file_dup2(apr_file_t *new_file, apr_file_t *old_file, apr_pool_t *p) { DWORD stdhandle = -1; @@ -134,17 +134,17 @@ APR_DECLARE(apr_status_t) apr_file_dup2(apr_file_t **new_file, FALSE, DUPLICATE_SAME_ACCESS)) { return apr_get_os_error(); } - if ((*new_file)->filehand) { - CloseHandle((*new_file)->filehand); + if (new_file->filehand) { + CloseHandle(new_file->filehand); } newflags = old_file->flags & ~APR_INHERIT; } - (*new_file)->flags = newflags; - (*new_file)->filehand = newhand; - (*new_file)->fname = apr_pstrdup((*new_file)->cntxt, old_file->fname); - (*new_file)->append = old_file->append; - (*new_file)->buffered = FALSE; + new_file->flags = newflags; + new_file->filehand = newhand; + new_file->fname = apr_pstrdup(new_file->cntxt, old_file->fname); + new_file->append = old_file->append; + new_file->buffered = FALSE; return APR_SUCCESS; } |