Change in ovirt-engine[master]: fix issues found by Coverity

emesika at redhat.com emesika at redhat.com
Sun Jun 21 13:53:05 UTC 2015


Eli Mesika has submitted this change and it was merged.

Change subject: fix issues found by Coverity
......................................................................


fix issues found by Coverity

Fixes the following issue (and in addition similiar one that from some
reason is not reported by is exactly the same)

*** CID 1306702:  Null pointer dereferences  (NULL_RETURNS)
> /frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/StorageDomainAdditionalStatusColumn.java:
> 43 in
> org.ovirt.engine.ui.webadmin.widget.table.column.StorageDomainAdditionalStatusColumn.getSafeHtmlStringMap(org.ovirt.engine.core.common.businessentities.StorageDomain)()
> 37
> 38         private Map<SafeHtml, String>
> getSafeHtmlStringMap(StorageDomain
> object) {
> 39             StorageDomain storageDomain = getEntityObject(object);
> 40             Map<SafeHtml, String> imagesToText = new
> LinkedHashMap<>();
> 41
> 42             if (storageDomain.getExternalStatus() !=
> ExternalStatus.Ok) {
> >>>     CID 1306702:  Null pointer dereferences  (NULL_RETURNS)
> >>>     Assigning: "statusImage" = null return value from
> >>>     "getStatusImage".
> 43                 ImageResource statusImage =
> getStatusImage(storageDomain.getExternalStatus());
> 44                 imagesToText.put(getImageSafeHtml(statusImage),
> 45                         constants.ExternalStatus() +
> storageDomain.getExternalStatus().name());
> 46             }
> 47             return imagesToText;
> 48         }

Change-Id: I500e4680dbae310cc3027cedff1fc54df353ece2
Signed-off-by: emesika <emesika at redhat.com>
---
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/HostAdditionalStatusColumn.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/StorageDomainAdditionalStatusColumn.java
2 files changed, 8 insertions(+), 6 deletions(-)

Approvals:
  Eli Mesika: Verified; Passed CI tests
  Daniel Erez: Looks good to me, approved
  Allon Mureinik: Looks good to me, but someone else must approve
  Oved Ourfali: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/42595
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I500e4680dbae310cc3027cedff1fc54df353ece2
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list