From nobody Wed Oct 23 01:37:32 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1729615870; cv=none; d=zohomail.com; s=zohoarc; b=ci5hhOZUvP/4W1tiYxQzZXIJqdfNDhYxF4DT6r0fH8j9Ptlc7tsFc3k/Bc1uWRgRNsk1WDMKnVT+D9HJLEPNpGjUkV/2n2ioE8ptIa1easMAaBAxSZthGGREY5vOeXNaqDQ92c8VXzQA7rI7tRuJdOywdl544rSD/BSlasa5zsM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1729615870; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=HrwhFD+yJwwRK0vf/7Ph/1BLKTn0dL60XtkUPqjTahc=; b=ZEu2q/bMwoqJ5wRBTmPY/9rnlKqZd2uw3JxewOPxotCTF2FjoCMx5OvU1xpB+m3p4/bAJmISZLLbG24PQAW7eVapFKRsB/FOey+1sX+X8NTgBjG120pFwPefY5ukXFNa9DmWURTFFrt+vdNIOshEdTlL2+CnblboFRX16RnrIx0= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1729615870364197.8843008418795; Tue, 22 Oct 2024 09:51:10 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1t3I4d-0000PF-Fw; Tue, 22 Oct 2024 12:49:35 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1t3I4a-0000Hv-1H for qemu-devel@nongnu.org; Tue, 22 Oct 2024 12:49:32 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1t3I4X-00064a-8j for qemu-devel@nongnu.org; Tue, 22 Oct 2024 12:49:31 -0400 Received: from mx-prod-mc-05.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-536-WRJkCBd6NyuXEFf1vxyxyQ-1; Tue, 22 Oct 2024 12:49:16 -0400 Received: from mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.17]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-05.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 0DA2B1956046; Tue, 22 Oct 2024 16:49:15 +0000 (UTC) Received: from merkur.fritz.box (unknown [10.39.194.70]) by mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id C0C3A1956056; Tue, 22 Oct 2024 16:49:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1729615759; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=HrwhFD+yJwwRK0vf/7Ph/1BLKTn0dL60XtkUPqjTahc=; b=E0rn3F7hq+7ZFw4p38azUOIKsyUuKYDyU0WXotKPGSuvdgrjcT0xMVqjTzqlVXNM6on4cv eM2x9bDYZB4Na8R6Yh0h7dtVvPXainfDqPWVxCpqCTJmpoeHXDk0dsW5dwzUm0UAZMS/Wd bx5CL4RipIADEryaIRWZnJormXftyhg= X-MC-Unique: WRJkCBd6NyuXEFf1vxyxyQ-1 From: Kevin Wolf To: qemu-block@nongnu.org Cc: kwolf@redhat.com, qemu-devel@nongnu.org Subject: [PULL 1/9] block/gluster: Use g_autofree for string in qemu_gluster_parse_json() Date: Tue, 22 Oct 2024 18:48:55 +0200 Message-ID: <20241022164903.282174-2-kwolf@redhat.com> In-Reply-To: <20241022164903.282174-1-kwolf@redhat.com> References: <20241022164903.282174-1-kwolf@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 3.0 on 10.30.177.17 Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=170.10.133.124; envelope-from=kwolf@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -25 X-Spam_score: -2.6 X-Spam_bar: -- X-Spam_report: (-2.6 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.519, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1729615871549116600 From: Peter Maydell In the loop in qemu_gluster_parse_json() we do: char *str =3D NULL; for(...) { str =3D g_strdup_printf(...); ... if (various errors) { goto out; } ... g_free(str); str =3D NULL; } return 0; out: various cleanups; g_free(str); ... return -errno; Coverity correctly complains that the assignment "str =3D NULL" at the end of the loop is unnecessary, because we will either go back to the top of the loop and overwrite it, or else we will exit the loop and then exit the function without ever reading str again. The assignment is there as defensive coding to ensure that str is only non-NULL if it's a live allocation, so this is intentional. We can make Coverity happier and simplify the code here by using g_autofree, since we never need 'str' outside the loop. Resolves: Coverity CID 1527385 Signed-off-by: Peter Maydell Reviewed-by: Kevin Wolf Reviewed-by: Philippe Mathieu-Daud=C3=A9 Message-ID: <20241008164708.2966400-2-peter.maydell@linaro.org> Reviewed-by: Richard Henderson Signed-off-by: Kevin Wolf --- block/gluster.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/block/gluster.c b/block/gluster.c index f03d05251e..e9c038042b 100644 --- a/block/gluster.c +++ b/block/gluster.c @@ -514,7 +514,6 @@ static int qemu_gluster_parse_json(BlockdevOptionsGlust= er *gconf, SocketAddressList **tail; QDict *backing_options =3D NULL; Error *local_err =3D NULL; - char *str =3D NULL; const char *ptr; int i, type, num_servers; =20 @@ -547,7 +546,8 @@ static int qemu_gluster_parse_json(BlockdevOptionsGlust= er *gconf, tail =3D &gconf->server; =20 for (i =3D 0; i < num_servers; i++) { - str =3D g_strdup_printf(GLUSTER_OPT_SERVER_PATTERN"%d.", i); + g_autofree char *str =3D g_strdup_printf(GLUSTER_OPT_SERVER_PATTER= N"%d.", + i); qdict_extract_subqdict(options, &backing_options, str); =20 /* create opts info from runtime_type_opts list */ @@ -658,8 +658,6 @@ static int qemu_gluster_parse_json(BlockdevOptionsGlust= er *gconf, =20 qobject_unref(backing_options); backing_options =3D NULL; - g_free(str); - str =3D NULL; } =20 return 0; @@ -668,7 +666,6 @@ out: error_propagate(errp, local_err); qapi_free_SocketAddress(gsconf); qemu_opts_del(opts); - g_free(str); qobject_unref(backing_options); errno =3D EINVAL; return -errno; --=20 2.47.0