Gilad Chaplik has uploaded a new change for review. Change subject: core: avoid NPE in VdsUpdateRunTimeInfo ......................................................................
core: avoid NPE in VdsUpdateRunTimeInfo adding missing NPE check for wrapped primitive value stat.getusage_mem_percent() (Integer) Change-Id: I3e32a45ed326de475a38541f5834956148e03054 Bug-Url: https://bugzilla.redhat.com/920986 Signed-off-by: Gilad Chaplik <[email protected]> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/96/12996/1 diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java index ce8c88d..67ef209 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java @@ -268,7 +268,7 @@ Integer minAvailableThreshold = Config.GetValue(ConfigValues.LogPhysicalMemoryThresholdInMB); Integer maxUsedPercentageThreshold = Config.GetValue(ConfigValues.LogMaxPhysicalMemoryUsedThresholdInPercentage); - if (stat.getmem_available() == null) { + if (stat.getmem_available() == null || stat.getusage_mem_percent() == null) { return; } -- To view, visit http://gerrit.ovirt.org/12996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I3e32a45ed326de475a38541f5834956148e03054 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
