[BP] MB-20598: AddressSanitizer problems

Two leaks and one stack overflow.

The forest-kvstore code should use dynamic_cast (like couch-kvstore)
else when the incoming callback is not a RememberingCallback, we will
access outside of the incoming object.

ep_testsuite has a leak in tap code where we must release items
during iteration.

Change-Id: If0db9e936ee141299c5a579235e828c7309b8118
Reviewed-on: http://review.couchbase.org/68106
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: buildbot <build@couchbase.com>

diff --git a/src/forest-kvstore/forest-kvstore.cc b/src/forest-kvstore/forest-kvstore.cc
index 8a7724d..19b68ca 100644 (file)
--- a/src/forest-kvstore/forest-kvstore.cc
+++ b/src/forest-kvstore/forest-kvstore.cc
@@ -512,7 +512,7 @@ void ForestKVStore::getWithHeader(void *dbHandle, const std::string &key,
                                   bool fetchDelete) {
     fdb_file_handle *dbFileHandle = (fdb_file_handle *)dbHandle;
     RememberingCallback<GetValue> *rc =
-                       static_cast<RememberingCallback<GetValue> *>(&cb);
+                       dynamic_cast<RememberingCallback<GetValue> *>(&cb);
     bool getMetaOnly = rc && rc->val.isPartial();
     GetValue rv;
     fdb_kvs_handle *kvsHandle = NULL;

diff --git a/tests/ep_testsuite.cc b/tests/ep_testsuite.cc
index e2f1ed0..7932290 100644 (file)
--- a/tests/ep_testsuite.cc
+++ b/tests/ep_testsuite.cc
@@ -5750,7 +5750,7 @@ static enum test_result test_mb17517_tap_with_locked_key(ENGINE_HANDLE *h,
                         "Got CAS of -1 in TAP_MUTATION\n");
                 return FAIL;
             }
-
+            h1->release(h, NULL, it);
             testHarness.lock_cookie(cookie);
             break;
         }