# Handle rfa # Vulnerability details ## Impact expensive gas ## Proof of Concept https://github.com/code-423n4/2022-01-openleverage/blob/main/openleverage-contracts/contracts/XOLE.sol#L93 instead of using `toShare` to store `oleToken.balanceOf(address(this)).sub(claimable).sub(totalLocked).sub(devFund);`. Just use `newReward` directly to save more gas. ## Tools Used ## Recommended Mitigation Steps ``` uint newReward; if (fromToken == address(oleToken)) { uint claimable = totalRewarded.sub(withdrewReward); newReward= oleToken.balanceOf(address(this)).sub(claimable).sub(totalLocked).sub(devFund); require(newReward >= amount, 'Exceed OLE balance'); ```
Handle
rfa
Vulnerability details
Impact
expensive gas
Proof of Concept
https://github.com/code-423n4/2022-01-openleverage/blob/main/openleverage-contracts/contracts/XOLE.sol#L93
instead of using
toShareto storeoleToken.balanceOf(address(this)).sub(claimable).sub(totalLocked).sub(devFund);. Just usenewRewarddirectly to save more gas.Tools Used
Recommended Mitigation Steps