Not sure what are the intentions with this event but I expect to see it not only in function transfer but also in other functions that update user balances.
Recommended Mitigation Steps
transferFrom should LogTransfer(sender, recipient, amount);
mint should LogTransfer(address(0), account, amount);
burn should LogTransfer(account, address(0), amount);
burnAll should LogTransfer(account, address(0), burnAmount);
Handle
pauliax
Vulnerability details
Impact
Not sure what are the intentions with this event but I expect to see it not only in function transfer but also in other functions that update user balances.
Recommended Mitigation Steps
transferFrom should LogTransfer(sender, recipient, amount); mint should LogTransfer(address(0), account, amount); burn should LogTransfer(account, address(0), amount); burnAll should LogTransfer(account, address(0), burnAmount);