Audit Suggestions for Recent Updates

0x132fabb5bc2fb61fc68bcfb5508841ddb11e9 commented 8 months ago

The following features have been added:

Added a factory for generating pools beyond the ~1m deposit limit, with expanded asset denomination tracking.
Implemented tests for various conditions.
Introduced a view method for proof verification.
Added an optional relayer constraint.
Tightened token transfer conditions and optimized operation order during withdraws.
Removed the registry from core contracts.
Enabled multi deposit functionality (multi withdraw remains external).
Merged native and token pools.

It would be really interesting to get an audit on the codebase - so far I've managed to spark some interest among a few auditors. To anyone out there willing to lend a hand in making this audit happen, a donation would be greatly valued.

madMax92221 commented 8 months ago

The donation is received and the audit is ongoing.

The report will be made public in the following repo when it is completed:

https://github.com/CDSecurity/audits

GeorgeHNTR commented 8 months ago

The audit by Hunter Security has been successfully completed.

The final report has just been made public and can be found here: https://www.huntersec.co/portfolio/audit/privacy-pools/full-report

CDSecurity commented 6 months ago

CD Security has effectively concluded the audit.

You can now access the recently released final report in our Github repo:

https://github.com/CDSecurity/audits/blob/main/audit%20reports/PrivacyPools-audit.pdf

0x132fabb5bc2fb61fc68bcfb5508841ddb11e9 / pools-sol

Audit Suggestions for Recent Updates #1