You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ed...@apache.org on 2012/09/19 01:42:53 UTC

git commit: CLOUDSTACK-123: fix NPE

Updated Branches:
  refs/heads/4.0 92fdeb53b -> 515b60826


CLOUDSTACK-123: fix NPE


Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/515b6082
Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/515b6082
Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/515b6082

Branch: refs/heads/4.0
Commit: 515b60826697a268a4f5d264f7af23113de42418
Parents: 92fdeb5
Author: Edison Su <su...@gmail.com>
Authored: Tue Sep 18 16:42:36 2012 -0700
Committer: Edison Su <su...@gmail.com>
Committed: Tue Sep 18 16:42:36 2012 -0700

----------------------------------------------------------------------
 .../consoleproxy/ConsoleProxyManagerImpl.java      |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/515b6082/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java b/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java
index 6de1463..38dfb06 100755
--- a/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java
+++ b/server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java
@@ -1184,7 +1184,11 @@ public class ConsoleProxyManagerImpl implements ConsoleProxyManager, ConsoleProx
                 }
             } else {
                 if (s_logger.isDebugEnabled()) {
-                    s_logger.debug("Zone host is ready, but console proxy template: " + template.getId() +  " is not ready on secondary storage: " + secondaryStorageHost.getId());
+                	if (secondaryStorageHost != null) {
+                		s_logger.debug("Zone host is ready, but console proxy template: " + template.getId() +  " is not ready on secondary storage: " + secondaryStorageHost.getId());
+                	} else {
+                		s_logger.debug("Zone host is ready, but console proxy template: " + template.getId() +  " is not ready on secondary storage.");
+                	}
                 }
             }
         }