20,000 USDC
View results
Submission Details
Severity: high

Hash collision in pools

Summary

because the pool id is stored as a hash, there is potential for another user to have the same hash for their pool id

Vulnerability Details

mapping(bytes32 => Pool) public pools;

The protocol hashes pool IDs with keccack256, multiple inputs can have the same hash even if they are different. For example if another users provided input where to be the same as an existing users poolId hash, the previous users poolID information will get overwritten by the new users. This will cause problems and users may lose their entire funds that are in the protocol.

Impact

Users can lose all funds in protocol

Tools Used

manual review

Recommendations

N/A

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.