Similarly, as you do with the tokens, consider only transferring the balance if it is not 0:
feeRecipient.transfer(address(this).balance);
or here:
payable(msg.sender).transfer(boughtETHAmount);
Recommended Mitigation Steps
Only do the external call if the amount is above zero.
Shadowfiend
commented
2 years ago
Handle
pauliax
Vulnerability details
Impact
Similarly, as you do with the tokens, consider only transferring the balance if it is not 0: feeRecipient.transfer(address(this).balance); or here: payable(msg.sender).transfer(boughtETHAmount);
Recommended Mitigation Steps
Only do the external call if the amount is above zero.