The unlock of a lock should, I would even say "must", always be done in
a finally block. Otherwise really bad things (deadlock) can happen if an exception
is thrown.

See http://java.sun.com/j2se/1.5.0/docs/api/java/util/concurrent/locks/Lock.html