DatingDapp

Summary

The NotEnoughApprovals error in MultiSig.sol is defined in the contract but is never used. Instead, the contract relies on a string-based revert message in require statements.

Vulnerability Details

The contract defines the custom error `NotEnoughApprovals, but is not used anywhere in the code. Instead, the following line uses a string-based error message:

Using string-based error messages instead of custom errors increases gas costs because strings consume more storage and processing compared to custom errors.

Impact

• Increased gas costs when reverting transactions due to inefficient error handling.

• The presence of an unused error may indicate incomplete implementation or code redundancy.

Tools Used

• Manual Review

Recommendations

Replace the string-based revert message with the NotEnoughApprovals custom error to optimize gas usage and improve contract readability