Vojtech Szocs has uploaded a new change for review. Change subject: webadmin: Fix potential NPE in MainTabHostView ......................................................................
webadmin: Fix potential NPE in MainTabHostView In Host "Load" column, fixed potential NPE coming from getProgressValue() method. Change-Id: I2552b3a917b848472ba9964e49d0e60d45cba0d3 Signed-off-by: Vojtech Szocs <[email protected]> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabHostView.java 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/98/7498/1 diff --git a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabHostView.java b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabHostView.java index bb5b302..4691534 100644 --- a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabHostView.java +++ b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabHostView.java @@ -138,7 +138,7 @@ @Override protected Integer getProgressValue(VDS object) { - return Math.min(object.getvm_active(), 100); + return object.getvm_active() != null ? Math.min(object.getvm_active(), 100) : 0; } }; getTable().addColumn(loadColumn, constants.loadHost(), "100px"); //$NON-NLS-1$ -- To view, visit http://gerrit.ovirt.org/7498 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I2552b3a917b848472ba9964e49d0e60d45cba0d3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
