Lack of zero address check

[code423n4]

Handle

a_delamo

Vulnerability details

Impact

Some methods are missing validators to verify that address != 0.
A human error can produce that an invalid address is introduced and this check would help a bit to reduce issues.

    // RCOrderbook.sol
    constructor(address _factoryAddress, address _treasuryAddress) {
        factoryAddress = _factoryAddress;
        treasuryAddress = _treasuryAddress;
        treasury = IRCTreasury(treasuryAddress);
        uberOwner = msgSender();
    }

Proof of Concept

Provide direct links to all referenced code in GitHub. Add screenshots, logs, or any other relevant proof that illustrates the concept.

Tools Used

Recommended Mitigation Steps

[Splidge]

Duplicate of #56

[dmvt]

duplicate of #56