Summary
100% of all REPORTED Findings have been addressed
- 0Risk Accepted
- 2Acknowledged
- 2Solved
- 4All Findings
- Critical1
- 1Solved
- High0
- Medium0
- Low1
- 1Solved
- Informational2
- 2Ack.
Introduction#
Noble engaged Halborn to conduct a security assessment on their smart contracts beginning on April 21st, 2025 and ending on April 25th, 2025. The security assessment was scoped to some changes made to a Cosmos module provided to the Halborn team. Commit hashes and further details can be found in the Scope section of this report.
Assessment Summary#
The team at Halborn was provided nine days for the engagement and assigned one full-time security engineer to assessment the security of the merge requests. The security engineers are blockchain and smart-contract security experts with advanced penetration testing, smart-contract hacking, and deep knowledge of multiple blockchain protocols.
The purpose of this assessment is to:
Ensure that the Golang components operate as intended.
Identify potential security issues with the Cosmos application.
In summary, Halborn identified some issues and improvements to reduce the likelihood and impact of risks, which were mostly addressed by the Noble team. The main ones were the following:
Implement IBC callback handlers and tracking system for pending transfers to recover tokens in case of transfer failures.Update dependencies with public known issues/vulnerabilities.Implement logic related to error handling.Improve ICS4Wrapper packet validations.
Caveats#
The security assessment is strictly limited to the files directly affected by the diff between the main (83a5546) and v2 (ffef804) branches. Only changes introduced or modified in this comparison are considered; any pre-existing vulnerabilities or issues outside of these specific files fall outside the scope of this review. Additionally, the audit does not encompass dependencies, configuration files, or runtime environments. As such, findings and recommendations pertain solely to the code and files that have been added, removed, or changed in this particular branch comparison.
Scope#
Findings Overview#
# | Title | Severity | Score | Status |
|---|---|---|---|---|
HAL-01 | Unhandled IBC Transfer Failures | Critical | 10.0 | Solved04/29/2025 |
HAL-02 | Use of vulnerable dependency | Low | 2.5 | Solved04/29/2025 |
HAL-03 | Unhandled Error Leading to Silent Failures | Informational | 0.0 | Acknowledged04/29/2025 |
HAL-04 | Potential Inadequate Packet Validation in ICS4Wrapper | Informational | 0.0 | Acknowledged04/29/2025 |
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.
