Adding "Per Address" functionality to existing distribution contracts on Tokensoft's platform.
Scope
Contest Results
On what chains are the smart contracts going to be deployed?
Any EVM-compatible networks (Ethereum, Base, Polygon, BSC, etc.)
If you are integrating tokens, are you allowing only whitelisted tokens to work with the codebase or any complying with the standard? Are they assumed to have certain properties, e.g. be non-reentrant? Are there any types of weird tokens you want to integrate?
Any Standard ERC-20 Token and USDC/USDT
Are the admins of the protocols your contracts integrate with (if any) TRUSTED or RESTRICTED? If these integrations are trusted, should auditors also assume they are always responsive, for example, are oracles trusted to provide non-stale information, or VRF providers to respond within a designated timeframe?
RESTRICTED - There are no external integrations.
Are there any protocol roles? Please list them and provide whether they are TRUSTED or RESTRICTED, or provide a more comprehensive description of what a role can and can't do/impact.
It is safe to assume that the owner of these distributor contracts are trusted. They should be able to sweep the funds and also add / remove tokens as well.
For permissioned functions, please list all checks and requirements that will be made before calling the function.
There are many functions which have an onlyOwner modifier, onlyRole(ADMIN_ROLE) modifier, and onlyInitializing modifier.
Is the codebase expected to comply with any EIPs? Can there be/are there any deviations from the specification?
No
Are there any off-chain mechanisms or off-chain procedures for the protocol (keeper bots, arbitrage bots, etc.)?
No
Are there any hardcoded values that you intend to change before (some) deployments?
If the codebase is to be deployed on an L2, what should be the behavior of the protocol in case of sequencer issues (if applicable)? Should Sherlock assume that the Sequencer won't misbehave, including going offline?
We assume Sequencer won't misbehave or go offline.
Should potential issues, like broken assumptions about function behavior, be reported if they could pose risks in future integrations, even if they might not be an issue in the context of the scope? If yes, can you elaborate on properties/invariants that should hold?
Yes, please tells us about any potential issues.
Please discuss any design choices you made.
The PerAddress prefix attached to contracts are a variation of existing contracts (i.e. PerAddressTrancheVestingMerkleDistributor.sol) which have been previously audited.
The distribution records are initialized by passing in a merkle proof, representing the user's vesting schedule, and on claim the merkle proof is also provided and validated before the claim is executed.
Please list any known issues/acceptable risks that should not result in a valid finding.
Some known issues may exist from a previous Sherlock audit (see: 2023-06-tokensoft-judging on Github).
We will report issues where the core protocol functionality is inaccessible for at least 7 days. Would you like to override this value?
No
Please provide links to previous audits (if any).
Please list any relevant protocol resources.
Additional audit information.
n/a
Total Rewards
Contest Pool
Lead Senior Watson
Judging Pool
Lead Judge
16,500 USDC
4,500 USDC
700 USDC
800 USDC
Status
Scope
Start Time
End Time
Judging Rules