-
Notifications
You must be signed in to change notification settings - Fork 914
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Audit Changes for v1.4.1-2
cherry picked from release/v1.4.1-2
#815
Conversation
Pull Request Test Coverage Report for Build 10597342302Details
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(make sure to not squash the commit, because it'll create a new one)
The commit SHA is already different because of the changes mentioned in the description. Original Commit SHA: 75018fb |
I think its fine to squash commit. We have to re-audit for 1.5.0 anyway. |
This PR makes the changes based on Certora's audit report from the branch
release/v1.4.1-2
tomain
.In the release branch, we used the
ISafe
contract written inside the respective library files, but withmain
, we use theISafe
standard interface in theinterfaces
folder. Similarly,Enum
was moved fromcommon
tolibraries
, and thus that is used for the same in the migration contracts.Note: A
fmt
change was made toDebugTransactionGuard.sol
.