Change in ovirt-engine[master]: engine: Fix coverity warnings (NULL_RETURNS)

mkolesni at redhat.com mkolesni at redhat.com
Sun Jan 5 14:14:36 UTC 2014


Mike Kolesnik has submitted this change and it was merged.

Change subject: engine: Fix coverity warnings (NULL_RETURNS)
......................................................................


engine: Fix coverity warnings (NULL_RETURNS)

Although the warned scenario is unlikely to occur,
it would be advisable to terminate network configuration
base on network labels if an external event (i.e. refresh
host capabilities) will cause an existing labeled nic to
disappear. This is a rare race, but can happen.

Change-Id: Iea07be387e71f0dfb8d9a78d8e03c504095c2062
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/AddNetworkParametersBuilder.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/RemoveNetworkParametersBuilder.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllErrors.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
7 files changed, 20 insertions(+), 0 deletions(-)

Approvals:
  Mike Kolesnik: Looks good to me, approved
  Moti Asayag: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iea07be387e71f0dfb8d9a78d8e03c504095c2062
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list