Summary
100% of all REPORTED Findings have been addressed
- 0Solved
- 2Acknowledged
- 3Risk Accepted
- 5All Findings
- Critical0
- High0
- Medium3
- 3Risk A.
- Low2
- 2Ack.
- Informational0
Introduction#
TAC engaged Halborn to conduct a security assessment of their smart contracts from May 21st, 2025 to May 30th, 2025. The assessment focused on specific changes made to a Cosmos module provided to the Halborn team. Commit hashes and additional details are available in the Scope section of this report.
Assessment Summary#
The Halborn team assigned two full-time security engineers to evaluate the security of the merge requests. These engineers are experts in blockchain and smart contract security, possessing advanced skills in penetration testing, smart contract auditing, and extensive knowledge of multiple blockchain protocols.
The objectives of this assessment were to:
Verify that the Golang components function as intended.
Identify potential security vulnerabilities within the Cosmos application.
In summary, Halborn identified some improvements to reduce the likelihood and impact of risks, which were acknowledged by the TAC team . The main ones were the following:
Reverse the condition so that TxHash consistently reflects the intended header.Assign rpcAddr directly from cfg.JSONRPC.Address to ensure the server binds to the configured host and port.Assign each CommissionRates field from its corresponding Commission field.
Test Approach and Methodology#
Halborn employed a combination of manual and automated security testing to balance efficiency, timeliness, practicality, and accuracy within the scope of the custom modules. Manual testing was used to uncover logical, procedural, and implementation flaws, while automated testing enhanced coverage and quickly identified deviations from security best practices. The following phases and tools were utilized during the assessment:
Research into architecture and purpose.
Static analysis of the scoped repository and imported functions using tools such as
staticcheck,gosec,unconvert,codeql,ineffassign, andsemgrep.Manual assessment to identify security vulnerabilities within the codebase.
Verification of codebase correctness.
Dynamic analysis of files and modules within scope.
Caveats#
CosmosEVM security assessment is limited to the files directly affected by the four Pull Requests listed in the Sources section. Only changes introduced or modified in this comparison were considered; any pre-existing vulnerabilities or issues outside these specific files are beyond the scope of this review. Additionally, the audit does not cover dependencies, configuration files, or runtime environments. Therefore, findings and recommendations apply solely to the code and files added, removed, or modified in this branch comparison.
TacChain is a fork of Cosmos EVM with minimal changes. This audit focused exclusively on the following modifications:
Custom Inflation Formulas
ERC20
WASM Removal
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 |
|---|---|---|---|---|
| Incorrect TxHash assignment in EthHeaderFromTendermint leads to malformed headers | Medium | 5.0 | Risk Accepted06/10/2025 | |
| Misassignment of CommissionRates fields in NewMsgCreateValidator | Medium | 5.0 | Risk Accepted06/10/2025 | |
| Hardcoded loopback binding in NewWebsocketsServer prevents intended network exposure | Medium | 5.0 | Risk Accepted06/10/2025 | |
| Using vulnerable dependencies | Low | 2.5 | Acknowledged06/10/2025 | |
| Division by Zero in Custom Inflation Formulas | Low | 2.1 | Acknowledged06/10/2025 |
Findings & Tech Details#
Description
Recommendation
Remediation Comment
Description
Recommendation
Remediation Comment
Description
Recommendation
Remediation Comment
Description
Recommendation
Remediation Comment
Description
Recommendation
Remediation Comment
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.
