We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
JMukesh
insecure randomness due to a modulo on block.timestamp, now or blockhash. These can be influenced by miners to some extent so they should be avoided
https://github.com/code-423n4/2021-05-nftx/blob/main/nftx-protocol-v2/contracts/solidity/NFTXVaultUpgradeable.sol#L418
slither
use chainlink vrf
The text was updated successfully, but these errors were encountered:
JMukesh issue #60
5ba3e3e
Duplicate of #78
Sorry, something went wrong.
No branches or pull requests
Handle
JMukesh
Vulnerability details
Impact
insecure randomness due to a modulo on block.timestamp, now or blockhash. These can be influenced by miners to some extent so they should be avoided
Proof of Concept
https://github.com/code-423n4/2021-05-nftx/blob/main/nftx-protocol-v2/contracts/solidity/NFTXVaultUpgradeable.sol#L418
Tools Used
slither
Recommended Mitigation Steps
use chainlink vrf
The text was updated successfully, but these errors were encountered: