'getLockIdsByOwner' uses array structure and maxLockId to match lockId and lock owner. By spamming/creating locks with value of 1 the attacker can increase gas cost of 'getLockIdsByOwner' significantly, which, while being a "view" function can be used elsewhere is dependencies.
MEDIUM
Manual review
Change iteration to mapping/array data structure, where key would be owner address and array contents are lockIds.
getLockIdsByOwner could be very gas intensive and revert
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.