Martin Peřina has uploaded a new change for review.

Change subject: core: Fixes NPE in ActivateVdsVDSCommand
......................................................................

core: Fixes NPE in ActivateVdsVDSCommand

Fixes NullPointerException caused by running
VdsManager.refreshCapabilities() with vds parameter set to null.

Change-Id: If5b27df8ff0f5e79fe90d1b19e604ab107741dc4
Bug-Url: https://bugzilla.redhat.com/991729
Signed-off-by: Martin Perina <mper...@redhat.com>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/91/18691/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
index 71aad99..da6e7f1 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
@@ -390,6 +390,7 @@
     public VDS activate() {
         VDS vds = null;
         try {
+            vds = DbFacade.getInstance().getVdsDao().get(getVdsId());
             refreshHost(vds);
         } catch (Exception e) {
             log.infoFormat("Failed to activate VDS = {0} with error: {1}.",


-- 
To view, visit http://gerrit.ovirt.org/18691
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If5b27df8ff0f5e79fe90d1b19e604ab107741dc4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <mper...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to