Lior Vernia has uploaded a new change for review.

Change subject: webadmin: Fixed potential NPE in URI
......................................................................

webadmin: Fixed potential NPE in URI

Justly noted by Findbugs.

Change-Id: Ie3216adef9caeaf32c2d0cf0f33c03807f62ba05
Signed-off-by: Lior Vernia <lver...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Uri.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/95/15095/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Uri.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Uri.java
index 1534559..31da637 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Uri.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Uri.java
@@ -67,7 +67,7 @@
 
     public void setAuthority(UriAuthority authority) {
         this.authority = (authority == null) ? new UriAuthority(null) : 
authority;
-        if (!authority.isValid()) {
+        if (!this.authority.isValid()) {
             valid = false;
         }
     }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie3216adef9caeaf32c2d0cf0f33c03807f62ba05
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lver...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to