Skip to content

Issues: code-423n4/2021-04-meebits-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.

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

84 Open 1 Closed
84 Open 1 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

Beebots.randomIndex() Can Be Manipulated To Not Be Random Without Costing Alice Anything 3 (High Risk) bug Something isn't working
#85 opened May 1, 2021 by code423n4
External over public bug Something isn't working G (Gas Optimization)
#84 opened May 1, 2021 by code423n4
Optimizations storage bug Something isn't working G (Gas Optimization) style
#83 opened May 1, 2021 by code423n4
Constructor lack of zero address validation 1 (Low Risk) bug Something isn't working
#82 opened May 1, 2021 by code423n4
Randomnesss can be manipulated 2 (Med Risk) bug Something isn't working
#81 opened May 1, 2021 by code423n4
Nonce not modified 1 (Low Risk) bug Something isn't working duplicate This issue or pull request already exists
#80 opened May 1, 2021 by code423n4
Beebots.contentHash() Is Currently Set As "todo" 0 (Non-critical) duplicate This issue or pull request already exists invalid This doesn't seem right
#79 opened May 1, 2021 by code423n4
Beebots.tokenURI() References A Non-Existent Domain at "TODO" 1 (Low Risk) bug Something isn't working duplicate This issue or pull request already exists
#78 opened May 1, 2021 by code423n4
Beebots.TradeValid() Will Erroneously Return True When Maker Is Set To Address(0) and makerIds Are Set To The TokenIds of Unminted Beebot NFTs 3 (High Risk) bug Something isn't working
#77 opened May 1, 2021 by code423n4
1
Can cancel the same offer several times 0 (Non-critical) bug Something isn't working
#76 opened Apr 30, 2021 by code423n4
NFT can be minted for free after sale ended 2 (Med Risk) bug Something isn't working
#75 opened Apr 30, 2021 by code423n4
The randomIndex() can be determined 3 (High Risk) bug Something isn't working
#74 opened Apr 30, 2021 by code423n4
function tokenByIndex treats last index as invalid 3 (High Risk) bug Something isn't working
#73 opened Apr 30, 2021 by code423n4
1
function ownerOf does not check if it is a valid _tokenId 0 (Non-critical) invalid This doesn't seem right
#72 opened Apr 30, 2021 by code423n4
Use always uint256 0 (Non-critical) bug Something isn't working
#71 opened Apr 30, 2021 by code423n4
contentHash is not used 0 (Non-critical) bug Something isn't working
#70 opened Apr 30, 2021 by code423n4
PauseMarket() can be optimized bug Something isn't working G (Gas Optimization)
#69 opened Apr 30, 2021 by code423n4
creatorNftMints is assigned only 0 or 1 0 (Non-critical) G (Gas Optimization) style
#68 opened Apr 30, 2021 by code423n4
Require() not needed bug Something isn't working G (Gas Optimization)
#67 opened Apr 30, 2021 by code423n4
1
Lack of chain information in the signed data leads to potential replay attacks. 2 (Med Risk) bug Something isn't working
#66 opened Apr 30, 2021 by code423n4
Should use SafeMath.add in the acceptTrade function. 1 (Low Risk) bug Something isn't working style
#65 opened Apr 30, 2021 by code423n4
Nonce does not increase during the entire sale. 1 (Low Risk) bug Something isn't working
#64 opened Apr 30, 2021 by code423n4
No zero check on constructor inputs. 1 (Low Risk) bug Something isn't working
#63 opened Apr 30, 2021 by code423n4
The ownerOf function requires checking if the owner is non-zero. 1 (Low Risk) bug Something isn't working
#62 opened Apr 30, 2021 by code423n4
Several functions in the ERC721 interface are not declared payable. invalid This doesn't seem right
#61 opened Apr 30, 2021 by code423n4
ProTip! Add no:assignee to see everything that’s not assigned.