From 3a67bffaa8c03f7c7fe06e81962eccc347f977cc Mon Sep 17 00:00:00 2001 From: Yueh-Hsuan Chiang Date: Wed, 10 Feb 2016 12:06:59 -0800 Subject: [PATCH] Fix an ASAN error in transaction_test.cc Summary: One test in transaction_test.cc forgets to call SyncPoint::DisableProcessing(). As a result, a program might to access the SyncPoint singleton after it already goes out of scope. This patch fix this error by calling SyncPoint::DisableProcessing(). Test Plan: transaction_test Reviewers: sdong, IslamAbdelRahman, kradhakrishnan, anthony Reviewed By: anthony Subscribers: dhruba, leveldb Differential Revision: https://reviews.facebook.net/D54033 --- utilities/transactions/transaction_test.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/utilities/transactions/transaction_test.cc b/utilities/transactions/transaction_test.cc index 98dfc732b..d51912b38 100644 --- a/utilities/transactions/transaction_test.cc +++ b/utilities/transactions/transaction_test.cc @@ -2876,6 +2876,7 @@ TEST_F(TransactionTest, ExpiredTransactionDataRace1) { ASSERT_EQ("1", value); delete txn1; + rocksdb::SyncPoint::GetInstance()->DisableProcessing(); } } // namespace rocksdb