Submission Details
Severity:
Redundant import statement in contract `SablierV2LockupLinear`
Summary
The SablierV2Lockup abstract contract is imported twice in the SablierV2LockupLinear contract.
Vulnerability Details
In src/SablierV2LockupLinear.sol, SablierV2Lockup was imported twice like this:
import { SablierV2Lockup } from "./abstracts/SablierV2Lockup.sol";
import { SablierV2Lockup } from "./abstracts/SablierV2Lockup.sol";
While this does not impact the functionality or gas costs directly, it is recommended to remove redundant imports to improve code clarity and maintainability.
Tools Used
Manual review
Recommendations
It is recommended to remove the unnecessary duplicate imports from the code of the contracts.
import { SablierV2Lockup } from "./abstracts/SablierV2Lockup.sol";
-import { SablierV2Lockup } from "./abstracts/SablierV2Lockup.sol";
Updates
Lead Judging Commences
inallhonesty about 1 year ago
Submission Judgement Published Reason: Non-acceptable severity
Assigned finding tags:
Info/Gas/Invalid as per Docs
https://docs.codehawks.com/hawks-auditors/how-to-determine-a-finding-validity
Prize pool breakdown
Total prize
53,440 USDC
nSLOC
2,65520 USDC / LOC
45,000 USDC
3,100 USDC
5,340 USDC
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.