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]

1 claimableUserDelayedWithdrawals does not take into account withdrawalDelayBlocks Low 1
L + 3

2 sharesToUnderlying and underlyingToShares functions marked as view but docs say they can make changes Low 2
R

3 slashQueuedWithdrawal and slashShares functions should emit an event Low 1
NC

4 Immutable state variables lack zero address checks Low 7
OOS
5 Adding a return statement when the function defines a named return variable is redundant NC 2
OOS

1L 1R 1NC +3

[c4-judge]

GalloDaSballo marked the issue as grade-b

[c4-judge]

GalloDaSballo marked the issue as grade-a