delvtech / council

Flexible DAO governance smart contracts, developed by Delv.
Apache License 2.0
93 stars 19 forks source link

Final Fixes from the RV Audit report #49

Closed aleph-v closed 2 years ago

aleph-v commented 3 years ago

Fixes problems: A17 - partial remediation with extra checks on call registration and deletion A18 - full fix by reverting on treasury call failure B09 - full fix by making the conversion safe B12 - full fix by upgrading all solidity versions to 0.8.3 B16 - full fix by validating that delegation addresses are non zero B18 - Partially fixes by addresses by adding some of the consistency checks for readily accessible values. B20 - full fix by renaming the function in question. B19 - full fix by inheriting from interfaces and adding overrides. B11 - By using setOwner in the constructor of core voting.