Several function calls are made in unbounded loops.
This pattern is error-prone as it can trap the contracts due to the gas limitations or failed transactions.
Short term, review all the loops mentioned above and either: 1. allow iteration over part of the loop, or 2. remove elements. Long term, subscribe to Crytic.io to review external calls in loops. Crytic catches bugs of this type.
- ToB Audit Origin Dollar Finding 20
- Denial-of-Service
- High Severity
- External Calls
- Unbounded Loops
- Bound Loops
- Youtube Reference
- High Risk severity finding from ToB’s Audit of Origin Dollar