TempleGold

Summary

implement two step transfer of daoExecutor admin rights to have a safer/fault proof method of transfering daoExecutor rights to another address.

Vulnerability Details

In a two step access transfer method, the current admin will propose a new admin and the new admin address will call another function to accept that role that was proposed on its behalf. This ensures that the proposed/new admin is a valid address which is controlled by the protocol and can make calls to the contract. This helps to further eliminate errors or mistakes that could happen in a single step transfer where one error/mistake may cause the role to be lost forever to a mistakenly set address

Recommendations

change the single step transfer method to a two step where

• the current admin proposes a new admin address

• the proposed admin address calls an "accept()" function in the contract to accept the admin priviledges.