Skip to content
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

QA Report #1085

Open
c4-bot-1 opened this issue Dec 28, 2023 · 5 comments
Open

QA Report #1085

c4-bot-1 opened this issue Dec 28, 2023 · 5 comments
Labels
bug Something isn't working edited-by-warden grade-a Q-04 QA (Quality Assurance) Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax sufficient quality report This report is of sufficient quality

Comments

@c4-bot-1
Copy link
Contributor

See the markdown file with the details of this report here.

@c4-bot-1 c4-bot-1 added bug Something isn't working QA (Quality Assurance) Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax labels Dec 28, 2023
c4-bot-2 added a commit that referenced this issue Dec 28, 2023
c4-bot-2 added a commit that referenced this issue Dec 28, 2023
c4-bot-1 added a commit that referenced this issue Dec 28, 2023
@0xSorryNotSorry
Copy link

A Market Can Not Be Deployed For A Token > 18 Decimals -> dup of #957

@c4-pre-sort
Copy link

0xSorryNotSorry marked the issue as sufficient quality report

@c4-pre-sort c4-pre-sort added the sufficient quality report This report is of sufficient quality label Jan 5, 2024
@Trumpero
Copy link

Trumpero commented Jan 31, 2024

Potential Loss of Assets in emergencyAction Function due to Unchecked msg.value
-> L

Unspent Native Coin gets rejected on calling emergencyAction function
-> L

Use of assert Instead of require in Smart Contract
-> bot finding (N-12)

CEI Should Be Followed
-> bot finding (L-19)

Add A Min Check On setGaugeWeightTolerance
-> L

surplusBufferSplit/guildSplit/othersSplit Might Round Down To 0
-> L

A Market Can Not Be Deployed For A Token > 18 Decimals
-> info (dup of #957)

Have A Minimum Value For maxDelayBetweenPartialRepay
-> info

Missing 0 Address/Value Checks (Missed By The Bots)
-> L

Incorrect Comment
-> NC

Handle The 0 surplusBuffer Case In The If Clause
-> info

Sanity Checks
-> NC

Make Sure CREDIT Can Not Be Redeemed To The SimplePSM Itself
-> info

@Trumpero
Copy link

+L from #1144

@c4-judge
Copy link
Contributor

Trumpero marked the issue as grade-a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working edited-by-warden grade-a Q-04 QA (Quality Assurance) Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax sufficient quality report This report is of sufficient quality
Projects
None yet
Development

No branches or pull requests

7 participants