Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Audit - M04 #9

Open
Kogaroshi opened this issue Apr 1, 2023 · 0 comments
Open

Audit - M04 #9

Kogaroshi opened this issue Apr 1, 2023 · 0 comments

Comments

@Kogaroshi
Copy link
Contributor

Inflation attack:

Dullahan Vault:
In case of the Vault, the shares are rebasing tokens, meaning any depositor will receive the same exact amount of dstkAAVE that the stkAAVE deposited. In case of a direct transfer to the Vault be an attacker before a user deposit, it will not impact the amount of dstkAAVE minted for the depositor. Instead, it will increase all previous depositors dstkAAVE balances (the same way it is increased when the Vault claims AAVE rewards and stake them for stkAAVE)

Dullahan Rewards Staking:
In the case of the Staking contract, this type of inflation attack would work only if the depositor was the 1st to deposit, and the contract was empty. This is why an initial deposit is done during the initialization, to prevent from such attacks.
After, if an attacker sends tokens directly to the contract right before an user deposit, the funds will be absorbed in the index (increasing all previous depositors position), but for the user depositing, the amount of shares minted will end up representing their share of the total funds in the contract correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant