Set admin without accept method

Lines of code Vulnerability details Impact Set admin without method for accept is a bad practice. Also there is no check for zero. Proof of Concept Tools Used Code analytics Recommended Mitigation Steps Create method acceptAdmin and and check for zero address. --- The text was updated successfull...

NounsDAOProxy.sol (with LogicV2) cannot call NounsDAOExecutor.acceptAdmin()

Lines of code Vulnerability details Impact It's designed that DAOProxy uses Executor to manage the project and treasury. So DAOProxy can call any external functions in the Executor. But it cannot acceptAdmin, and it's strange. So, DAOProxy instances applying LogicV2 will not be able to acceptAdmi...