fix(fuzz): remove invalid assertion in decode_tx harness

The decode_tx fuzz target asserted that DecodeHexTx() always fails,
inherited from Bitcoin Core where random fuzz bytes essentially never
produce valid transactions. With real Dash chain data as corpus input
(e.g. AssetUnlockTx), valid transactions decode successfully, causing
a false-positive crash. Remove the assertion to allow both success
and failure paths to be exercised.

Author: thepastaclaw

src/test/fuzz/decode_tx.cpp

@@ -16,6 +16,5 @@ FUZZ_TARGET(decode_tx)
 {
     const std::string tx_hex = HexStr(std::string{buffer.begin(), buffer.end()});
     CMutableTransaction mtx;
-    const bool result_none = DecodeHexTx(mtx, tx_hex);
-    assert(!result_none);
+    (void)DecodeHexTx(mtx, tx_hex);
 }

src/test/fuzz/fuzz.cpp

@@ -7,6 +7,7 @@
 #include <fs.h>
 #include <netaddress.h>
 #include <netbase.h>
+#include <stats/client.h>
 #include <test/util/setup_common.h>
 #include <util/check.h>
 #include <util/sock.h>
@@ -98,6 +99,12 @@ void ResetCoverageCounters() {}
 
 void initialize()
 {
+    // Initialize a no-op stats client to prevent null dereferences in production
+    // code that unconditionally calls g_stats_client->timing()/inc()/count().
+    if (!::g_stats_client) {
+        ::g_stats_client = std::make_unique<StatsdClient>();
+    }
+
     // Terminate immediately if a fuzzing harness ever tries to create a TCP socket.
     CreateSock = [](const sa_family_t&) -> std::unique_ptr<Sock> { std::terminate(); };