Two threads contend for the same lock. After first one releases it, the second will still acquire it. But before the second unlocks, the first will remove the lock from the map leading to NullPointerException.

Fix: You must hold a reference to the lock on the first get. (Use computeIfAbsent for simplicity). This ensures that even if the cache is cleared, the threads that are waiting on the lock are not affected.