diff --git a/src/uml/uml_conf.c b/src/uml/uml_conf.c index a55b5b1f103cdfbbd7e25ac28edf4832d3373556..52b705c03a32637300d488787404d728fb9ddd32 100644 --- a/src/uml/uml_conf.c +++ b/src/uml/uml_conf.c @@ -114,8 +114,8 @@ umlConnectTapDevice(virConnectPtr conn, STRPREFIX(net->ifname, VIR_NET_GENERATED_PREFIX) || strchr(net->ifname, '%')) { VIR_FREE(net->ifname); - if (!(net->ifname = strdup(VIR_NET_GENERATED_PREFIX "%d"))) - goto no_memory; + if (VIR_STRDUP(net->ifname, VIR_NET_GENERATED_PREFIX "%d") < 0) + goto error; /* avoid exposing vnet%d in getXMLDesc or error outputs */ template_ifname = true; } @@ -141,8 +141,6 @@ umlConnectTapDevice(virConnectPtr conn, return 0; -no_memory: - virReportOOMError(); error: return -1; } @@ -463,8 +461,8 @@ virCommandPtr umlBuildCommandLine(virConnectPtr conn, if (vm->def->os.cmdline) { char *args, *next_arg; char *cmdline; - if ((cmdline = strdup(vm->def->os.cmdline)) == NULL) - goto no_memory; + if (VIR_STRDUP(cmdline, vm->def->os.cmdline) < 0) + goto error; args = cmdline; while (*args == ' ') diff --git a/src/uml/uml_driver.c b/src/uml/uml_driver.c index ded079827da09659ef4ccb14589f3d7bef7ea166..321be0fe22329799df44814f8e840666e66d85c0 100644 --- a/src/uml/uml_driver.c +++ b/src/uml/uml_driver.c @@ -248,8 +248,7 @@ requery: if (res && STRPREFIX(res, "pts:")) { VIR_FREE(def->source.data.file.path); - if ((def->source.data.file.path = strdup(res + 4)) == NULL) { - virReportOOMError(); + if (VIR_STRDUP(def->source.data.file.path, res + 4) < 0) { VIR_FREE(res); VIR_FREE(cmd); return -1; @@ -491,8 +490,8 @@ umlStateInitialize(bool privileged, "%s/log/libvirt/uml", LOCALSTATEDIR) == -1) goto out_of_memory; - if ((base = strdup(SYSCONFDIR "/libvirt")) == NULL) - goto out_of_memory; + if (VIR_STRDUP(base, SYSCONFDIR "/libvirt") < 0) + goto error; if (virAsprintf(¨_driver->monitorDir, "%s/run/libvirt/uml-guest", LOCALSTATEDIR) == -1) @@ -1672,8 +1671,7 @@ static char *umlDomainGetOSType(virDomainPtr dom) { goto cleanup; } - if (!(type = strdup(vm->def->os.type))) - virReportOOMError(); + ignore_value(VIR_STRDUP(type, vm->def->os.type)); cleanup: if (vm)