Attackers can DoS redemptions by cancelling requests just before the batch is processed
Summary
SyrupBitcoinRouter::processRedemptions() reverts if the request owner is null, so any user can frontrun the batch call and delete their request to make it revert.
Vulnerability Detail
- User creates a request.
- User quickly removes the request, frontrunning the
processRedemptions()call.
Impact
Temporary DoS, but not feasible for an attacker, pure griefing. There may be some economical benefits to stop users withdrawals, but it is theoretical.
Code Snippet
Tool Used
Manual Review
Recommendation
Consider skipping the request if it has been removed.