QA Report #43
Labels
bug
Something isn't working
QA (Quality Assurance)
Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax
QA
get_decimals
computation is inefficient. Consider something like this (not tested)addressproxy.execute
: Authenticating onmsg.sender
(instead ofinfo.sender
) inexecute
is dangerous and it's easy to do mistakes in the future if this code is extended to handle more messages. Consider moving this authentication part to thereceive_cw20
function.String::from_utf8(denom)?
can panic if malicious users provide non-UTF8 strings. The function will revert. Consider using from_utf8_lossyThe text was updated successfully, but these errors were encountered: