Additional token-sanity checks
We recommend adding some missing checks to the token contract:
The
DoTransfer
method should check whetherTo
is null in case anyone attempts to burn tokens by sending them to the null address, rather than callingBurn
.The
Create
method should check thatDecimals
is nonnegative and not too large. User-interface denial-of-service issues may arise if the number of decimals is not within a reasonable logarithm of the range of the token balances.The
AddMinter
andRemoveMinter
methods should have null checks on the addresses added/removed.