Skip to content

Issues: juniormp/audit-rubicon-2023-04

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.

Sign up for GitHub

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

21 Open 0 Closed
21 Open 0 Closed
Author
Filter by author
Loading
Label
Filter by label
Loading
Use alt + click/return to exclude labels
or ⇧ + click/return for logical OR
Projects
Filter by project
Loading
Milestones
Filter by milestone
Loading
Assignee
Filter by who’s assigned
Sort
Sort by
Newest Oldest Most commented Least commented Recently updated Least recently updated Best match
Most reactions
👍 👎 😄 🎉 😕 ❤️ 🚀 👀

Issues list

[G-5] Division by two should use bit shifting
#22 opened Apr 11, 2023 by juniormp
[G-4] State variables should be cached in stack variables rather than re-reading them from storage
#21 opened Apr 11, 2023 by juniormp
[G-3] Use nested if and, avoid multiple check combinations
#20 opened Apr 11, 2023 by juniormp
[G-2] ++i/i++ should be unchecked {++i}/unchecked{i++} when it is not possible for them to overflow, as is the case when used in for- and while-loops
#19 opened Apr 11, 2023 by juniormp
[G-1] Use custom errors rather than revert()/require() strings to save gas
#18 opened Apr 11, 2023 by juniormp
[L-6] Function may run out of gas
#16 opened Apr 11, 2023 by juniormp
[L-5] Duplicated requires should be refactored to modifiers
#15 opened Apr 11, 2023 by juniormp
[L-4] Constants should be defined rather than using magic numbers
#14 opened Apr 11, 2023 by juniormp
[L-3] Use OpenZeppelin ReentrancyGuard
#13 opened Apr 11, 2023 by juniormp
[N-10] Remove unused code
#12 opened Apr 11, 2023 by juniormp
[N-9] Split complex require statements into smaller, more specific requirements
#11 opened Apr 11, 2023 by juniormp
[N-8] Require statements should have descriptive reason strings
#10 opened Apr 11, 2023 by juniormp
[N-7] Remove commented out code
#9 opened Apr 11, 2023 by juniormp
[N-6] Order of layout
#8 opened Apr 11, 2023 by juniormp
[N-5] Remove hardhat console import
#7 opened Apr 11, 2023 by juniormp
[N-4] Use the latest Solidity version
#6 opened Apr 11, 2023 by juniormp
[N-3] Use a fixed version number in Solidity
#5 opened Apr 11, 2023 by juniormp
[N-2] Inadequate NatSpec
#4 opened Apr 10, 2023 by juniormp
[L-2] Change function visibility to external
#3 opened Apr 10, 2023 by juniormp
[L-1] Use Ownable OpenZeppelin contract
#2 opened Apr 10, 2023 by juniormp
[N-1] Event is missing indexed fields
#1 opened Apr 10, 2023 by juniormp
ProTip! Mix and match filters to narrow down what you’re looking for.