search

code-423n4 / 2022-01-sandclock-findings

0 stars 0 forks source link

Lack of input check in deposit amount #186

Closed code423n4 closed 2 years ago

code423n4 commented 2 years ago

Handle

pedroais

Vulnerability details

Impact

The deposit amount could be 0

https://github.com/code-423n4/2022-01-sandclock/blob/a90ad3824955327597be00bb0bd183a9c228a4fb/sandclock/contracts/Vault.sol#L420

Recommended Mitigation Steps

Require amount != 0

naps62 commented 2 years ago

duplicate of #155

dmvt commented 2 years ago

duplicate of #64