From c5b603522ec0c5b7780e4c5f20086e8b8b946f86 Mon Sep 17 00:00:00 2001 From: Alex Gorrod Date: Wed, 26 Feb 2014 09:00:47 +1100 Subject: Fix compiler warnings re: ignored return values. --- ext/datasources/helium/helium.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'ext') diff --git a/ext/datasources/helium/helium.c b/ext/datasources/helium/helium.c index ee08cbd97b3..24b0f595d62 100644 --- a/ext/datasources/helium/helium.c +++ b/ext/datasources/helium/helium.c @@ -2102,7 +2102,7 @@ helium_session_open_cursor(WT_DATA_SOURCE *wtds, WT_SESSION *session, WT_CURSOR *wtcursor; WT_EXTENSION_API *wtext; WT_SOURCE *ws; - int locked, ret = 0; + int locked, ret, t_ret; const char *value; *new_cursor = NULL; @@ -2113,6 +2113,7 @@ helium_session_open_cursor(WT_DATA_SOURCE *wtds, WT_SESSION *session, wtext = ds->wtext; ws = NULL; locked = 0; + ret = t_ret = 0; value = NULL; /* Allocate and initialize a cursor. */ @@ -2226,8 +2227,9 @@ err: if (ws != NULL && locked) ESET(unlock(wtext, session, &ws->lock)); cursor_destroy(cursor); } - if (config_parser != NULL) - config_parser->close(config_parser); + if (config_parser != NULL && (t_ret = + (void)config_parser->close(config_parser)) != 0 && ret == 0) + ret = t_ret; free((void *)value); return (ret); } -- cgit v1.2.1