CodeHawks Escrow Contract - Competition Details

Summary

Constructors can be marked payable

Vulnerability Details

payable functions cost less gas to execute, since the compiler does not have to add extra checks to ensure that a payment wasn't provided.

A constructor can safely be marked as payable, since only the deployer would be able to pass funds, and the project itself would not pass any funds.

Impact

Cost more gas to execute.

Tools Used

manually vs code

Recommendations

Consider constructor as a payable.