From c9a8e5944034a92df52de93a58a4f05f5b76e6c6 Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Mon, 4 May 2015 12:00:40 -0400 Subject: [PATCH] xen: Resolve Coverity FORWARD_NULL Coverity found that xenXMConfigCacheAddFile has an error path in which no error message and a -1 was not returned which could have resulted in a NULL dereference in a VIR_DEBUG statement and of course an erroneous 0 value returned! --- src/xen/xm_internal.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/xen/xm_internal.c b/src/xen/xm_internal.c index 64752dfa63..59b1cd4afa 100644 --- a/src/xen/xm_internal.c +++ b/src/xen/xm_internal.c @@ -279,6 +279,9 @@ xenXMConfigCacheAddFile(virConnectPtr conn, const char *filename) virDomainDefFree(entry->def); VIR_FREE(entry->filename); VIR_FREE(entry); + virReportError(VIR_ERR_INTERNAL_ERROR, "%s", + _("xenXMConfigCacheRefresh: virHashAddEntry name")); + return -1; } } VIR_DEBUG("Added config %s %s", entry->def->name, filename); -- 2.39.5