Presence of unused imports in several contracts
Summary
Several contracts contain unused imports.
Impact
Importing files that are not used in a contract likely indicates a mistake. The imports should be removed until they are needed.
Occurences: 4
-
CollectionManager.sol - https://github.com/Cyfrin/2024-07-ark-project/blob/273b7b94986d3914d5ee737c99a59ec8728b1517/apps/blockchain/ethereum/src/token/CollectionManager.sol#L5
import "openzeppelin-contracts/contracts/proxy/ERC1967/ERC1967Proxy.sol"; -
Cairo.sol - import "openzeppelin-contracts/contracts/utils/Strings.sol";
import "openzeppelin-contracts/contracts/utils/Strings.sol"; -
import "openzeppelin-contracts/contracts/token/ERC1155/extensions/IERC1155MetadataURI.sol";import "openzeppelin-contracts/contracts/token/ERC1155/extensions/ERC1155Supply.sol";
Tools Used
Manual Code Review: Analyzing the contract code directly
Static Analysis Tools: Slither - https://github.com/crytic/slither
Recommendations
Unused imports ought to be removed and added once again when needed.
Lead Judging Commences
Informational / Gas
Please, do not suppose impacts, think about the real impact of the bug and check the CodeHawks documentation to confirm: https://docs.codehawks.com/hawks-auditors/how-to-determine-a-finding-validity A PoC always helps to understand the real impact possible.
Prize pool breakdown
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.