No amountCollateral > balance check
There is a moreThanZero modifier to unsure that the user won't input 0 as amountCollateral but there isn't any check to ensure that the user is inputting an amountCollateral <= to his balance.
The transaction will fail is the user inputs an amountCollateral > to his balance.
Manual review
Add a if or require check to unsure user is inputting amountCollateral <= to his balance.
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.