QA Report

[code423n4]

See the markdown file with the details of this report here.

[c4-pre-sort]

0xSorryNotSorry marked the issue as high quality report

[GalloDaSballo]

• [L-1] queueWithdrawal() and slashShares() can revert with an out of bounds error if strategies are removed
R
• [L-2] slashQueuedWithdrawal() can leave stuck assets in strategies if passed an incorrect indicesToSkip
L, interesting
• [L-3] Allowing anyone to claim delayed withdrawals on behalf of the users can be detrimental to them
I, they can just not queue it
• [L-4] Missing events for important actions
NC
• [L-5] _claimDelayedWithdrawals emits "empty" events when maxNumberOfDelayedWithdrawalsToClaim == 0
NC

[NC-1] Use i as the increment variable
I, they are replaceable i,j is very common in math

1L 1R 1NC

[c4-judge]

GalloDaSballo marked the issue as grade-b