Remove unused SERIALIZE_METHODS for CBanEntry

Author: MarcoFalke

src/addrdb.h

@@ -8,10 +8,8 @@
 
 #include <fs.h>
 #include <net_types.h> // For banmap_t
-#include <serialize.h>
 #include <univalue.h>
 
-#include <string>
 #include <vector>
 
 class CAddress;
@@ -44,12 +42,6 @@ class CBanEntry
      */
     explicit CBanEntry(const UniValue& json);
 
-    SERIALIZE_METHODS(CBanEntry, obj)
-    {
-        uint8_t ban_reason = 2; //! For backward compatibility
-        READWRITE(obj.nVersion, obj.nCreateTime, obj.nBanUntil, ban_reason);
-    }
-
     void SetNull()
     {
         nVersion = CBanEntry::CURRENT_VERSION;

src/test/fuzz/addrdb.cpp

@@ -18,20 +18,6 @@ FUZZ_TARGET(addrdb)
     FuzzedDataProvider fuzzed_data_provider(buffer.data(), buffer.size());
 
     // The point of this code is to exercise all CBanEntry constructors.
-    const CBanEntry ban_entry = [&] {
-        switch (fuzzed_data_provider.ConsumeIntegralInRange<int>(0, 2)) {
-        case 0:
-            return CBanEntry{fuzzed_data_provider.ConsumeIntegral<int64_t>()};
-            break;
-        case 1: {
-            const std::optional<CBanEntry> ban_entry = ConsumeDeserializable<CBanEntry>(fuzzed_data_provider);
-            if (ban_entry) {
-                return *ban_entry;
-            }
-            break;
-        }
-        }
-        return CBanEntry{};
-    }();
+    const CBanEntry ban_entry{fuzzed_data_provider.ConsumeIntegral<int64_t>()};
     (void)ban_entry; // currently unused
 }

src/test/fuzz/deserialize.cpp

@@ -195,10 +195,6 @@ FUZZ_TARGET_DESERIALIZE(blockheader_deserialize, {
     CBlockHeader bh;
     DeserializeFromFuzzingInput(buffer, bh);
 })
-FUZZ_TARGET_DESERIALIZE(banentry_deserialize, {
-    CBanEntry be;
-    DeserializeFromFuzzingInput(buffer, be);
-})
 FUZZ_TARGET_DESERIALIZE(txundo_deserialize, {
     CTxUndo tu;
     DeserializeFromFuzzingInput(buffer, tu);