summaryrefslogtreecommitdiff
path: root/api
diff options
context:
space:
mode:
authorMichael Cahill <michael.cahill@wiredtiger.com>2014-06-25 17:18:51 +1000
committerMichael Cahill <michael.cahill@wiredtiger.com>2014-06-25 17:18:51 +1000
commit7d6e2dabd333270dfb78c9a469fdc826dfcb0b79 (patch)
tree7d3e916fef5f2f1624af5937915dd86c3e70b331 /api
parent24984c05c687df86d894ba56e016d504d4a2cfe1 (diff)
downloadmongo-7d6e2dabd333270dfb78c9a469fdc826dfcb0b79.tar.gz
Fix a logic bug releasing cursors after DB::Get operations.
Diffstat (limited to 'api')
-rw-r--r--api/leveldb/leveldb_wt.cc4
1 files changed, 2 insertions, 2 deletions
diff --git a/api/leveldb/leveldb_wt.cc b/api/leveldb/leveldb_wt.cc
index c9f28cf1e05..b6657046f00 100644
--- a/api/leveldb/leveldb_wt.cc
+++ b/api/leveldb/leveldb_wt.cc
@@ -670,8 +670,8 @@ DbImpl::Get(const ReadOptions& options,
} else if (ret == WT_NOTFOUND)
errmsg = "DB::Get key not found";
err:
- // There is no need to release the cursor if we are in a snapshot
- if (si != NULL)
+ // Release the cursor if we are not in a snapshot
+ if (si == NULL)
releaseCursor(cursor);
return WiredTigerErrorToStatus(ret, errmsg);
}