From 228d2385254ee136ef64d229a414fdd7cefea9a0 Mon Sep 17 00:00:00 2001 From: Luke Dashjr Date: Wed, 31 Dec 2014 03:19:24 +0000 Subject: Make CCoinsViewCache's copy constructor private It is easily confused with CCoinsViewCache(CCoinsView*), which creates a sub-cache, but instead of creating a sub-cache, the copy constructor would copy the original and use that original's base, defeating the intended isolation. --- src/coins.h | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'src/coins.h') diff --git a/src/coins.h b/src/coins.h index dbe3f8bd31..e6ba168949 100644 --- a/src/coins.h +++ b/src/coins.h @@ -441,6 +441,11 @@ public: private: CCoinsMap::iterator FetchCoins(const uint256 &txid); CCoinsMap::const_iterator FetchCoins(const uint256 &txid) const; + + /** + * By making the copy constructor private, we prevent accidentally using it when one intends to create a cache on top of a base cache. + */ + CCoinsViewCache(const CCoinsViewCache &); }; #endif // BITCOIN_COINS_H -- cgit v1.2.3