Closed deluca-mike closed 2 years ago
It was found some transfer, approve or transferFrom without checking the boolean result, ERC20 standard specify that the token can return false if this call was not made, so it's mandatory to check the result of approve methods.
transfer
approve
transferFrom
Duplicate of #52
Folded back into #59.
Lacks nuance: it does not explain why this is specifically an issue in Joyn's contracts or provide and valid attack vectors.
It was found some
transfer
,approve
ortransferFrom
without checking the boolean result, ERC20 standard specify that the token can return false if this call was not made, so it's mandatory to check the result of approve methods.