Phi — Exposed public _add/_removeCredIdPerAddress bricks victim sells

Source & credit. Reproduction of a public audit finding curated by AuditVault — the original finding: 41091-h-05-exposed-removecredidperaddress-addcredidperaddress-al. Standalone Foundry PoC and full write-up: 41091-h-05-exposed-removecredidperaddress-addcredidperaddress-al_exp in the evm-hack-registry mirror.

Vulnerability classes: vuln/access-control/missing-visibility · vuln/dos/state-corruption

Reproduction: a self-contained Foundry PoC that compiles & runs in an isolated project with only forge-std — no fork, no RPC, no anvil_state. Full trace: output.txt. PoC: test/41091-h-05-exposed-removecredidperaddress-addcredidperaddress-al_exp.sol.

AuditVault taxonomy: lang/solidity · sector/nft · platform/code4rena · severity/high · genome: frozen-funds · permanent · dos-resistance

Key info#


Impact	HIGH — Public _removeCredIdPerAddress lets anyone strip a victim's credId list so their subsequent sell reverts and shares are frozen
Protocol	Phi
Finding	Code4rena — Phi, 2024-08 · #41091
Report	2024-08-phi
Source	AuditVault
Status	Audit finding — reproduced as a standalone local synthetic PoC.
Compiler	`^0.8.24` (PoC)

TL;DR#

Public _removeCredIdPerAddress lets anyone strip a victim's credId list so their subsequent sell reverts and shares are frozen

See the vulnerable line marked // @> VULN in test/41091-h-05-exposed-removecredidperaddress-addcredidperaddress-al.sol and the end-to-end Exploit.run() that asserts the harm.

Diagrams#

flowchart TD A[Setup vulnerable state] --> B[Attacker triggers vulnerable path] B --> C["Vulnerable line executes @> VULN"] C --> D[Harm asserted in run]

Impact#

Public _removeCredIdPerAddress lets anyone strip a victim's credId list so their subsequent sell reverts and shares are frozen

Sources#

AuditVault finding
Code4rena report
Reduced source: synthetic reconstruction of the blamed functions from the contest repo (see finding for verbatim snippets).

Sources & further analysis#

Reproductions & code

Standalone PoC + full trace: 41091-h-05-exposed-removecredidperaddress-addcredidperaddress-al_exp (evm-hack-registry mirror).
AuditVault finding: 41091-h-05-exposed-removecredidperaddress-addcredidperaddress-al.

Alerts & third-party analyses

Web3Sec X hacked database: search.
Rekt leaderboard: search.
Solodit incident search: search.

These dashboards index community alerts tweets, post-mortems, and independent write-ups. Reach them through the protocol name above to cross-check this reproduction against other analyses.

Source-level debugger — step opcodes and Solidity in sync