Change in ovirt-engine[master]: core: Return emptyList to avoid possible NPE

[rgolan at redhat.com]

Roy Golan has submitted this change and it was merged.

Change subject: core: Return emptyList to avoid possible NPE
......................................................................


core: Return emptyList to avoid possible NPE

Found possible null pointer exception in the Affinity Rules Enforcement
Manager.
The NPE is caused when findVmViolatingPositiveAg returns null and then
the method isEmpty (From the List Interface) is invoked on that value.
This bug was fixed by Returning empty list instead of null.

Change-Id: I92a7df2771bba55db485fc0c265dcb367c092102
Signed-off-by: Tomer Saban <tsaban at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcer.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Roman Mohr: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Tomer Saban: Verified
  Roy Golan: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I92a7df2771bba55db485fc0c265dcb367c092102
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>