Remove redundant nested LOCK(pto->cs_inventory) in Dandelion block

The inner LOCK(pto->cs_inventory) at the Dandelion transaction processing
block was redundant because the outer scope already holds the same lock.
While CCriticalSection is a recursive mutex so this wouldn't deadlock, it
added unnecessary overhead. Remove the redundant lock and its enclosing
braces.

Author: cursoragent

src/net_processing.cpp

@@ -3616,24 +3616,21 @@ bool SendMessages(CNode* pto, CConnman& connman, const std::atomic<bool>& interr
             pto->vInventoryBlockToSend.clear();
 
             // Add Dandelion transactions
-            {
-                LOCK(pto->cs_inventory);
-                for (const uint256& hash : pto->vInventoryDandelionTxToSend) {
-                    pto->filterDandelionInventoryKnown.insert(hash);
-                    uint256 dandelionServiceDiscoveryHash;
-                    dandelionServiceDiscoveryHash.SetHex(
-                            "0xffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff");
-                    if (!pto->fSupportsDandelion && hash != dandelionServiceDiscoveryHash) {
-                        //LogPrintf("Pushing transaction MSG_TX %s to %s.",
-                        //          hash.ToString(), pto->addr.ToString());
-                        vInv.push_back(CInv(MSG_TX, hash));
-                    } else {
-                        vInv.push_back(CInv(MSG_DANDELION_TX, hash));
-                    }
-                    if (vInv.size() == MAX_INV_SZ) {
-                            connman.PushMessage(pto, msgMaker.Make(NetMsgType::INV, vInv));
-                        vInv.clear();
-                    }
+            for (const uint256& hash : pto->vInventoryDandelionTxToSend) {
+                pto->filterDandelionInventoryKnown.insert(hash);
+                uint256 dandelionServiceDiscoveryHash;
+                dandelionServiceDiscoveryHash.SetHex(
+                        "0xffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff");
+                if (!pto->fSupportsDandelion && hash != dandelionServiceDiscoveryHash) {
+                    //LogPrintf("Pushing transaction MSG_TX %s to %s.",
+                    //          hash.ToString(), pto->addr.ToString());
+                    vInv.push_back(CInv(MSG_TX, hash));
+                } else {
+                    vInv.push_back(CInv(MSG_DANDELION_TX, hash));
+                }
+                if (vInv.size() == MAX_INV_SZ) {
+                        connman.PushMessage(pto, msgMaker.Make(NetMsgType::INV, vInv));
+                    vInv.clear();
                 }
             }
             pto->vInventoryDandelionTxToSend.clear();