ubiquity / ubiquity-dollar

Ubiquity Dollar (uAD) smart contracts and user interface.
https://uad.ubq.fi
Apache License 2.0
31 stars 82 forks source link

feat: check if collateral is enabled in collectRedemption #894

Closed gitcoindev closed 5 months ago

gitcoindev commented 5 months ago

Also add a unit test that verifies the check.

Resolves: https://github.com/sherlock-audit/2023-12-ubiquity-judging/issues/29

rndquu commented 5 months ago

This is wrong to push all fixes (low and medium/high) to the same fix-review branch according to the sherlock rules but we've already merged a couple of them. Chances are we will have to cherry pick PRs again when the sherlock's judging stage is over.