Closed github-actions[bot] closed 1 year ago
Qeew
low
View visibility should be used here instead of pure as pure cannot read from state variables. View can read from state variables.
See here
https://github.com/sherlock-audit/2023-01-ajna/blob/main/contracts/src/ERC20Pool.sol#L103
Manual Review
Change Pure to View
Qeew
low
View should be used instead of pure
Summary
Vulnerability Detail
View visibility should be used here instead of pure as pure cannot read from state variables. View can read from state variables.
See here
Impact
Code Snippet
https://github.com/sherlock-audit/2023-01-ajna/blob/main/contracts/src/ERC20Pool.sol#L103
Tool used
Manual Review
Recommendation
Change Pure to View