should this not also take the lock ?