1inch Farming > Business Logic > FarmingPool

ERC20 decimals() method may be unimplemented

Informational Severity
Informational Impact
N/A Likelihood


FarmingPool provides a public method to get the number of decimals of the stakingToken which calls the decimals method of the underlying IERC20 token. However, the EIP-20 standard declares that the decimals method is optional and that other contracts and interfaces should not rely on it being present.


The function could revert or return incorrect data. This may pose a composability risk for other contracts that try to interact with the farming pool.


Add documentation stipulating that the decimals method is required, or that the implementation may be unreliable.


The issue has been acknowledged by 1inch, and they will add natspec documentation when it will be implemented.