-
Notifications
You must be signed in to change notification settings - Fork 7
Issues: code-423n4/2023-08-verwa-findings
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
Author
Label
Projects
Milestones
Assignee
Sort
Issues list
QA Report
bug
Something isn't working
edited-by-warden
Q-09
QA (Quality Assurance)
Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax
selected for report
This submission will be included/highlighted in the audit report
#448
opened Aug 10, 2023 by
code423n4
Analysis
A-06
analysis-advanced
grade-a
high quality report
This report is of especially high quality
selected for report
This submission will be included/highlighted in the audit report
sponsor acknowledged
Technically the issue is correct, but we're not going to resolve it for XYZ reasons
#446
opened Aug 10, 2023 by
code423n4
User don't have to deposit for a week into the market to get his weekly reward from the Assets can be stolen/lost/compromised directly
bug
Something isn't working
H-01
primary issue
Highest quality submission among a set of duplicates
satisfactory
satisfies C4 submission criteria; eligible for awards
selected for report
This submission will be included/highlighted in the audit report
upgraded by judge
Original issue severity upgraded from QA/Gas by judge
LendingLedger
3 (High Risk)
#416
opened Aug 10, 2023 by
code423n4
Voters from VotingEscrow can vote infinite times in vote_for_gauge_weights() of GaugeController
3 (High Risk)
Assets can be stolen/lost/compromised directly
bug
Something isn't working
H-02
primary issue
Highest quality submission among a set of duplicates
satisfactory
satisfies C4 submission criteria; eligible for awards
selected for report
This submission will be included/highlighted in the audit report
upgraded by judge
Original issue severity upgraded from QA/Gas by judge
#396
opened Aug 10, 2023 by
code423n4
Users can front-run calls to Assets not at direct risk, but function/availability of the protocol could be impacted or leak value
bug
Something isn't working
downgraded by judge
Judge downgraded the risk level of this issue
M-01
satisfactory
satisfies C4 submission criteria; eligible for awards
selected for report
This submission will be included/highlighted in the audit report
change_gauge_weight
to gain extra voting power
2 (Med Risk)
#294
opened Aug 10, 2023 by
code423n4
When adding a gauge, its initial value has to be set by an admin or all voting power towards it will be lost
3 (High Risk)
Assets can be stolen/lost/compromised directly
bug
Something isn't working
edited-by-warden
H-03
primary issue
Highest quality submission among a set of duplicates
satisfactory
satisfies C4 submission criteria; eligible for awards
selected for report
This submission will be included/highlighted in the audit report
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
upgraded by judge
Original issue severity upgraded from QA/Gas by judge
#288
opened Aug 10, 2023 by
code423n4
Delegated votes are locked when owner lock is expired
3 (High Risk)
Assets can be stolen/lost/compromised directly
bug
Something isn't working
edited-by-warden
H-04
primary issue
Highest quality submission among a set of duplicates
satisfactory
satisfies C4 submission criteria; eligible for awards
selected for report
This submission will be included/highlighted in the audit report
upgraded by judge
Original issue severity upgraded from QA/Gas by judge
#268
opened Aug 10, 2023 by
code423n4
It is possible to DoS all the functions related to some gauge in Assets can be stolen/lost/compromised directly
bug
Something isn't working
edited-by-warden
H-05
primary issue
Highest quality submission among a set of duplicates
selected for report
This submission will be included/highlighted in the audit report
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
GaugeController
3 (High Risk)
#206
opened Aug 10, 2023 by
code423n4
Users may be forced into long lock times to be able to undelegate back to themselves.
3 (High Risk)
Assets can be stolen/lost/compromised directly
bug
Something isn't working
H-06
primary issue
Highest quality submission among a set of duplicates
satisfactory
satisfies C4 submission criteria; eligible for awards
selected for report
This submission will be included/highlighted in the audit report
upgraded by judge
Original issue severity upgraded from QA/Gas by judge
#182
opened Aug 10, 2023 by
code423n4
Upon IncreaseAmount the lock may not align to the nearest weekly increment
2 (Med Risk)
Assets not at direct risk, but function/availability of the protocol could be impacted or leak value
bug
Something isn't working
M-02
satisfactory
satisfies C4 submission criteria; eligible for awards
selected for report
This submission will be included/highlighted in the audit report
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
#145
opened Aug 9, 2023 by
itsmetechjay
lack of access control in LendingLedger.sol#checkpoint_lender and function checkpoint_market
3 (High Risk)
Assets can be stolen/lost/compromised directly
bug
Something isn't working
H-07
satisfactory
satisfies C4 submission criteria; eligible for awards
selected for report
This submission will be included/highlighted in the audit report
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
#142
opened Aug 9, 2023 by
itsmetechjay
Replace old_sum_bias by old_bias
2 (Med Risk)
Assets not at direct risk, but function/availability of the protocol could be impacted or leak value
bug
Something isn't working
M-03
satisfactory
satisfies C4 submission criteria; eligible for awards
selected for report
This submission will be included/highlighted in the audit report
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
#140
opened Aug 9, 2023 by
itsmetechjay
If governance removes a gauge, user's voting power for that gauge will be lost.
3 (High Risk)
Assets can be stolen/lost/compromised directly
bug
Something isn't working
H-08
primary issue
Highest quality submission among a set of duplicates
selected for report
This submission will be included/highlighted in the audit report
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
upgraded by judge
Original issue severity upgraded from QA/Gas by judge
#62
opened Aug 9, 2023 by
code423n4
ProTip!
Mix and match filters to narrow down what you’re looking for.