Zellic - Audit Reports

Assessment reports>SAX>Discussion>Unused and redundant fields of structures

Findings

DiscussionUnused and redundant fields of structures Checks-effects-interactions pattern Redundant if conditions in `getSellPrice` function Centralization risk Initial mint sellable Token creator can set virality Incomplete test coverage Token share rounding errors

Threat ModelWhat are threat models?BondingViralityController.sol MerkleTokenShare.sol

Audit ResultsSummary

Unused and redundant fields of structures

The HashtagData.lastPrice is not used in the contract BondingViralityController and can be deleted. Also, the BatchClaimTokensInput.index from the MerkleTokenShare contract is used to calculate the hash when verifying the proof, but in fact it is redundant and can be deleted without consequences to the functionality and security of the contract.

The HashtagData.lastPrice was removed in commit .