Open code423n4 opened 3 years ago
cmichel
The SherXERC20.transfer/transferFrom actions allow transferring tokens to the zero address. This is usually prohibited to accidentally avoid "burning" tokens by sending them to an unrecoverable zero address.
SherXERC20.transfer
transferFrom
Does it make more sense to include an extra burn() function? As removing the possibility to send to zero address removes the ability to burn.
burn()
Handle
cmichel
Vulnerability details
The
SherXERC20.transfer
/transferFrom
actions allow transferring tokens to the zero address. This is usually prohibited to accidentally avoid "burning" tokens by sending them to an unrecoverable zero address.