When checking to see if address is equal to 0 then reverting, it could also check to see that the address of the token given is in the s_collateralTokens array. Also making a separate modifier instead that specifically checks if address given is equal to 0 would be better.
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.