From nobody Sun Feb 8 12:19:19 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 207.211.31.81 as permitted sender) client-ip=207.211.31.81; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-1.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zoho.com: domain of redhat.com designates 207.211.31.81 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1571576214; cv=none; d=zoho.com; s=zohoarc; b=KiKi3ySMRwOU0w8JufxtKEtdBmeMirvAWUDgukKZelMxqdEoyMiKHjdGoixQURlkT5gnuqdwTP8F35yI8Zljgj69a4a2XDeCnnmbih16R50lMhKBddQWEYszBf3YG2PiohFRVQcCF+/O0bHS+i8ICoN/N0ktt01LNw+YJHCE7Qk= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1571576214; h=Content-Type:Content-Transfer-Encoding:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=mYuQ0f/c951Wg8tZ4qt2iTpBJ/uLuzIBH99nJvwmUO8=; b=LIFuMjWDS1egb1b0VMIPCy2VjDtqfO/173eB46JMEHGY8ZI4baX1A1QTC/sBI1Mtit7iehCugousWARALK16xynyCs4S1xCs6aqg3FKLJqAxS+q3JmyiVhctTwNShC4gCBZAua8TkciqOo265bq04EpLkcDyp/4uaM/NywUzV3M= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=pass; spf=pass (zoho.com: domain of redhat.com designates 207.211.31.81 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) header.from= Return-Path: Received: from us-smtp-delivery-1.mimecast.com (us-smtp-1.mimecast.com [207.211.31.81]) by mx.zohomail.com with SMTPS id 1571576214639238.26774387263617; Sun, 20 Oct 2019 05:56:54 -0700 (PDT) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-217-RxEtWuw4OByimdVwREX7uQ-1; Sun, 20 Oct 2019 08:56:50 -0400 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 6329D8797DE; Sun, 20 Oct 2019 12:56:46 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.20]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 3A01360C4E; Sun, 20 Oct 2019 12:56:46 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id F3B5F180BA9C; Sun, 20 Oct 2019 12:56:45 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id x9KCuDr2023645 for ; Sun, 20 Oct 2019 08:56:13 -0400 Received: by smtp.corp.redhat.com (Postfix) id E67283DA3; Sun, 20 Oct 2019 12:56:13 +0000 (UTC) Received: from lpt.redhat.com (ovpn-200-23.brq.redhat.com [10.40.200.23]) by smtp.corp.redhat.com (Postfix) with ESMTP id 46ED5194B9 for ; Sun, 20 Oct 2019 12:56:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1571576213; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=mYuQ0f/c951Wg8tZ4qt2iTpBJ/uLuzIBH99nJvwmUO8=; b=R/lpxghVlzw6VpMEYKptb/kBnf8eglx4Gf0K302IisC/QpPAYIhC4Bax0loV/CN2t6KA0c O7zyGLqnsKhyf4hRKWcXk3QXLZwbrnqd+nlPuL7cIQJCee4YW517VfYp2i8Jp4v6tRM7SW sZvzClgbIyxoFeaDL3KXUrwOS/gT3fw= From: =?UTF-8?q?J=C3=A1n=20Tomko?= To: libvir-list@redhat.com Date: Sun, 20 Oct 2019 14:55:50 +0200 Message-Id: <330122d2ee2eb510df605751045ce31bc690165a.1571575890.git.jtomko@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-loop: libvir-list@redhat.com Subject: [libvirt] [PATCH 32/34] tests: use g_strdup instead of VIR_STRDUP X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-MC-Unique: RxEtWuw4OByimdVwREX7uQ-1 X-Mimecast-Spam-Score: 0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) Content-Type: text/plain; charset="utf-8" Replace all occurrences of if (VIR_STRDUP(a, b) < 0) /* effectively dead code */ with: a =3D g_strdup(b); Signed-off-by: J=C3=A1n Tomko --- tests/bhyvexml2argvmock.c | 6 +-- tests/commandtest.c | 5 +-- tests/cputest.c | 4 +- tests/domaincapstest.c | 13 +++--- tests/nssmock.c | 3 +- tests/openvzutilstest.c | 5 ++- tests/qemublocktest.c | 3 +- tests/qemudomaincheckpointxml2xmltest.c | 9 ++-- tests/qemufirmwaretest.c | 3 +- tests/qemuhotplugtest.c | 8 +--- tests/qemumemlocktest.c | 5 +-- tests/qemumonitorjsontest.c | 3 +- tests/qemumonitortestutils.c | 32 ++++++-------- tests/qemusecuritymock.c | 6 +-- tests/qemuvhostusertest.c | 3 +- tests/qemuxml2argvtest.c | 55 ++++++++----------------- tests/qemuxml2xmltest.c | 5 +-- tests/scsihosttest.c | 5 +-- tests/securityselinuxlabeltest.c | 8 +--- tests/securityselinuxtest.c | 13 +++--- tests/storagebackendsheepdogtest.c | 6 +-- tests/testutils.c | 3 +- tests/testutilsqemu.c | 31 +++++--------- tests/vboxsnapshotxmltest.c | 4 +- tests/viralloctest.c | 3 +- tests/vircgrouptest.c | 5 +-- tests/virfilecachetest.c | 7 +--- tests/virfirewalltest.c | 8 ++-- tests/virhostdevtest.c | 8 +--- tests/virnetmessagetest.c | 11 ++--- tests/virpcimock.c | 19 +++++---- tests/virpcitest.c | 5 +-- tests/virscsitest.c | 5 +-- tests/virstoragetest.c | 6 +-- tests/virstringtest.c | 9 ++-- tests/virtestmock.c | 3 +- tests/virusbmock.c | 4 +- tests/vmx2xmltest.c | 3 +- tests/xml2vmxtest.c | 3 +- 39 files changed, 115 insertions(+), 222 deletions(-) diff --git a/tests/bhyvexml2argvmock.c b/tests/bhyvexml2argvmock.c index 717ceb4847..b0d38187f9 100644 --- a/tests/bhyvexml2argvmock.c +++ b/tests/bhyvexml2argvmock.c @@ -34,8 +34,7 @@ int virNetDevTapCreateInBridgePort(const char *brname G_G= NUC_UNUSED, unsigned int fakeflags G_GNUC_UNUSED) { VIR_FREE(*ifname); - if (VIR_STRDUP(*ifname, "vnet0") < 0) - return -1; + *ifname =3D g_strdup("vnet0"); return 0; } =20 @@ -43,8 +42,7 @@ char *virNetDevTapGetRealDeviceName(char *name G_GNUC_UNU= SED) { char *fakename; =20 - if (VIR_STRDUP(fakename, "faketapdev") < 0) - return NULL; + fakename =3D g_strdup("faketapdev"); return fakename; } =20 diff --git a/tests/commandtest.c b/tests/commandtest.c index e79baa2946..6ff2039ab1 100644 --- a/tests/commandtest.c +++ b/tests/commandtest.c @@ -686,10 +686,7 @@ static int test17(const void *unused G_GNUC_UNUSED) goto cleanup; } VIR_FREE(outbuf); - if (VIR_STRDUP(outbuf, "should not be leaked") < 0) { - puts("test framework failure"); - goto cleanup; - } + outbuf =3D g_strdup("should not be leaked"); =20 virCommandSetErrorBuffer(cmd, &errbuf); if (errbuf !=3D NULL) { diff --git a/tests/cputest.c b/tests/cputest.c index ab68c3b4af..83b494857f 100644 --- a/tests/cputest.c +++ b/tests/cputest.c @@ -492,9 +492,11 @@ cpuTestMakeQEMUCaps(const struct data *data) if (!(testMon =3D qemuMonitorTestNewFromFile(json, driver.xmlopt, true= ))) goto error; =20 - if (VIR_ALLOC(cpu) < 0 || VIR_STRDUP(cpu->model, "host") < 0) + if (VIR_ALLOC(cpu) < 0) goto cleanup; =20 + cpu->model =3D g_strdup("host"); + if (ARCH_IS_S390(data->arch)) fail_no_props =3D false; =20 diff --git a/tests/domaincapstest.c b/tests/domaincapstest.c index 07bc99cc03..61bc019671 100644 --- a/tests/domaincapstest.c +++ b/tests/domaincapstest.c @@ -36,11 +36,11 @@ fillStringValues(virDomainCapsStringValuesPtr values, .= ..) =20 va_start(list, values); while ((str =3D va_arg(list, const char *))) { - if (VIR_REALLOC_N(values->values, values->nvalues + 1) < 0 || - VIR_STRDUP(values->values[values->nvalues], str) < 0) { + if (VIR_REALLOC_N(values->values, values->nvalues + 1) < 0) { ret =3D -1; break; } + values->values[values->nvalues] =3D g_strdup(str); values->nvalues++; } va_end(list); @@ -95,9 +95,7 @@ fillQemuCaps(virDomainCapsPtr domCaps, =20 if (machine) { VIR_FREE(domCaps->machine); - if (VIR_STRDUP(domCaps->machine, - virQEMUCapsGetCanonicalMachine(qemuCaps, machine)) = < 0) - goto cleanup; + domCaps->machine =3D g_strdup(virQEMUCapsGetCanonicalMachine(qemuC= aps, machine)); } =20 if (!domCaps->machine) @@ -153,9 +151,8 @@ fillXenCaps(virDomainCapsPtr domCaps) =20 if (VIR_ALLOC(firmwares[0]) < 0 || VIR_ALLOC(firmwares[1]) < 0) goto cleanup; - if (VIR_STRDUP(firmwares[0]->name, "/usr/lib/xen/boot/hvmloader") < 0 = || - VIR_STRDUP(firmwares[1]->name, "/usr/lib/xen/boot/ovmf.bin") < 0) - goto cleanup; + firmwares[0]->name =3D g_strdup("/usr/lib/xen/boot/hvmloader"); + firmwares[1]->name =3D g_strdup("/usr/lib/xen/boot/ovmf.bin"); =20 if (libxlMakeDomainCapabilities(domCaps, firmwares, 2) < 0) goto cleanup; diff --git a/tests/nssmock.c b/tests/nssmock.c index 230044275a..ea4b26ffab 100644 --- a/tests/nssmock.c +++ b/tests/nssmock.c @@ -62,8 +62,7 @@ getrealpath(char **newpath, return -1; } } else { - if (VIR_STRDUP_QUIET(*newpath, path) < 0) - return -1; + *newpath =3D g_strdup(path); } =20 return 0; diff --git a/tests/openvzutilstest.c b/tests/openvzutilstest.c index 1fcd4d2caa..c15ce9a47b 100644 --- a/tests/openvzutilstest.c +++ b/tests/openvzutilstest.c @@ -99,10 +99,11 @@ testReadNetworkConf(const void *data G_GNUC_UNUSED) " \n" "\n"; =20 - if (!(def =3D virDomainDefNew()) || - VIR_STRDUP(def->os.init, "/sbin/init") < 0) + if (!(def =3D virDomainDefNew())) goto cleanup; =20 + def->os.init =3D g_strdup("/sbin/init"); + def->virtType =3D VIR_DOMAIN_VIRT_OPENVZ; def->os.type =3D VIR_DOMAIN_OSTYPE_EXE; =20 diff --git a/tests/qemublocktest.c b/tests/qemublocktest.c index 0f073682ca..a8d2f57c6e 100644 --- a/tests/qemublocktest.c +++ b/tests/qemublocktest.c @@ -161,8 +161,7 @@ testQemuDiskXMLToJSONFakeSecrets(virStorageSourcePtr sr= c) return -1; =20 srcpriv->secinfo->type =3D VIR_DOMAIN_SECRET_INFO_TYPE_AES; - if (VIR_STRDUP(srcpriv->secinfo->s.aes.username, src->auth->userna= me) < 0) - return -1; + srcpriv->secinfo->s.aes.username =3D g_strdup(src->auth->username); =20 if (virAsprintf(&srcpriv->secinfo->s.aes.alias, "%s-secalias", NULLSTR(src->nodestorage)) < 0) diff --git a/tests/qemudomaincheckpointxml2xmltest.c b/tests/qemudomainchec= kpointxml2xmltest.c index f6eb1f69b2..4540f15ddd 100644 --- a/tests/qemudomaincheckpointxml2xmltest.c +++ b/tests/qemudomaincheckpointxml2xmltest.c @@ -63,8 +63,7 @@ testCompareXMLToXMLFiles(const char *inxml, if (flags & TEST_PARENT) { if (def->parent.parent_name) return -1; - if (VIR_STRDUP(def->parent.parent_name, "1525111885") < 0) - return -1; + def->parent.parent_name =3D g_strdup("1525111885"); } if (flags & TEST_VDA_BITMAP) { virDomainCheckpointDiskDefPtr disk; @@ -76,13 +75,11 @@ testCompareXMLToXMLFiles(const char *inxml, return -1; if (!disk->name) { disk->type =3D VIR_DOMAIN_CHECKPOINT_TYPE_BITMAP; - if (VIR_STRDUP(disk->name, "vda") < 0) - return -1; + disk->name =3D g_strdup("vda"); } else if (STRNEQ(disk->name, "vda")) { return -1; } - if (VIR_STRDUP(disk->bitmap, def->parent.name) < 0) - return -1; + disk->bitmap =3D g_strdup(def->parent.name); } if (flags & TEST_SIZE) { def->disks[0].size =3D 1048576; diff --git a/tests/qemufirmwaretest.c b/tests/qemufirmwaretest.c index 9287ea554b..ad7ed7a384 100644 --- a/tests/qemufirmwaretest.c +++ b/tests/qemufirmwaretest.c @@ -70,8 +70,7 @@ testFWPrecedence(const void *opaque G_GNUC_UNUSED) }; const size_t nexpected =3D G_N_ELEMENTS(expected); =20 - if (VIR_STRDUP(fakehome, abs_srcdir "/qemufirmwaredata/home/user/.conf= ig") < 0) - return -1; + fakehome =3D g_strdup(abs_srcdir "/qemufirmwaredata/home/user/.config"= ); =20 setenv("XDG_CONFIG_HOME", fakehome, 1); =20 diff --git a/tests/qemuhotplugtest.c b/tests/qemuhotplugtest.c index d3da08875a..4ff2b38c83 100644 --- a/tests/qemuhotplugtest.c +++ b/tests/qemuhotplugtest.c @@ -597,10 +597,7 @@ mymain(void) struct testQemuHotplugCpuParams cpudata; char *fakerootdir; =20 - if (VIR_STRDUP_QUIET(fakerootdir, FAKEROOTDIRTEMPLATE) < 0) { - fprintf(stderr, "Out of memory\n"); - abort(); - } + fakerootdir =3D g_strdup(FAKEROOTDIRTEMPLATE); =20 if (!mkdtemp(fakerootdir)) { fprintf(stderr, "Cannot create fakerootdir"); @@ -622,8 +619,7 @@ mymain(void) VIR_FREE(driver.config->spiceListen); VIR_FREE(driver.config->vncListen); /* some dummy values from 'config file' */ - if (VIR_STRDUP_QUIET(driver.config->spicePassword, "123456") < 0) - return EXIT_FAILURE; + driver.config->spicePassword =3D g_strdup("123456"); =20 if (!(driver.domainEventState =3D virObjectEventStateNew())) return EXIT_FAILURE; diff --git a/tests/qemumemlocktest.c b/tests/qemumemlocktest.c index c9484ac9cb..490e2ef11f 100644 --- a/tests/qemumemlocktest.c +++ b/tests/qemumemlocktest.c @@ -61,10 +61,7 @@ mymain(void) char *fakerootdir; virQEMUCapsPtr qemuCaps =3D NULL; =20 - if (VIR_STRDUP_QUIET(fakerootdir, FAKEROOTDIRTEMPLATE) < 0) { - fprintf(stderr, "Out of memory\n"); - abort(); - } + fakerootdir =3D g_strdup(FAKEROOTDIRTEMPLATE); =20 if (!mkdtemp(fakerootdir)) { fprintf(stderr, "Cannot create fakerootdir"); diff --git a/tests/qemumonitorjsontest.c b/tests/qemumonitorjsontest.c index f754e4d94b..c0eee921cf 100644 --- a/tests/qemumonitorjsontest.c +++ b/tests/qemumonitorjsontest.c @@ -2165,8 +2165,7 @@ testQemuMonitorJSONqemuMonitorJSONSetBlockIoThrottle(= const void *opaque) return -1; =20 expectedInfo =3D (virDomainBlockIoTuneInfo) {1, 2, 3, 4, 5, 6, 7, 8, 9= , 10, 11, 12, 13, NULL, 15, 16, 17, 18, 19, 20}; - if (VIR_STRDUP(expectedInfo.group_name, "group14") < 0) - return -1; + expectedInfo.group_name =3D g_strdup("group14"); =20 if (qemuMonitorTestAddItem(test, "query-block", queryBlockReply) < 0 || qemuMonitorTestAddItemParams(test, "block_set_io_throttle", diff --git a/tests/qemumonitortestutils.c b/tests/qemumonitortestutils.c index 6b3aa97e39..fb76bf185d 100644 --- a/tests/qemumonitortestutils.c +++ b/tests/qemumonitortestutils.c @@ -604,11 +604,8 @@ qemuMonitorTestAddItem(qemuMonitorTestPtr test, if (VIR_ALLOC(data) < 0) return -1; =20 - if (VIR_STRDUP(data->command_name, command_name) < 0 || - VIR_STRDUP(data->response, response) < 0) { - qemuMonitorTestHandlerDataFree(data); - return -1; - } + data->command_name =3D g_strdup(command_name); + data->response =3D g_strdup(response); =20 return qemuMonitorTestAddHandler(test, qemuMonitorTestProcessCommandDefault, @@ -692,9 +689,8 @@ qemuMonitorTestAddItemVerbatim(qemuMonitorTestPtr test, if (VIR_ALLOC(data) < 0) return -1; =20 - if (VIR_STRDUP(data->response, response) < 0 || - VIR_STRDUP(data->cmderr, cmderr) < 0) - goto error; + data->response =3D g_strdup(response); + data->cmderr =3D g_strdup(cmderr); =20 data->command_name =3D virJSONStringReformat(command, false); if (!data->command_name) @@ -868,9 +864,8 @@ qemuMonitorTestAddItemParams(qemuMonitorTestPtr test, if (VIR_ALLOC(data) < 0) goto error; =20 - if (VIR_STRDUP(data->command_name, cmdname) < 0 || - VIR_STRDUP(data->response, response) < 0) - goto error; + data->command_name =3D g_strdup(cmdname); + data->response =3D g_strdup(response); =20 while ((argname =3D va_arg(args, char *))) { size_t i; @@ -885,9 +880,8 @@ qemuMonitorTestAddItemParams(qemuMonitorTestPtr test, if (VIR_EXPAND_N(data->args, data->nargs, 1)) goto error; =20 - if (VIR_STRDUP(data->args[i].argname, argname) < 0 || - VIR_STRDUP(data->args[i].argval, argval) < 0) - goto error; + data->args[i].argname =3D g_strdup(argname); + data->args[i].argval =3D g_strdup(argval); } =20 va_end(args); @@ -984,10 +978,9 @@ qemuMonitorTestAddItemExpect(qemuMonitorTestPtr test, if (VIR_ALLOC(data) < 0) goto error; =20 - if (VIR_STRDUP(data->command_name, cmdname) < 0 || - VIR_STRDUP(data->response, response) < 0 || - VIR_STRDUP(data->expectArgs, cmdargs) < 0) - goto error; + data->command_name =3D g_strdup(cmdname); + data->response =3D g_strdup(response); + data->expectArgs =3D g_strdup(cmdargs); =20 if (apostrophe) { char *tmp =3D data->expectArgs; @@ -1065,8 +1058,7 @@ qemuMonitorCommonTestNew(virDomainXMLOptionPtr xmlopt, return NULL; } =20 - if (VIR_STRDUP(tmpdir_template, "/tmp/libvirt_XXXXXX") < 0) - goto error; + tmpdir_template =3D g_strdup("/tmp/libvirt_XXXXXX"); =20 if (!(test->tmpdir =3D mkdtemp(tmpdir_template))) { virReportSystemError(errno, "%s", diff --git a/tests/qemusecuritymock.c b/tests/qemusecuritymock.c index 8b857b0cc1..50435bbd02 100644 --- a/tests/qemusecuritymock.c +++ b/tests/qemusecuritymock.c @@ -146,8 +146,7 @@ virFileGetXAttrQuiet(const char *path, goto cleanup; } =20 - if (VIR_STRDUP(*value, val) < 0) - goto cleanup; + *value =3D g_strdup(val); =20 ret =3D 0; cleanup: @@ -194,8 +193,7 @@ int virFileSetXAttr(const char *path, char *val; =20 key =3D get_key(path, name); - if (VIR_STRDUP(val, value) < 0) - return -1; + val =3D g_strdup(value); =20 virMutexLock(&m); init_syms(); diff --git a/tests/qemuvhostusertest.c b/tests/qemuvhostusertest.c index 0ee96d9ecc..3e1890a88f 100644 --- a/tests/qemuvhostusertest.c +++ b/tests/qemuvhostusertest.c @@ -68,8 +68,7 @@ testVUPrecedence(const void *opaque G_GNUC_UNUSED) }; const size_t nexpected =3D G_N_ELEMENTS(expected); =20 - if (VIR_STRDUP(fakehome, abs_srcdir "/qemuvhostuserdata/home/user/.con= fig") < 0) - return -1; + fakehome =3D g_strdup(abs_srcdir "/qemuvhostuserdata/home/user/.config= "); =20 setenv("XDG_CONFIG_HOME", fakehome, 1); =20 diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c index 7bbc44451d..122e14b071 100644 --- a/tests/qemuxml2argvtest.c +++ b/tests/qemuxml2argvtest.c @@ -43,8 +43,7 @@ fakeSecretGetValue(virSecretPtr obj G_GNUC_UNUSED, unsigned int internalFlags G_GNUC_UNUSED) { char *secret; - if (VIR_STRDUP(secret, "AQCVn5hO6HzFAhAAq0NCv8jtJcIcE+HOBlMQ1A") < 0) - return NULL; + secret =3D g_strdup("AQCVn5hO6HzFAhAAq0NCv8jtJcIcE+HOBlMQ1A"); *value_size =3D strlen(secret); return (unsigned char *) secret; } @@ -517,9 +516,7 @@ testCompareXMLToArgv(const void *data) switch (vm->def->tpm->type) { case VIR_DOMAIN_TPM_TYPE_EMULATOR: VIR_FREE(vm->def->tpm->data.emulator.source.data.file.path); - if (VIR_STRDUP(vm->def->tpm->data.emulator.source.data.file.pa= th, - "/dev/test") < 0) - goto cleanup; + vm->def->tpm->data.emulator.source.data.file.path =3D g_strdup= ("/dev/test"); vm->def->tpm->data.emulator.source.type =3D VIR_DOMAIN_CHR_TYP= E_FILE; break; case VIR_DOMAIN_TPM_TYPE_PASSTHROUGH: @@ -620,10 +617,7 @@ mymain(void) char *fakerootdir; virHashTablePtr capslatest =3D NULL; =20 - if (VIR_STRDUP_QUIET(fakerootdir, FAKEROOTDIRTEMPLATE) < 0) { - fprintf(stderr, "Out of memory\n"); - abort(); - } + fakerootdir =3D g_strdup(FAKEROOTDIRTEMPLATE); =20 if (!mkdtemp(fakerootdir)) { fprintf(stderr, "Cannot create fakerootdir"); @@ -648,43 +642,33 @@ mymain(void) driver.privileged =3D true; =20 VIR_FREE(driver.config->defaultTLSx509certdir); - if (VIR_STRDUP_QUIET(driver.config->defaultTLSx509certdir, "/etc/pki/q= emu") < 0) - return EXIT_FAILURE; + driver.config->defaultTLSx509certdir =3D g_strdup("/etc/pki/qemu"); VIR_FREE(driver.config->vncTLSx509certdir); - if (VIR_STRDUP_QUIET(driver.config->vncTLSx509certdir, "/etc/pki/libvi= rt-vnc") < 0) - return EXIT_FAILURE; + driver.config->vncTLSx509certdir =3D g_strdup("/etc/pki/libvirt-vnc"); VIR_FREE(driver.config->spiceTLSx509certdir); - if (VIR_STRDUP_QUIET(driver.config->spiceTLSx509certdir, "/etc/pki/lib= virt-spice") < 0) - return EXIT_FAILURE; + driver.config->spiceTLSx509certdir =3D g_strdup("/etc/pki/libvirt-spic= e"); VIR_FREE(driver.config->chardevTLSx509certdir); - if (VIR_STRDUP_QUIET(driver.config->chardevTLSx509certdir, "/etc/pki/l= ibvirt-chardev") < 0) - return EXIT_FAILURE; + driver.config->chardevTLSx509certdir =3D g_strdup("/etc/pki/libvirt-ch= ardev"); VIR_FREE(driver.config->vxhsTLSx509certdir); - if (VIR_STRDUP_QUIET(driver.config->vxhsTLSx509certdir, "/etc/pki/libv= irt-vxhs/dummy,path") < 0) - return EXIT_FAILURE; + driver.config->vxhsTLSx509certdir =3D g_strdup("/etc/pki/libvirt-vxhs/= dummy,path"); VIR_FREE(driver.config->nbdTLSx509certdir); - if (VIR_STRDUP_QUIET(driver.config->nbdTLSx509certdir, "/etc/pki/libvi= rt-nbd/dummy,path") < 0) - return EXIT_FAILURE; + driver.config->nbdTLSx509certdir =3D g_strdup("/etc/pki/libvirt-nbd/du= mmy,path"); =20 VIR_FREE(driver.config->hugetlbfs); if (VIR_ALLOC_N(driver.config->hugetlbfs, 2) < 0) return EXIT_FAILURE; driver.config->nhugetlbfs =3D 2; - if (VIR_STRDUP(driver.config->hugetlbfs[0].mnt_dir, "/dev/hugepages2M"= ) < 0 || - VIR_STRDUP(driver.config->hugetlbfs[1].mnt_dir, "/dev/hugepages1G"= ) < 0) - return EXIT_FAILURE; + driver.config->hugetlbfs[0].mnt_dir =3D g_strdup("/dev/hugepages2M"); + driver.config->hugetlbfs[1].mnt_dir =3D g_strdup("/dev/hugepages1G"); driver.config->hugetlbfs[0].size =3D 2048; driver.config->hugetlbfs[0].deflt =3D true; driver.config->hugetlbfs[1].size =3D 1048576; driver.config->spiceTLS =3D 1; - if (VIR_STRDUP_QUIET(driver.config->spicePassword, "123456") < 0) - return EXIT_FAILURE; + driver.config->spicePassword =3D g_strdup("123456"); VIR_FREE(driver.config->memoryBackingDir); - if (VIR_STRDUP_QUIET(driver.config->memoryBackingDir, "/var/lib/libvir= t/qemu/ram") < 0) - return EXIT_FAILURE; + driver.config->memoryBackingDir =3D g_strdup("/var/lib/libvirt/qemu/ra= m"); VIR_FREE(driver.config->nvramDir); - if (VIR_STRDUP(driver.config->nvramDir, "/var/lib/libvirt/qemu/nvram")= < 0) - return EXIT_FAILURE; + driver.config->nvramDir =3D g_strdup("/var/lib/libvirt/qemu/nvram"); =20 capslatest =3D testQemuGetLatestCaps(); if (!capslatest) @@ -1222,9 +1206,7 @@ mymain(void) DO_TEST("graphics-vnc-tls", QEMU_CAPS_VNC, QEMU_CAPS_DEVICE_CIRRUS_VGA= ); DO_TEST_CAPS_VER("graphics-vnc-tls", "2.4.0"); DO_TEST_CAPS_LATEST("graphics-vnc-tls"); - if (VIR_STRDUP_QUIET(driver.config->vncTLSx509secretUUID, - "6fd3f62d-9fe7-4a4e-a869-7acd6376d8ea") < 0) - return EXIT_FAILURE; + driver.config->vncTLSx509secretUUID =3D g_strdup("6fd3f62d-9fe7-4a4e-a= 869-7acd6376d8ea"); DO_TEST_CAPS_LATEST("graphics-vnc-tls-secret"); VIR_FREE(driver.config->vncTLSx509secretUUID); driver.config->vncSASL =3D driver.config->vncTLSx509verify =3D driver.= config->vncTLS =3D 0; @@ -1407,11 +1389,8 @@ mymain(void) QEMU_CAPS_DEVICE_ISA_SERIAL, QEMU_CAPS_OBJECT_TLS_CREDS_X509); VIR_FREE(driver.config->chardevTLSx509certdir); - if (VIR_STRDUP_QUIET(driver.config->chardevTLSx509certdir, "/etc/pki/l= ibvirt-chardev") < 0) - return EXIT_FAILURE; - if (VIR_STRDUP_QUIET(driver.config->chardevTLSx509secretUUID, - "6fd3f62d-9fe7-4a4e-a869-7acd6376d8ea") < 0) - return EXIT_FAILURE; + driver.config->chardevTLSx509certdir =3D g_strdup("/etc/pki/libvirt-ch= ardev"); + driver.config->chardevTLSx509secretUUID =3D g_strdup("6fd3f62d-9fe7-4a= 4e-a869-7acd6376d8ea"); DO_TEST("serial-tcp-tlsx509-secret-chardev", QEMU_CAPS_OBJECT_SECRET, QEMU_CAPS_DEVICE_ISA_SERIAL, diff --git a/tests/qemuxml2xmltest.c b/tests/qemuxml2xmltest.c index 4c7ba98367..e7c029a74d 100644 --- a/tests/qemuxml2xmltest.c +++ b/tests/qemuxml2xmltest.c @@ -157,10 +157,7 @@ mymain(void) if (!capslatest) return EXIT_FAILURE; =20 - if (VIR_STRDUP_QUIET(fakerootdir, FAKEROOTDIRTEMPLATE) < 0) { - fprintf(stderr, "Out of memory\n"); - abort(); - } + fakerootdir =3D g_strdup(FAKEROOTDIRTEMPLATE); =20 if (!mkdtemp(fakerootdir)) { fprintf(stderr, "Cannot create fakerootdir"); diff --git a/tests/scsihosttest.c b/tests/scsihosttest.c index c1a7e2d603..35c7d93376 100644 --- a/tests/scsihosttest.c +++ b/tests/scsihosttest.c @@ -253,10 +253,7 @@ mymain(void) char *fakerootdir =3D NULL; char *fakesysfsdir =3D NULL; =20 - if (VIR_STRDUP_QUIET(fakerootdir, FAKEROOTDIRTEMPLATE) < 0) { - fprintf(stderr, "Out of memory\n"); - goto cleanup; - } + fakerootdir =3D g_strdup(FAKEROOTDIRTEMPLATE); =20 if (!mkdtemp(fakerootdir)) { fprintf(stderr, "Cannot create fakerootdir"); diff --git a/tests/securityselinuxlabeltest.c b/tests/securityselinuxlabelt= est.c index 6f9b5c0e70..9156403c51 100644 --- a/tests/securityselinuxlabeltest.c +++ b/tests/securityselinuxlabeltest.c @@ -144,10 +144,7 @@ testSELinuxLoadFileList(const char *testname, abs_builddir, line) < 0) goto cleanup; if (*tmp !=3D '\0' && *tmp !=3D '\n') { - if (VIR_STRDUP(context, tmp) < 0) { - VIR_FREE(file); - goto cleanup; - } + context =3D g_strdup(tmp); =20 tmp =3D strchr(context, '\n'); if (tmp) @@ -270,8 +267,7 @@ testSELinuxCheckLabels(testSELinuxFile *files, size_t n= files) if (errno =3D=3D ENODATA) { /* nothing to do */ } else if (errno =3D=3D EOPNOTSUPP) { - if (VIR_STRDUP(ctx, "EOPNOTSUPP") < 0) - return -1; + ctx =3D g_strdup("EOPNOTSUPP"); } else { virReportSystemError(errno, "Cannot read label on %s", diff --git a/tests/securityselinuxtest.c b/tests/securityselinuxtest.c index 5dccb708d8..6c8314de6b 100644 --- a/tests/securityselinuxtest.c +++ b/tests/securityselinuxtest.c @@ -77,17 +77,14 @@ testBuildDomainDef(bool dynamic, if (VIR_ALLOC(secdef) < 0) goto error; =20 - if (VIR_STRDUP(secdef->model, "selinux") < 0) - goto error; + secdef->model =3D g_strdup("selinux"); =20 secdef->type =3D dynamic ? VIR_DOMAIN_SECLABEL_DYNAMIC : VIR_DOMAIN_SE= CLABEL_STATIC; - if (label && - VIR_STRDUP(secdef->label, label) < 0) - goto error; + if (label) + secdef->label =3D g_strdup(label); =20 - if (baselabel && - VIR_STRDUP(secdef->baselabel, baselabel) < 0) - goto error; + if (baselabel) + secdef->baselabel =3D g_strdup(baselabel); =20 def->seclabels[0] =3D secdef; def->nseclabels++; diff --git a/tests/storagebackendsheepdogtest.c b/tests/storagebackendsheep= dogtest.c index 6504881920..a5403e7136 100644 --- a/tests/storagebackendsheepdogtest.c +++ b/tests/storagebackendsheepdogtest.c @@ -63,8 +63,7 @@ test_node_info_parser(const void *opaque) if (!(pool =3D virStoragePoolDefParseFile(data->poolxml))) return -1; =20 - if (VIR_STRDUP(output, test.output) < 0) - return -1; + output =3D g_strdup(test.output); =20 if (virStorageBackendSheepdogParseNodeInfo(pool, output) !=3D test.expected_return) @@ -95,8 +94,7 @@ test_vdi_list_parser(const void *opaque) if (!(vol =3D virStorageVolDefParseFile(pool, data->volxml, 0))) return -1; =20 - if (VIR_STRDUP(output, test.output) < 0) - return -1; + output =3D g_strdup(test.output); =20 if (virStorageBackendSheepdogParseVdiList(vol, output) !=3D test.expected_return) diff --git a/tests/testutils.c b/tests/testutils.c index 1c89d503d9..a141abc91b 100644 --- a/tests/testutils.c +++ b/tests/testutils.c @@ -816,8 +816,7 @@ virTestSetEnvPath(void) virAsprintf(&new_path, "%s:%s", abs_builddir, path) < 0) goto cleanup; } else { - if (VIR_STRDUP(new_path, abs_builddir) < 0) - goto cleanup; + new_path =3D g_strdup(abs_builddir); } =20 if (new_path && diff --git a/tests/testutilsqemu.c b/tests/testutilsqemu.c index e6eca5cb2e..0faa9cb84a 100644 --- a/tests/testutilsqemu.c +++ b/tests/testutilsqemu.c @@ -74,8 +74,7 @@ static virCapsGuestMachinePtr *testQemuAllocNewerMachines= (int *nmachines) "pc-0.11", "pc", "pc-0.10", "isapc" }; =20 - if (VIR_STRDUP(canonical, x86_machines[0]) < 0) - return NULL; + canonical =3D g_strdup(x86_machines[0]); =20 machines =3D virCapabilitiesAllocMachines(x86_machines, G_N_ELEMENTS(x86_machines)); @@ -503,9 +502,8 @@ virCapsPtr testQemuCapsInit(void) goto cleanup; caps->host.nsecModels =3D 1; =20 - if (VIR_STRDUP(caps->host.secModels[0].model, "none") < 0 || - VIR_STRDUP(caps->host.secModels[0].doi, "0") < 0) - goto cleanup; + caps->host.secModels[0].model =3D g_strdup("none"); + caps->host.secModels[0].doi =3D g_strdup("0"); =20 if (!(cpuDefault =3D virCPUDefCopy(&cpuDefaultData)) || !(cpuHaswell =3D virCPUDefCopy(&cpuHaswellData)) || @@ -710,29 +708,22 @@ int qemuTestDriverInit(virQEMUDriver *driver) /* Overwrite some default paths so it's consistent for tests. */ VIR_FREE(driver->config->libDir); VIR_FREE(driver->config->channelTargetDir); - if (VIR_STRDUP(driver->config->libDir, "/tmp/lib") < 0 || - VIR_STRDUP(driver->config->channelTargetDir, "/tmp/channel") < 0) - goto error; + driver->config->libDir =3D g_strdup("/tmp/lib"); + driver->config->channelTargetDir =3D g_strdup("/tmp/channel"); =20 if (!mkdtemp(statedir)) { virFilePrintf(stderr, "Cannot create fake stateDir"); goto error; } =20 - if (VIR_STRDUP(driver->config->stateDir, statedir) < 0) { - rmdir(statedir); - goto error; - } + driver->config->stateDir =3D g_strdup(statedir); =20 if (!mkdtemp(configdir)) { virFilePrintf(stderr, "Cannot create fake configDir"); goto error; } =20 - if (VIR_STRDUP(driver->config->configDir, configdir) < 0) { - rmdir(configdir); - goto error; - } + driver->config->configDir =3D g_strdup(configdir); =20 driver->caps =3D testQemuCapsInit(); if (!driver->caps) @@ -827,10 +818,9 @@ testQemuGetLatestCapsForArch(const char *arch, while ((rc =3D virDirRead(dir, &ent, TEST_QEMU_CAPS_PATH)) > 0) { VIR_FREE(tmp); =20 - if ((rc =3D VIR_STRDUP(tmp, STRSKIP(ent->d_name, "caps_"))) < 0) - goto cleanup; + tmp =3D g_strdup(STRSKIP(ent->d_name, "caps_")); =20 - if (rc =3D=3D 0) + if (!tmp) continue; =20 if (!virStringStripSuffix(tmp, fullsuffix)) @@ -1051,8 +1041,7 @@ testQemuInfoSetArgs(struct testQemuInfo *info, bool stripmachinealiases =3D false; =20 if (STREQ(capsver, "latest")) { - if (VIR_STRDUP(capsfile, virHashLookup(capslatest, capsarch)) = < 0) - goto cleanup; + capsfile =3D g_strdup(virHashLookup(capslatest, capsarch)); stripmachinealiases =3D true; } else if (virAsprintf(&capsfile, "%s/caps_%s.%s.xml", TEST_QEMU_CAPS_PATH, capsver, capsarch) < 0= ) { diff --git a/tests/vboxsnapshotxmltest.c b/tests/vboxsnapshotxmltest.c index c70d7ab100..3d6171e266 100644 --- a/tests/vboxsnapshotxmltest.c +++ b/tests/vboxsnapshotxmltest.c @@ -58,9 +58,7 @@ testCompareXMLtoXMLFiles(const char *xml) int ret =3D -1; virVBoxSnapshotConfMachinePtr machine =3D NULL; =20 - if (VIR_STRDUP(pathResult, - abs_builddir "/vboxsnapshotxmldata/testResult.vbox") < = 0) - return -1; + pathResult =3D g_strdup(abs_builddir "/vboxsnapshotxmldata/testResult.= vbox"); =20 if (virFileMakePath(abs_builddir "/vboxsnapshotxmldata") < 0) goto cleanup; diff --git a/tests/viralloctest.c b/tests/viralloctest.c index 75e639675b..c52483998b 100644 --- a/tests/viralloctest.c +++ b/tests/viralloctest.c @@ -409,8 +409,7 @@ testDispose(const void *opaque G_GNUC_UNUSED) =20 VIR_DISPOSE_N(nums, nnums); =20 - if (VIR_STRDUP(str, "test") < 0) - return -1; + str =3D g_strdup("test"); =20 VIR_DISPOSE_STRING(str); =20 diff --git a/tests/vircgrouptest.c b/tests/vircgrouptest.c index cecb15ae5b..cec07580ac 100644 --- a/tests/vircgrouptest.c +++ b/tests/vircgrouptest.c @@ -992,10 +992,7 @@ initFakeFS(const char *mode, { char *fakerootdir; =20 - if (VIR_STRDUP_QUIET(fakerootdir, FAKEROOTDIRTEMPLATE) < 0) { - fprintf(stderr, "Out of memory\n"); - abort(); - } + fakerootdir =3D g_strdup(FAKEROOTDIRTEMPLATE); =20 if (!mkdtemp(fakerootdir)) { fprintf(stderr, "Cannot create fakerootdir"); diff --git a/tests/virfilecachetest.c b/tests/virfilecachetest.c index e3c6401934..6d280b3bec 100644 --- a/tests/virfilecachetest.c +++ b/tests/virfilecachetest.c @@ -71,14 +71,9 @@ testFileCacheObjNew(const char *data) if (!(obj =3D virObjectNew(testFileCacheObjClass))) return NULL; =20 - if (VIR_STRDUP(obj->data, data) < 0) - goto error; + obj->data =3D g_strdup(data); =20 return obj; - - error: - virObjectUnref(obj); - return NULL; } =20 =20 diff --git a/tests/virfirewalltest.c b/tests/virfirewalltest.c index 824e9f46c2..a5323a4256 100644 --- a/tests/virfirewalltest.c +++ b/tests/virfirewalltest.c @@ -961,19 +961,17 @@ testFirewallQueryHook(const char *const*args, const char *input G_GNUC_UNUSED, char **output, char **error G_GNUC_UNUSED, - int *status, + int *status G_GNUC_UNUSED, void *opaque G_GNUC_UNUSED) { if (STREQ(args[0], IPTABLES_PATH) && STREQ(args[1], "-L")) { - if (VIR_STRDUP(*output, TEST_FILTER_TABLE_LIST) < 0) - *status =3D 127; + *output =3D g_strdup(TEST_FILTER_TABLE_LIST); } else if (STREQ(args[0], IPTABLES_PATH) && STREQ(args[1], "-t") && STREQ(args[2], "nat") && STREQ(args[3], "-L")) { - if (VIR_STRDUP(*output, TEST_NAT_TABLE_LIST) < 0) - *status =3D 127; + *output =3D g_strdup(TEST_NAT_TABLE_LIST); } } =20 diff --git a/tests/virhostdevtest.c b/tests/virhostdevtest.c index 8e21d3eae4..1f3f730433 100644 --- a/tests/virhostdevtest.c +++ b/tests/virhostdevtest.c @@ -117,8 +117,7 @@ myInit(void) goto cleanup; if ((mgr->activeSCSIHostdevs =3D virSCSIDeviceListNew()) =3D=3D NULL) goto cleanup; - if (VIR_STRDUP(mgr->stateDir, TEST_STATE_DIR) < 0) - goto cleanup; + mgr->stateDir =3D g_strdup(TEST_STATE_DIR); if (virFileMakePath(mgr->stateDir) < 0) goto cleanup; =20 @@ -501,10 +500,7 @@ mymain(void) int ret =3D 0; g_autofree char *fakerootdir =3D NULL; =20 - if (VIR_STRDUP_QUIET(fakerootdir, FAKEROOTDIRTEMPLATE) < 0) { - fprintf(stderr, "Out of memory\n"); - abort(); - } + fakerootdir =3D g_strdup(FAKEROOTDIRTEMPLATE); =20 if (!mkdtemp(fakerootdir)) { fprintf(stderr, "Cannot create fakerootdir"); diff --git a/tests/virnetmessagetest.c b/tests/virnetmessagetest.c index 53544abbdb..b0a01e8b9d 100644 --- a/tests/virnetmessagetest.c +++ b/tests/virnetmessagetest.c @@ -228,15 +228,16 @@ static int testMessagePayloadEncode(const void *args = G_GNUC_UNUSED) err.level =3D VIR_ERR_ERROR; =20 if (VIR_ALLOC(err.message) < 0 || - VIR_STRDUP(*err.message, "Hello World") < 0 || VIR_ALLOC(err.str1) < 0 || - VIR_STRDUP(*err.str1, "One") < 0 || VIR_ALLOC(err.str2) < 0 || - VIR_STRDUP(*err.str2, "Two") < 0 || - VIR_ALLOC(err.str3) < 0 || - VIR_STRDUP(*err.str3, "Three") < 0) + VIR_ALLOC(err.str3) < 0) goto cleanup; =20 + *err.message =3D g_strdup("Hello World"); + *err.str1 =3D g_strdup("One"); + *err.str2 =3D g_strdup("Two"); + *err.str3 =3D g_strdup("Three"); + err.int1 =3D 1; err.int2 =3D 2; =20 diff --git a/tests/virpcimock.c b/tests/virpcimock.c index ba4256e2fb..d7dc0b00f0 100644 --- a/tests/virpcimock.c +++ b/tests/virpcimock.c @@ -289,8 +289,7 @@ getrealpath(char **newpath, return -1; } } else { - if (VIR_STRDUP_QUIET(*newpath, path) < 0) - return -1; + *newpath =3D g_strdup(path); } =20 return 0; @@ -324,12 +323,12 @@ add_fd(int fd, const char *path) fd, path, cb.fd, cb.path); } =20 - if (VIR_REALLOC_N_QUIET(callbacks, nCallbacks + 1) < 0 || - VIR_STRDUP_QUIET(callbacks[nCallbacks].path, path) < 0) { + if (VIR_REALLOC_N_QUIET(callbacks, nCallbacks + 1) < 0) { errno =3D ENOMEM; goto cleanup; } =20 + callbacks[nCallbacks].path =3D g_strdup(path); callbacks[nCallbacks++].fd =3D fd; ret =3D 0; cleanup: @@ -477,10 +476,11 @@ pci_device_new_from_stub(const struct pciDevice *data) struct stat sb; bool configSrcExists =3D false; =20 - if (!(devid =3D pci_address_format(&data->addr)) || - VIR_STRDUP_QUIET(id, devid) < 0) + if (!(devid =3D pci_address_format(&data->addr))) ABORT_OOM(); =20 + id =3D g_strdup(devid); + /* Replace ':' with '-' to create the config filename from the * device ID. The device ID cannot be used directly as filename * because it contains ':' and Windows does not allow ':' in @@ -724,9 +724,10 @@ pci_driver_new(const char *name, ...) int vendor, device; g_autofree char *driverpath =3D NULL; =20 - if (VIR_ALLOC_QUIET(driver) < 0 || - VIR_STRDUP_QUIET(driver->name, name) < 0 || - !(driverpath =3D pci_driver_get_path(driver, NULL, true))) + if (VIR_ALLOC_QUIET(driver) < 0) + ABORT_OOM(); + driver->name =3D g_strdup(name); + if (!(driverpath =3D pci_driver_get_path(driver, NULL, true))) ABORT_OOM(); =20 if (virFileMakePath(driverpath) < 0) diff --git a/tests/virpcitest.c b/tests/virpcitest.c index 4078dc4c69..de2ffc7d41 100644 --- a/tests/virpcitest.c +++ b/tests/virpcitest.c @@ -328,10 +328,7 @@ mymain(void) int ret =3D 0; char *fakerootdir; =20 - if (VIR_STRDUP_QUIET(fakerootdir, FAKEROOTDIRTEMPLATE) < 0) { - VIR_TEST_DEBUG("Out of memory"); - abort(); - } + fakerootdir =3D g_strdup(FAKEROOTDIRTEMPLATE); =20 if (!mkdtemp(fakerootdir)) { VIR_TEST_DEBUG("Cannot create fakerootdir"); diff --git a/tests/virscsitest.c b/tests/virscsitest.c index d4019e82f7..99374cc80c 100644 --- a/tests/virscsitest.c +++ b/tests/virscsitest.c @@ -222,10 +222,7 @@ mymain(void) =20 VIR_FREE(virscsi_prefix); =20 - if (VIR_STRDUP(virscsi_prefix, tmpdir) < 0) { - ret =3D -1; - goto cleanup; - } + virscsi_prefix =3D g_strdup(tmpdir); =20 if (virTestRun("test1", test1, NULL) < 0) ret =3D -1; diff --git a/tests/virstoragetest.c b/tests/virstoragetest.c index 5d3edbc854..8f1d93fa5a 100644 --- a/tests/virstoragetest.c +++ b/tests/virstoragetest.c @@ -102,8 +102,7 @@ testStorageFileGetMetadata(const char *path, } } =20 - if (VIR_STRDUP(def->path, path) < 0) - return NULL; + def->path =3D g_strdup(path); =20 if (virStorageFileGetMetadata(def, uid, gid, false) < 0) return NULL; @@ -480,8 +479,7 @@ testPathCanonicalizeReadlink(const char *path, =20 for (i =3D 0; i < G_N_ELEMENTS(testPathCanonicalizeSymlinks); i++) { if (STREQ(path, testPathCanonicalizeSymlinks[i][0])) { - if (VIR_STRDUP(*linkpath, testPathCanonicalizeSymlinks[i][1]) = < 0) - return -1; + *linkpath =3D g_strdup(testPathCanonicalizeSymlinks[i][1]); =20 return 0; } diff --git a/tests/virstringtest.c b/tests/virstringtest.c index 5d010f7f59..1e408f2757 100644 --- a/tests/virstringtest.c +++ b/tests/virstringtest.c @@ -718,8 +718,7 @@ static int testStripIPv6Brackets(const void *args) int ret =3D -1; char *res =3D NULL; =20 - if (VIR_STRDUP(res, data->string) < 0) - goto cleanup; + res =3D g_strdup(data->string); =20 virStringStripIPv6Brackets(res); =20 @@ -742,8 +741,7 @@ static int testStripControlChars(const void *args) int ret =3D -1; char *res =3D NULL; =20 - if (VIR_STRDUP(res, data->string) < 0) - goto cleanup; + res =3D g_strdup(data->string); =20 virStringStripControlChars(res); =20 @@ -772,8 +770,7 @@ static int testFilterChars(const void *args) int ret =3D -1; char *res =3D NULL; =20 - if (VIR_STRDUP(res, data->string) < 0) - goto cleanup; + res =3D g_strdup(data->string); =20 virStringFilterChars(res, data->valid); =20 diff --git a/tests/virtestmock.c b/tests/virtestmock.c index 9fe774836b..e0f9c6c2a1 100644 --- a/tests/virtestmock.c +++ b/tests/virtestmock.c @@ -122,8 +122,7 @@ checkPath(const char *path, } else { /* Yeah, our worst nightmares just became true. Path does * not exist. Cut off the last component and retry. */ - if (VIR_STRDUP_QUIET(crippledPath, relPath ? relPath : path) < 0) - goto error; + crippledPath =3D g_strdup(relPath ? relPath : path); =20 virFileRemoveLastComponent(crippledPath); =20 diff --git a/tests/virusbmock.c b/tests/virusbmock.c index f35ad407ad..cace140725 100644 --- a/tests/virusbmock.c +++ b/tests/virusbmock.c @@ -56,8 +56,8 @@ static char *get_fake_path(const char *real_path) if ((p =3D STRSKIP(real_path, USB_SYSFS)) && virAsprintfQuiet(&path, "%s/%s/%s", abs_srcdir, FAKE_USB_SYSFS, p)= < 0) goto error; - else if (!p && VIR_STRDUP_QUIET(path, real_path) < 0) - goto error; + else if (!p) + path =3D g_strdup(real_path); =20 return path; =20 diff --git a/tests/vmx2xmltest.c b/tests/vmx2xmltest.c index 4c5d41aee5..706fec7bac 100644 --- a/tests/vmx2xmltest.c +++ b/tests/vmx2xmltest.c @@ -142,8 +142,7 @@ testParseVMXFileName(const char *fileName, void *opaque= G_GNUC_UNUSED) =20 if (STRPREFIX(fileName, "/vmfs/volumes/")) { /* Found absolute path referencing a file inside a datastore */ - if (VIR_STRDUP(copyOfFileName, fileName) < 0) - goto cleanup; + copyOfFileName =3D g_strdup(fileName); =20 /* Expected format: '/vmfs/volumes//' */ if ((tmp =3D STRSKIP(copyOfFileName, "/vmfs/volumes/")) =3D=3D NUL= L || diff --git a/tests/xml2vmxtest.c b/tests/xml2vmxtest.c index 0cfdb1facf..42ab4ed929 100644 --- a/tests/xml2vmxtest.c +++ b/tests/xml2vmxtest.c @@ -153,8 +153,7 @@ testFormatVMXFileName(const char *src, void *opaque G_G= NUC_UNUSED) =20 if (STRPREFIX(src, "[")) { /* Found potential datastore path */ - if (VIR_STRDUP(copyOfDatastorePath, src) < 0) - goto cleanup; + copyOfDatastorePath =3D g_strdup(src); =20 /* Expected format: '[] ' where is optiona= l */ if ((tmp =3D STRSKIP(copyOfDatastorePath, "[")) =3D=3D NULL || *tm= p =3D=3D ']' || --=20 2.21.0 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list