[Cleanup] Fix cs_main in net_processing

add extern declaration for now. cs_main will be removed from here.

(Also fix AcceptBlockHeader method signature).

src/net_processing.cpp

@@ -27,6 +27,7 @@ struct COrphanTx {
     CTransaction tx;
     NodeId fromPeer;
 };
+
 std::map<uint256, COrphanTx> mapOrphanTransactions GUARDED_BY(cs_main);
 std::map<uint256, std::set<uint256> > mapOrphanTransactionsByPrev GUARDED_BY(cs_main);
 

src/net_processing.h

@@ -10,6 +10,8 @@
 #include "net.h"
 #include "validationinterface.h"
 
+extern RecursiveMutex cs_main; // !TODO: change mutex to cs_orphans
+
 /** Default for -maxorphantx, maximum number of orphan transactions kept in memory */
 static const unsigned int DEFAULT_MAX_ORPHAN_TRANSACTIONS = 100;
 /** Default for -blockspamfilter, use header spam filter */

src/validation.h

@@ -341,7 +341,7 @@ bool TestBlockValidity(CValidationState& state, const CBlock& block, CBlockIndex
 
 /** Store block on disk. If dbp is provided, the file is known to already reside on disk */
 bool AcceptBlock(const CBlock& block, CValidationState& state, CBlockIndex** pindex, CDiskBlockPos* dbp = NULL, bool fAlreadyCheckedBlock = false);
-bool AcceptBlockHeader(const CBlockHeader& block, CValidationState& state, CBlockIndex** ppindex = NULL);
+bool AcceptBlockHeader(const CBlock& block, CValidationState& state, CBlockIndex** ppindex = NULL);
 
 
 /** RAII wrapper for VerifyDB: Verify consistency of the block and coin databases */