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