From 718af45895cc3d95401df71908f707472d3f7398 Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Thu, 3 Jan 2013 14:16:14 -0500 Subject: [PATCH] parallels: check and handle error for virAsprintf() calls Ignore the return in parallelsMakePoolName() since subsequent check validates name was allocated. --- src/parallels/parallels_storage.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/parallels/parallels_storage.c b/src/parallels/parallels_storage.c index e768d881f8..2908bee4f3 100644 --- a/src/parallels/parallels_storage.c +++ b/src/parallels/parallels_storage.c @@ -146,7 +146,7 @@ static char *parallelsMakePoolName(virConnectPtr conn, const char *path) if (i == 0) name = strdup(path); else - virAsprintf(&name, "%s-%u", path, i); + ignore_value(virAsprintf(&name, "%s-%u", path, i)); if (!name) { virReportOOMError(); @@ -310,8 +310,7 @@ static int parallelsAddDiskVolume(virStoragePoolObjPtr pool, if (VIR_ALLOC(def)) goto no_memory; - virAsprintf(&def->name, "%s-%s", dom->def->name, diskName); - if (!def->name) + if (virAsprintf(&def->name, "%s-%s", dom->def->name, diskName) < 0) goto no_memory; def->type = VIR_STORAGE_VOL_FILE; -- 2.47.2