`swapExactTokensForETH` checks the wrong return value.md

7 - swapExactTokensForETH checks the wrong return value

Instead of checking that the amount of tokens received from a swap is greater than the minimum amount expected from this swap, it calculates the difference between the initial receiver’s balance and the balance of the router

Recommendation:

Check the intended values

---

Slide Screenshot

---

Slide Text

• ConsenSys Audit DAOfi Finding 4.4
• Error Handling
• Major Severity
• Incorrect Check
• Swap Tokens
• Use Correct Check

---

References

• Youtube Reference
• Major severity finding from Consensys Diligence Audit of DAOfi

---

Tags