Summary
100% of all REPORTED Findings have been addressed
- 3Acknowledged
- 1Risk Accepted
- 2Solved
- 6All Findings
- Critical0
- High0
- Medium0
- Low1
- 1Risk A.
- Informational5
- 3Ack.
- 2Solved
Introduction#
Decent Labs engaged Halborn to perform a security assessment of their smart contracts from August 18th, 2025, to August 26th, 2025. The assessment scope was limited to the smart contracts provided to Halborn. Commit hashes and additional details are available in the Scope section of this report.
The Decent Labs codebase in scope consists of smart contracts implementing a modular token sale system with EIP-712-based buyer verification, a non-transferable ERC20 staking and rewards mechanism, and a system deployer contract for proxy deployments.
Assessment Summary#
Halborn was allocated 7 days for this engagement and assigned 1 full-time security engineer to conduct a comprehensive review of the smart contracts within scope. The engineer is an expert in blockchain and smart contract security, with advanced skills in penetration testing and smart contract exploitation, as well as extensive knowledge of multiple blockchain protocols.
The objectives of this assessment were to:
Identify potential security vulnerabilities within the smart contracts.
Verify that the smart contract functionality operates as intended.
In summary, Halborn identified areas for improvement to reduce the likelihood and impact of potential risks, which were partially addressed by the Decent Labs team:
Update the _distributeRewards() function to increment rewardsDistributed only by the amount actually allocated to stakers, not the full amountToDistribute.Add a check in the initialize() function to ensure that protocolFeeReceiver is not the zero address.Add a validation check in _validateHedgeyParams to ensure that cliff_ is greater than or equal to start_.
Test Approach and Methodology#
Halborn conducted a combination of manual code review and automated security testing to balance efficiency, timeliness, practicality, and accuracy within the scope of this assessment. While manual testing is crucial for identifying flaws in logic, processes, and implementation, automated testing enhances coverage of smart contracts and quickly detects deviations from established security best practices.
The following phases and associated tools were employed throughout the term of the assessment:
Research into the platform's architecture, purpose and use.
Manual code review and walkthrough of smart contracts to identify any logical issues.
Comprehensive assessment of the safety and usage of critical Solidity variables and functions within scope that could lead to arithmetic-related vulnerabilities.
Local testing using custom scripts (
Foundry).Fork testing against main networks (
Foundry).Static security analysis of scoped contracts, and imported functions (
Slither).
Caveats#
After the conclusion of this assessment, the Decent Labs team introduced a minor update to the VotesERC20Staked contract in commit d7137b1 in order to support compatibility with both version 0 and version 1 deployments. Specifically, a clockModeTimestamp boolean variable was added to storage and initialization, with corresponding updates to the CLOCK_MODE() and clock() functions. This change allows the contract to operate using either block timestamps or block numbers as the underlying clock mechanism.
Although this modification was introduced after the commit reviewed during the assessment and therefore falls outside the official scope of this engagement, Halborn reviewed the update at a high level. Based on this review, it was concluded that the change is limited in scope and does not appear to introduce any new security risks.
Risk Methodology#
5.1 EXPLOITABILITY
Attack Origin (AO):
Attack Cost (AC):
Attack Complexity (AX):
Metrics:
| EXPLOITABILITY METRIC () | METRIC VALUE | NUMERICAL VALUE |
|---|---|---|
| Attack Origin (AO) | Arbitrary (AO:A) | 1 |
| Specific (AO:S) | 0.2 | |
| Attack Cost (AC) | Low (AC:L) | 1 |
| Medium (AC:M) | 0.67 | |
| High (AC:H) | 0.33 | |
| Attack Complexity (AX) | Low (AX:L) | 1 |
| Medium (AX:M) | 0.67 | |
| High (AX:H) | 0.33 |
5.2 IMPACT
Confidentiality (C):
Integrity (I):
Availability (A):
Deposit (D):
Yield (Y):
Metrics:
| IMPACT METRIC () | METRIC VALUE | NUMERICAL VALUE |
|---|---|---|
| Confidentiality (C) | None (C:N) | 0 |
| Low (C:L) | 0.25 | |
| Medium (C:M) | 0.5 | |
| High (C:H) | 0.75 | |
| Critical (C:C) | 1 | |
| Integrity (I) | None (I:N) | 0 |
| Low (I:L) | 0.25 | |
| Medium (I:M) | 0.5 | |
| High (I:H) | 0.75 | |
| Critical (I:C) | 1 | |
| Availability (A) | None (A:N) | 0 |
| Low (A:L) | 0.25 | |
| Medium (A:M) | 0.5 | |
| High (A:H) | 0.75 | |
| Critical (A:C) | 1 | |
| Deposit (D) | None (D:N) | 0 |
| Low (D:L) | 0.25 | |
| Medium (D:M) | 0.5 | |
| High (D:H) | 0.75 | |
| Critical (D:C) | 1 | |
| Yield (Y) | None (Y:N) | 0 |
| Low (Y:L) | 0.25 | |
| Medium (Y:M) | 0.5 | |
| High (Y:H) | 0.75 | |
| Critical (Y:C) | 1 |
5.3 SEVERITY COEFFICIENT
Reversibility (R):
Scope (S):
Metrics:
| SEVERITY COEFFICIENT () | COEFFICIENT VALUE | NUMERICAL VALUE |
|---|---|---|
| Reversibility () | None (R:N) | 1 |
| Partial (R:P) | 0.5 | |
| Full (R:F) | 0.25 | |
| Scope () | Changed (S:C) | 1.25 |
| Unchanged (S:U) | 1 |
| Critical | High | Medium | Low | Informational |
| 9 - 10 | 7 - 8.9 | 4.5 - 6.9 | 2 - 4.4 | 0 - 1.9 |
Scope#
Assessment Summary & Findings Overview#
# | Title | Severity | Score | Status |
|---|---|---|---|---|
| Suboptimal reward accounting can permanently lock dust amounts in contract | Low | 2.5 | Risk Accepted09/02/2025 | |
| Missing validation for protocolFeeReceiver may result in lost funds | Informational | 1.5 | Acknowledged09/02/2025 | |
| Vesting schedule allows cliff to be set before vesting start | Informational | 1.0 | Solved09/02/2025 | |
| Staked token can be added as a reward token | Informational | 0.5 | Solved09/02/2025 | |
| Missing input validation | Informational | 0.5 | Acknowledged09/02/2025 | |
| Floating pragma | Informational | 0.0 | Acknowledged09/02/2025 |
Findings & Tech Details#
Description
Recommendation
Remediation Comment
Description
Recommendation
Remediation Comment
Description
Recommendation
Description
Recommendation
Description
Recommendation
Remediation Comment
Description
Recommendation
Remediation Comment
9. Automated Testing#
Disclaimer#
Halborn strongly recommends conducting a follow-up assessment of the project either within six months or immediately following any material changes to the codebase, whichever comes first. This approach is crucial for maintaining the project’s integrity and addressing potential vulnerabilities introduced by code modifications.
