Change in ovirt-engine[master]: webadmin: Fix FindBugs warning in ImportVmModel

vszocs at redhat.com vszocs at redhat.com
Tue Mar 19 14:09:33 UTC 2013


Vojtech Szocs has submitted this change and it was merged.

Change subject: webadmin: Fix FindBugs warning in ImportVmModel
......................................................................


webadmin: Fix FindBugs warning in ImportVmModel

In ImportVmModel.init() method, 'dataCenter' reference
variable can theoretically be null, which triggers FindBugs
warning "Possible null pointer dereference".

This patch fixes the above mentioned warning.

Change-Id: Ib0b452b0a998068250dc4431f1f57346f7888000
Signed-off-by: Vojtech Szocs <vszocs at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ImportVmModel.java
1 file changed, 34 insertions(+), 32 deletions(-)

Approvals:
  Vojtech Szocs: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib0b452b0a998068250dc4431f1f57346f7888000
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>



More information about the Engine-commits mailing list