forget to release lock