Change in ovirt-engine[master]: core: checking for model only when not null

msalem at redhat.com msalem at redhat.com
Mon May 21 11:55:37 UTC 2012


Muli Salem has submitted this change and it was merged.

Change subject: core: checking for model only when not null
......................................................................


core: checking for model only when not null

Previous patch added a test for the model of USB controllers, and
ordering them accordingly. But, in legacy USB support, the model is
null, which leads to NPE. This patch fixes that behaviour.

Change-Id: I2f5f9253c145c2250f30a58d16c80109fa54a563
Signed-off-by: Oved Ourfali <oourfali at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Muli Salem: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2f5f9253c145c2250f30a58d16c80109fa54a563
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Muli Salem <msalem at redhat.com>



More information about the Engine-commits mailing list