Foundry DeFi Stablecoin CodeHawks Audit Contest

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 are 2 instances of this issue:

• DecentralizedStableCoin.constructor() should be set to payable

• DSCEngine.constructor(address[],address[],address) should be set to payable

Impact

save gas

Tools Used

Recommendations

Set the constructor to payable.