From nobody Sat May 18 21:45:37 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=1615550737; cv=none; d=zohomail.com; s=zohoarc; b=EeP3/YeLwEw7/HnUjSYskDGQmyy/PoBD1VzI6K3tbq+v6efyA2mdUTfeqmBWK72Nf7264slBJb17S/LNBlhYw2gxEqv4PqjbQlPdQ8FSgeMAakLoOHEXYcNJ3SjUxgQN4aFWkzBaAeMU1KBccsPsa+MSlDtohW1aFew4l113Msc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1615550737; h=Content-Type:Content-Transfer-Encoding:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To; bh=5Dxd9jL3320Hf0LGI2OTGoGcNCm49MRnexQ8seuB0Ng=; b=DXWt2yfFpr04p/kP9/r6h24qfZlToDfCoSxuke1CNEK0WEbiI6o4OYLMTigg6a4n1RI54/9JSGScDlevwapwm/Oph/qngt5wlSE0RjKN0xCTheFnHZsqHlxuPOcLZl3LWYDQ60XZFGUp+GKZtN/xQUvKXke5SAhlLfmPWAAoCdM= 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) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1615550737471719.5303417406426; Fri, 12 Mar 2021 04:05:37 -0800 (PST) Received: from localhost ([::1]:43168 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lKgXw-0001Op-CL for importer@patchew.org; Fri, 12 Mar 2021 07:05:36 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:42720) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lKgVq-0008Vp-6v for qemu-devel@nongnu.org; Fri, 12 Mar 2021 07:03:27 -0500 Received: from us-smtp-delivery-124.mimecast.com ([216.205.24.124]:43052) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1lKgVg-0003zQ-BZ for qemu-devel@nongnu.org; Fri, 12 Mar 2021 07:03:25 -0500 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-45-mai7fW78M4WXgKBBS4a4RQ-1; Fri, 12 Mar 2021 07:03:12 -0500 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 0CCD4E5761 for ; Fri, 12 Mar 2021 12:03:11 +0000 (UTC) Received: from workimage2020.rezanina.moe.rezanina.moe (unknown [10.40.193.51]) by smtp.corp.redhat.com (Postfix) with ESMTP id 6CF1D60864 for ; Fri, 12 Mar 2021 12:03:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1615550593; h=from:from: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; bh=5Dxd9jL3320Hf0LGI2OTGoGcNCm49MRnexQ8seuB0Ng=; b=AzDd+wWu32biY3Y98GRLhsVt9e2mUh8sHV0xAVcsEqpMLwYEBQsaO4FYLO7O+rx/8qxaqP YU7BT6lN7DUc2zkHB00r0QF/N/ArpRHVwDLtUkLVKIXo8cX1ShxjPbV69qduXCSpSgPQ7w nGZw2NLjI6HFM1RFFG9AaPSRZNn6CkY= X-MC-Unique: mai7fW78M4WXgKBBS4a4RQ-1 From: mrezanin@redhat.com To: qemu-devel@nongnu.org Subject: [PATCH] Add missing initialization for g_autofree variables Date: Fri, 12 Mar 2021 13:03:09 +0100 Message-Id: <20210312120309.138913-1-mrezanin@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=mrezanin@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: quoted-printable 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=216.205.24.124; envelope-from=mrezanin@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.25, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 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" X-ZohoMail-DKIM: pass (identity @redhat.com) Content-Type: text/plain; charset="utf-8" From: Miroslav Rezanina When declaring g_autofree variable without inicialization, compiler will raise "may be used uninitialized in this function" warning due to automatic free handling. This usage can cause gfree using unknown. Such behavior can be dangerous and exploitable. Add inicialization to NULL for these declaration to ensure uninitialized value is not used. Signed-off-by: Miroslav Rezanina --- hw/remote/memory.c | 2 +- hw/remote/proxy.c | 2 +- hw/s390x/s390-pci-vfio.c | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/hw/remote/memory.c b/hw/remote/memory.c index 32085b1e05..bf0047a81b 100644 --- a/hw/remote/memory.c +++ b/hw/remote/memory.c @@ -43,7 +43,7 @@ void remote_sysmem_reconfig(MPQemuMsg *msg, Error **errp) remote_sysmem_reset(); =20 for (region =3D 0; region < msg->num_fds; region++) { - g_autofree char *name; + g_autofree char *name =3D NULL; subregion =3D g_new(MemoryRegion, 1); name =3D g_strdup_printf("remote-mem-%u", suffix++); memory_region_init_ram_from_fd(subregion, NULL, diff --git a/hw/remote/proxy.c b/hw/remote/proxy.c index 4fa4be079d..253c1aa7ec 100644 --- a/hw/remote/proxy.c +++ b/hw/remote/proxy.c @@ -347,7 +347,7 @@ static void probe_pci_info(PCIDevice *dev, Error **errp) PCI_BASE_ADDRESS_SPACE_IO : PCI_BASE_ADDRESS_SPACE_MEMO= RY; =20 if (size) { - g_autofree char *name; + g_autofree char *name =3D NULL; pdev->region[i].dev =3D pdev; pdev->region[i].present =3D true; if (type =3D=3D PCI_BASE_ADDRESS_SPACE_MEMORY) { diff --git a/hw/s390x/s390-pci-vfio.c b/hw/s390x/s390-pci-vfio.c index ead4f222d5..0ee7dc21f2 100644 --- a/hw/s390x/s390-pci-vfio.c +++ b/hw/s390x/s390-pci-vfio.c @@ -29,7 +29,7 @@ */ bool s390_pci_update_dma_avail(int fd, unsigned int *avail) { - g_autofree struct vfio_iommu_type1_info *info; + g_autofree struct vfio_iommu_type1_info *info =3D NULL; uint32_t argsz; =20 assert(avail); @@ -230,7 +230,7 @@ static void s390_pci_read_pfip(S390PCIBusDevice *pbdev, */ void s390_pci_get_clp_info(S390PCIBusDevice *pbdev) { - g_autofree struct vfio_device_info *info; + g_autofree struct vfio_device_info *info =3D NULL; VFIOPCIDevice *vfio_pci; uint32_t argsz; int fd; --=20 2.27.0