From 7c72aa96dad1b0a5563aec938ffbe828c6939f0a Mon Sep 17 00:00:00 2001 From: Darren Shepherd Date: Fri, 4 Oct 2013 16:55:10 -0700 Subject: [PATCH] ConcurrentHashMap will through NPE on null key --- .../datastore/provider/DataStoreProviderManagerImpl.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/engine/storage/src/org/apache/cloudstack/storage/datastore/provider/DataStoreProviderManagerImpl.java b/engine/storage/src/org/apache/cloudstack/storage/datastore/provider/DataStoreProviderManagerImpl.java index c5d904be8b8..e186bd82017 100644 --- a/engine/storage/src/org/apache/cloudstack/storage/datastore/provider/DataStoreProviderManagerImpl.java +++ b/engine/storage/src/org/apache/cloudstack/storage/datastore/provider/DataStoreProviderManagerImpl.java @@ -58,6 +58,9 @@ public class DataStoreProviderManagerImpl extends ManagerBase implements DataSto @Override public DataStoreProvider getDataStoreProvider(String name) { + if (name == null) + return null; + return providerMap.get(name); }