Change in ovirt-engine[ovirt-engine-3.5]: core: fix possible NPE on run/migrate commands

tnisan at redhat.com tnisan at redhat.com
Tue May 26 09:56:48 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: core: fix possible NPE on run/migrate commands
......................................................................


core: fix possible NPE on run/migrate commands

In some cases we might reach the runningFailed method when no host was
selected by the scheduler, so we could get NPE when trying to decrease
the pending memory. So null-check is added.

Change-Id: I664b2aaabc855ed3b28a9b8fdee64d32c4191e65
Bug-Url: https://bugzilla.redhat.com/1220285
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommandBase.java
1 file changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Jenkins CI: Looks good to me, but someone else must approve
  Omer Frenkel: Looks good to me, approved
  Arik Hadas: Verified



-- 
To view, visit https://gerrit.ovirt.org/41411
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I664b2aaabc855ed3b28a9b8fdee64d32c4191e65
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list