From 856b2e2c259e94516c9b04e57368018fd44f58cf Mon Sep 17 00:00:00 2001 From: Quincey Koziol Date: Fri, 18 Dec 2009 23:15:36 -0500 Subject: [svn-r18034] Description: Bring r18033 from trunk to 1.8 branch: Correct error in merging Coverity changes in r18011 Tested on: None - eyeballed only (very simple) --- src/H5FDlog.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/H5FDlog.c b/src/H5FDlog.c index bde63cc..0da5765 100644 --- a/src/H5FDlog.c +++ b/src/H5FDlog.c @@ -425,7 +425,7 @@ H5FD_log_fapl_copy(const void *_old_fa) FUNC_ENTER_NOAPI(H5FD_log_fapl_copy, NULL) - HDassert(new_fa); + HDassert(old_fa); /* Allocate the new FAPL info */ if(NULL == (new_fa = (H5FD_log_fapl_t *)H5MM_calloc(sizeof(H5FD_log_fapl_t)))) @@ -471,17 +471,15 @@ static herr_t H5FD_log_fapl_free(void *_fa) { H5FD_log_fapl_t *fa = (H5FD_log_fapl_t*)_fa; - herr_t ret_value = SUCCEED; /* Return value */ - FUNC_ENTER_NOAPI(H5FD_log_fapl_free, FAIL) + FUNC_ENTER_NOAPI_NOINIT_NOFUNC(H5FD_log_fapl_free) /* Free the fapl information */ if(fa->logfile) H5MM_xfree(fa->logfile); H5MM_xfree(fa); -done: - FUNC_LEAVE_NOAPI(ret_value) + FUNC_LEAVE_NOAPI(SUCCEED) } /* end H5FD_log_fapl_free() */ -- cgit v0.12