QA Report

[code423n4]

Contracts BridgeEscrow,L2GraphToken,L1GraphTokenGateway,and can always be reinitialized

Affected contracts and lines of code

• contracts/gateway/BridgeEscrow.sol
• contracts/l2/token/L2GraphToken.sol
• contracts/gateway/L1GraphTokenGateway.sol

Recommended mitigation check

• Add a modifier check or a bool variable is stored in storage, and stored as true at the first initialization, and only initializes the contract, only when it is set to false
• If the intention is not to allow re-initialization every time, then a reinitialzation function should be implemented

bool interanal initialized;

modifier initialized {
   if(initialized) revert();
}

[pcarranzav]

Dupe of #149 ?