CodeHawks Escrow Contract - Competition Details

Summary

Vulnerability Details

You can cut out 10 opcodes in the creation-time EVM bytecode
if you declare a constructor payable.
Making the constructor payable eliminates the need for an initial check
of msg.value == 0 and saves 13 gas on deployment with no security risks.

There is 1 instance of this issue:

• Escrow.constructor(uint256,IERC20,address,address,address,uint256) should be set to payable

Impact

save gas

Tools Used

Recommendations

Set the constructor to payable.