Asaf Shakarchi has uploaded a new change for review.

Change subject: webadmin: A check for possible NPE in 
NicActivateStatusColumn:getTooltip
......................................................................

webadmin: A check for possible NPE in NicActivateStatusColumn:getTooltip

A check for a possible NPE In NicActivateStatusColumn:getTooltip.

Change-Id: Ib350579b4ff7d7193a9870c83f4014df57bb3da4
Signed-off-by: Asaf Shakarchi <[email protected]>
---
M 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/column/NicActivateStatusColumn.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/57/10957/1

diff --git 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/column/NicActivateStatusColumn.java
 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/column/NicActivateStatusColumn.java
index fdaff25..1115d9c 100644
--- 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/column/NicActivateStatusColumn.java
+++ 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/column/NicActivateStatusColumn.java
@@ -44,7 +44,7 @@
         }
 
         StringBuilder tooltip = new StringBuilder(""); //$NON-NLS-1$
-        if (vnic.isPlugged()) {
+        if (vnic != null && vnic.isPlugged()) {
             tooltip =
                     
tooltip.append(templates.cardStatus(constants.pluggedNetworkInteface()).asString());
         } else {


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib350579b4ff7d7193a9870c83f4014df57bb3da4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Asaf Shakarchi <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to