code-423n4 / 2022-05-rubicon-findings

5 stars 2 forks source link

Upgraded Q -> M from 130 [1656344996086] #510

Closed code423n4 closed 2 years ago

code423n4 commented 2 years ago

Judge has assessed an item in Issue #130 as Medium risk. The relevant finding follows:

HickupHH3 commented 2 years ago

[L-04] Use safeTransferFrom instead of transferFrom Calling ERC20.transferFrom without handling the returned value is unsafe.

Consider using OpenZeppelin's SafeERC20 library with safe versions of transfer functions.

dup of #316