You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There are a number of slither warnings on the GoodDollar & Bancor contracts that need to be analyzed and resolved.
What
Run slither (yarn slither)
Resolve all issues (either by code change for legitimate warnings or adding inline ignore comments to silence them, along with an explanation on why we're silencing them)
Make sure CI is green (incl. slither job)
The text was updated successfully, but these errors were encountered:
### Description
Fixes#525
Mostly false positives where I added comments.
One change I did make was adding reentrancy guards to the `mint`
functions in the `GoodDollarExpansionController`.
Even though the slither warnings were only a "medium" around possibly
manipulating event data via re-entrancy, I thought it'd still be a good
idea to disallow reentrancy for all mint functions.
There are a number of slither warnings on the GoodDollar & Bancor contracts that need to be analyzed and resolved.
What
yarn slither
)The text was updated successfully, but these errors were encountered: