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 #58

Open
howlbot-integration bot opened this issue Oct 27, 2024 · 2 comments
Open

QA Report #58

howlbot-integration bot opened this issue Oct 27, 2024 · 2 comments
Labels
bug Something isn't working grade-b Q-06 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

@howlbot-integration
Copy link

howlbot-integration bot commented Oct 27, 2024

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

@howlbot-integration howlbot-integration bot 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 sufficient quality report This report is of sufficient quality labels Oct 27, 2024
howlbot-integration bot added a commit that referenced this issue Oct 27, 2024
howlbot-integration bot added a commit that referenced this issue Oct 27, 2024
@c4-judge
Copy link

c4-judge commented Nov 4, 2024

GalloDaSballo marked the issue as grade-b

@GalloDaSballo
Copy link

| QA-01 | Kleidi's Timelock can easily be broken via MEV |
I
| QA-02 | Wrong event scope when adding new call data |
I
| QA-03 | Pausing functionality is not implemented as documented |
I
| QA-04 | Make Timelock#isOperationExpired() more efficient |
I
| QA-05 | configuration changes should be behind a timelock from loop update |
TODO
| QA-07 | _addCalldataCheck() is too strenuous |
TODO
| QA-08 | Subtle invariant about not holding funds can be broken |
TODO
| QA-09 | Setters should always have equality checkers |
TODO

Not high signal enough, stopping, 0

@C4-Staff C4-Staff added the Q-06 label Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working grade-b Q-06 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

3 participants