qemu_driver.c: use g_strdup_printf

This patch changes all virAsprintf calls to use the GLib API
g_strdup_printf in qemu_driver.c

Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Daniel Henrique Barboza 2019-10-16 17:54:33 -03:00 committed by Michal Privoznik
parent 89026383d3
commit b83884d1a0
1 changed files with 17 additions and 21 deletions

View File

@ -402,7 +402,7 @@ qemuDomainSnapshotLoad(virDomainObjPtr vm,
priv = vm->privateData; priv = vm->privateData;
if (virAsprintf(&snapDir, "%s/%s", baseDir, vm->def->name) < 0) { if (!(snapDir = g_strdup_printf("%s/%s", baseDir, vm->def->name))) {
virReportError(VIR_ERR_INTERNAL_ERROR, virReportError(VIR_ERR_INTERNAL_ERROR,
_("Failed to allocate memory for " _("Failed to allocate memory for "
"snapshot directory for domain %s"), "snapshot directory for domain %s"),
@ -427,7 +427,7 @@ qemuDomainSnapshotLoad(virDomainObjPtr vm,
kill the whole process */ kill the whole process */
VIR_INFO("Loading snapshot file '%s'", entry->d_name); VIR_INFO("Loading snapshot file '%s'", entry->d_name);
if (virAsprintf(&fullpath, "%s/%s", snapDir, entry->d_name) < 0) { if (!(fullpath = g_strdup_printf("%s/%s", snapDir, entry->d_name))) {
virReportError(VIR_ERR_INTERNAL_ERROR, "%s", virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
_("Failed to allocate memory for path")); _("Failed to allocate memory for path"));
continue; continue;
@ -514,7 +514,7 @@ qemuDomainCheckpointLoad(virDomainObjPtr vm,
virObjectLock(vm); virObjectLock(vm);
priv = vm->privateData; priv = vm->privateData;
if (virAsprintf(&chkDir, "%s/%s", baseDir, vm->def->name) < 0) { if (!(chkDir = g_strdup_printf("%s/%s", baseDir, vm->def->name))) {
virReportError(VIR_ERR_INTERNAL_ERROR, virReportError(VIR_ERR_INTERNAL_ERROR,
_("Failed to allocate memory for " _("Failed to allocate memory for "
"checkpoint directory for domain %s"), "checkpoint directory for domain %s"),
@ -539,7 +539,7 @@ qemuDomainCheckpointLoad(virDomainObjPtr vm,
kill the whole process */ kill the whole process */
VIR_INFO("Loading checkpoint file '%s'", entry->d_name); VIR_INFO("Loading checkpoint file '%s'", entry->d_name);
if (virAsprintf(&fullpath, "%s/%s", chkDir, entry->d_name) < 0) { if (!(fullpath = g_strdup_printf("%s/%s", chkDir, entry->d_name))) {
virReportError(VIR_ERR_INTERNAL_ERROR, "%s", virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
_("Failed to allocate memory for path")); _("Failed to allocate memory for path"));
continue; continue;
@ -690,7 +690,7 @@ qemuStateInitialize(bool privileged,
if (!(qemu_driver->config = cfg = virQEMUDriverConfigNew(privileged))) if (!(qemu_driver->config = cfg = virQEMUDriverConfigNew(privileged)))
goto error; goto error;
if (virAsprintf(&driverConf, "%s/qemu.conf", cfg->configBaseDir) < 0) if (!(driverConf = g_strdup_printf("%s/qemu.conf", cfg->configBaseDir)))
goto error; goto error;
if (virQEMUDriverConfigLoadFile(cfg, driverConf, privileged) < 0) if (virQEMUDriverConfigLoadFile(cfg, driverConf, privileged) < 0)
@ -1359,10 +1359,10 @@ qemuGetSchedInfo(unsigned long long *cpuWait,
/* In general, we cannot assume pid_t fits in int; but /proc parsing /* In general, we cannot assume pid_t fits in int; but /proc parsing
* is specific to Linux where int works fine. */ * is specific to Linux where int works fine. */
if (tid) if (tid)
ret = virAsprintf(&proc, "/proc/%d/task/%d/sched", (int)pid, (int)tid); proc = g_strdup_printf("/proc/%d/task/%d/sched", (int)pid, (int)tid);
else else
ret = virAsprintf(&proc, "/proc/%d/sched", (int)pid); proc = g_strdup_printf("/proc/%d/sched", (int)pid);
if (ret < 0) if (!proc)
goto cleanup; goto cleanup;
ret = -1; ret = -1;
@ -1426,15 +1426,14 @@ qemuGetProcessInfo(unsigned long long *cpuTime, int *lastCpu, long *vm_rss,
unsigned long long usertime = 0, systime = 0; unsigned long long usertime = 0, systime = 0;
long rss = 0; long rss = 0;
int cpu = 0; int cpu = 0;
int ret;
/* In general, we cannot assume pid_t fits in int; but /proc parsing /* In general, we cannot assume pid_t fits in int; but /proc parsing
* is specific to Linux where int works fine. */ * is specific to Linux where int works fine. */
if (tid) if (tid)
ret = virAsprintf(&proc, "/proc/%d/task/%d/stat", (int)pid, tid); proc = g_strdup_printf("/proc/%d/task/%d/stat", (int)pid, tid);
else else
ret = virAsprintf(&proc, "/proc/%d/stat", (int)pid); proc = g_strdup_printf("/proc/%d/stat", (int)pid);
if (ret < 0) if (!proc)
return -1; return -1;
pidinfo = fopen(proc, "r"); pidinfo = fopen(proc, "r");
@ -3522,7 +3521,7 @@ qemuDomainManagedSavePath(virQEMUDriverPtr driver, virDomainObjPtr vm)
char *ret; char *ret;
g_autoptr(virQEMUDriverConfig) cfg = virQEMUDriverGetConfig(driver); g_autoptr(virQEMUDriverConfig) cfg = virQEMUDriverGetConfig(driver);
if (virAsprintf(&ret, "%s/%s.save", cfg->saveDir, vm->def->name) < 0) if (!(ret = g_strdup_printf("%s/%s.save", cfg->saveDir, vm->def->name)))
return NULL; return NULL;
return ret; return ret;
@ -4033,7 +4032,7 @@ qemuDomainScreenshot(virDomainPtr dom,
} }
} }
if (virAsprintf(&tmp, "%s/qemu.screendump.XXXXXX", cfg->cacheDir) < 0) if (!(tmp = g_strdup_printf("%s/qemu.screendump.XXXXXX", cfg->cacheDir)))
goto endjob; goto endjob;
if ((tmp_fd = mkostemp(tmp, O_CLOEXEC)) == -1) { if ((tmp_fd = mkostemp(tmp, O_CLOEXEC)) == -1) {
@ -4096,10 +4095,7 @@ getAutoDumpPath(virQEMUDriverPtr driver,
localtime_r(&curtime, &time_info); localtime_r(&curtime, &time_info);
strftime(timestr, sizeof(timestr), "%Y-%m-%d-%H:%M:%S", &time_info); strftime(timestr, sizeof(timestr), "%Y-%m-%d-%H:%M:%S", &time_info);
ignore_value(virAsprintf(&dumpfile, "%s/%s-%s", dumpfile = g_strdup_printf("%s/%s-%s", cfg->autoDumpPath, domname, timestr);
cfg->autoDumpPath,
domname,
timestr));
return dumpfile; return dumpfile;
} }
@ -5883,7 +5879,7 @@ qemuDomainHotplugAddIOThread(virQEMUDriverPtr driver,
virDomainIOThreadIDDefPtr iothrid; virDomainIOThreadIDDefPtr iothrid;
virJSONValuePtr props = NULL; virJSONValuePtr props = NULL;
if (virAsprintf(&alias, "iothread%u", iothread_id) < 0) if (!(alias = g_strdup_printf("iothread%u", iothread_id)))
return -1; return -1;
if (qemuMonitorCreateObjectProps(&props, "iothread", alias, NULL) < 0) if (qemuMonitorCreateObjectProps(&props, "iothread", alias, NULL) < 0)
@ -6001,7 +5997,7 @@ qemuDomainHotplugDelIOThread(virQEMUDriverPtr driver,
int new_niothreads = 0; int new_niothreads = 0;
qemuMonitorIOThreadInfoPtr *new_iothreads = NULL; qemuMonitorIOThreadInfoPtr *new_iothreads = NULL;
if (virAsprintf(&alias, "iothread%u", iothread_id) < 0) if (!(alias = g_strdup_printf("iothread%u", iothread_id)))
return -1; return -1;
qemuDomainObjEnterMonitor(driver, vm); qemuDomainObjEnterMonitor(driver, vm);
@ -11983,7 +11979,7 @@ qemuDomainMemoryPeek(virDomainPtr dom,
if (virDomainObjCheckActive(vm) < 0) if (virDomainObjCheckActive(vm) < 0)
goto endjob; goto endjob;
if (virAsprintf(&tmp, "%s/qemu.mem.XXXXXX", cfg->cacheDir) < 0) if (!(tmp = g_strdup_printf("%s/qemu.mem.XXXXXX", cfg->cacheDir)))
goto endjob; goto endjob;
/* Create a temporary filename. */ /* Create a temporary filename. */