Skip to content
This repository has been archived by the owner on Nov 5, 2023. It is now read-only.

Bauer - The quotes from Curve may be subject to manipulation #44

Closed
sherlock-admin opened this issue Apr 30, 2023 · 0 comments
Closed
Labels
Duplicate A valid issue that is a duplicate of an issue with `Has Duplicates` label High A valid High severity issue Reward A payout will be made for this issue

Comments

@sherlock-admin
Copy link
Contributor

sherlock-admin commented Apr 30, 2023

Bauer

high

The quotes from Curve may be subject to manipulation

Summary

The get_virtual_price() function in Curve has a reentrancy risk, which can affect the price if the protocol fetches quotes from pools integrated with ETH on Curve.

Vulnerability Detail

The CurveOracle protocol calls the function get_virtual_price_from_lp_token() to obtain a quote from Curve. However, all pools integrated with ETH pose a read-only reentrancy risk. Please refer below link for detail.
https://chainsecurity.com/heartbreaks-curve-lp-oracles/

Impact

The read-only reentrancy operation manipulates the price.

Code Snippet

https://github.com/sherlock-audit/2023-04-blueberry/blob/main/blueberry-core/contracts/oracle/CurveOracle.sol#L101

Tool used

Manual Review

Recommendation

Duplicate of #123

@github-actions github-actions bot closed this as completed May 3, 2023
@github-actions github-actions bot added High A valid High severity issue Duplicate A valid issue that is a duplicate of an issue with `Has Duplicates` label labels May 3, 2023
@sherlock-admin sherlock-admin added the Reward A payout will be made for this issue label May 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Duplicate A valid issue that is a duplicate of an issue with `Has Duplicates` label High A valid High severity issue Reward A payout will be made for this issue
Projects
None yet
Development

No branches or pull requests

1 participant