Missing Zero Address Check

[hats-bug-reporter[bot]]

Github username: -- Submission hash (on-chain): 0x1ea5e7dc68fc256935962e08fa02d9a865328a999bcfd0be50463eb43c4123ae Severity: low

Description: Description\

This Function is lack of zero address check in important operation, which may cause some unexpected result.

Attack Scenario\

contracts/bridge/Bridge.sol

    function setSequencerInbox(address _sequencerInbox) external onlyRollupOrOwner {
        sequencerInbox = _sequencerInbox;
        emit SequencerInboxUpdated(_sequencerInbox);
    }
    function setDelayedInbox(address inbox, bool enabled) external onlyRollupOrOwner {
        InOutInfo storage info = allowedDelayedInboxesMap[inbox];
        bool alreadyEnabled = info.allowed;
            delete allowedDelayedInboxesMap[inbox];
        }
    }

contracts/bridge/SequencerInbox.sol

    function setIsBatchPoster(address addr, bool isBatchPoster_) external onlyRollupOwner {
        isBatchPoster[addr] = isBatchPoster_;
        emit OwnerFunctionCalled(1);
    }
    function setIsSequencer(address addr, bool isSequencer_) external onlyRollupOwner {
        isSequencer[addr] = isSequencer_;
        emit OwnerFunctionCalled(4);
    }

Attachments

1. Proof of Concept (PoC) File

The entered address is address(0)

2. Revised Code File (Optional)

Add check of zero address in important operation.

[QYuQianchen]

Irrelevant submission