that slither is flagging as a reentrancy risk, anyone know why? This is the contract and the audit results: https://vscode.blockscan.com/bsc/0x2823C3E6Ff2165B2B4aD40C965B5F3Cc7a8b4529, https://pastebin.com/81jYGbnk
bet.amount = 0; bet.party1 = payable(address(0)); bet.party2 = payable(address(0)); it's crazy. You just waste gas
modify state variables before external call
Ah, so just an order of operations thing? For the assigning things to 0, I suppose you are right! It's redundant
Обсуждают сегодня