Change the logic around reverting touched Precompiled contracts to ag…

…ree with geth.

libethereum/State.cpp

@@ -36,6 +36,11 @@ using namespace dev;
 using namespace dev::eth;
 namespace fs = boost::filesystem;
 
+namespace
+{
+Address const c_RipemdPrecompiledAddress{0x03};
+}
+
 State::State(u256 const& _accountStartNonce, OverlayDB const& _db, BaseState _bs):
     m_db(_db),
     m_state(&m_db),
@@ -589,8 +594,16 @@ void State::rollback(size_t _savepoint)
             account.resetCode();
             break;
         case Change::Touch:
-            account.untouch();
-            m_unchangedCacheEntries.emplace_back(change.address);
+            // Empty RIPEMD contract needs to be deleted even in case of exception.
+            // This doesn't affect main net, but is needed for some consensus tests.
+            // https://github.com/ethereum/go-ethereum/pull/3341/files#diff-2433aa143ee4772026454b8abd76b9dd
+            // https://github.com/ethereum/EIPs/issues/716
+            // https://github.com/ethereum/aleth/pull/5652
+            if (change.address != c_RipemdPrecompiledAddress)
+            {
+                account.untouch();
+                m_unchangedCacheEntries.emplace_back(change.address);
+            }
             break;
         }
         m_changeLog.pop_back();