We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
hickuphh3
The lines below of the addLiquiditySingleForMember() function
addLiquiditySingleForMember()
address _token = token; if(token == address(0)){_token = WBNB;} // Handle BNB -> WBNB
are redundant since _token is not used subsequently. Note that _handleTransferIn() will perform the handling of native BNB transfers.
_token
_handleTransferIn()
The mentioned lines above can be removed.
The text was updated successfully, but these errors were encountered:
hickuphh3 issue #57
630be14
verifyfirst
No branches or pull requests
Handle
hickuphh3
Vulnerability details
Impact
The lines below of the
addLiquiditySingleForMember()
functionare redundant since
_token
is not used subsequently. Note that_handleTransferIn()
will perform the handling of native BNB transfers.Recommended Mitigation Steps
The mentioned lines above can be removed.
The text was updated successfully, but these errors were encountered: