Always lock mutex in ItemRepository::finalCleanup

The code is clearly not thread safe and thus must always lock the
mutex. It is only used through the AbstractItemRepository API from
ItemRepositoryRegistry which definitely doesn't lock the item repo
lock, only the register mutex.