From nobody Sun May 19 02:38:12 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=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1615692312; cv=none; d=zohomail.com; s=zohoarc; b=jo71pmaMK36oauUWaUo8Hpg6MR5TIpc7Am07FWjtuDUXzkEJQw4GWShMCZafIuvCc1rTOI+UTl7pEsrHYVNfaM9/UTSn3xPzv1UKsmfcUOBQJ4lDoEu5e8wZbQ2oPwUL41uvZt7gGYPUW4zJTj91CScm8u+Ajw/trwFHByhq0Gs= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1615692312; h=Content-Transfer-Encoding:Cc: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=2rnIQur47bUqEW0oQ7uZPNdfpmlBj++ZpXvn5yiPVOw=; b=Dw2pWI/HUsmb0KH9CJwhyGT/Ai2UpWUTm62SclR4WBd32tRrg1NpAez+3W20eC8izLDYDSui8MqphKcZBa9upG/u03sNA1LMJsjaD5wqDmYfdN29neUgHPQQkc2vFQ27XN/AZDSc21DEUaYBTTTTEGMsSQlQJ8nrOtMejAw+QZg= 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 1615692312706884.1868579852144; Sat, 13 Mar 2021 19:25:12 -0800 (PST) Received: from localhost ([::1]:38844 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lLHNP-0007Nc-Mg for importer@patchew.org; Sat, 13 Mar 2021 22:25:11 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:48896) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lLHMC-0006Sr-Cb for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:23:56 -0500 Received: from mail-wm1-x32e.google.com ([2a00:1450:4864:20::32e]:44773) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lLHMA-0004Oe-KT for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:23:56 -0500 Received: by mail-wm1-x32e.google.com with SMTP id j4-20020a05600c4104b029010c62bc1e20so17443257wmi.3 for ; Sat, 13 Mar 2021 19:23:53 -0800 (PST) Received: from localhost.localdomain ([197.61.164.30]) by smtp.googlemail.com with ESMTPSA id e1sm14922249wrd.44.2021.03.13.19.23.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 13 Mar 2021 19:23:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=2rnIQur47bUqEW0oQ7uZPNdfpmlBj++ZpXvn5yiPVOw=; b=QW3UbyXrUNi0DsP4sIz21ve0giEJ1Y95R38w7rbzSb4FbcYcIRJuG/JBHcrlVPuJoV 4dm1VCu1ZPWYatVwaLhdmUViW6BTAfpxejqPe8g7hoSeFqd3uRfg/4n6ZecwwEA4u8No b8CT9qxOdXxkM7WeoY7iY9eY7xqZg5U6Sgfyrpr7zfIegU5radvC4woYaE8+jBNrtA2H nGi8tL33ZkyMmpcuvFOdMI7NMymBuDJnuXLgwk3RslnvGGp4xPhrz9pUobDBeyRo9lny 3D39lK1dyJ1grYU8v8XdPeDMqmilsyBp+7OBYCkChRQSGciu/onlC69c+SvSKcQ3if7q y8LA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=2rnIQur47bUqEW0oQ7uZPNdfpmlBj++ZpXvn5yiPVOw=; b=AQmQdMm6Lqe1btAdZBqQLH7eFdDC20txGeaDNyq6YZh0zx17Klrkv6gZVR8BtmL5SF teurpM4zPHFQZHHL8D835MbfROxETSwrTcy7cQJRlpK+xyh4ErgACNmwdvhe6T1JOiCp a0ee/9pXEt4Zis7kufL8zwtIkv4vK/+VuzXpUZoj67uUjmyFmNl6kX9WQAAKr2KJnnJQ v5kjWK8H3M9r8XTJS7TrQWQ1S4L3wrfntN4ion0LqJr2AulfVd663bEaym6m3V5A+6Ea RA4c9w+3Vrg0BIeF9W1GsA7ivwqqYZ5PfubeG3J73n8kjIdVizuQdOBTTiV0to3QV3V0 iCTw== X-Gm-Message-State: AOAM532cd3OtpEXD7Mft3doknxXY5/WMB8fCbH4zpmOA1xydmArEcGL4 eTKQGr+AqE95yvZEx7LVjg6iFxE54ig= X-Google-Smtp-Source: ABdhPJxglU6hC5IdRe0q0K1qGVCCpORnCTZOjQxtQPFYT/Juxl8bqIWgZ8whmfTd+aHSP/TXRBze0Q== X-Received: by 2002:a1c:e244:: with SMTP id z65mr20079649wmg.130.1615692232156; Sat, 13 Mar 2021 19:23:52 -0800 (PST) From: Mahmoud Mandour To: qemu-devel@nongnu.org Subject: [PATCH 1/8] bsd-user/elfload.c: Replaced calls to malloc/free with GLib variants Date: Sun, 14 Mar 2021 05:23:17 +0200 Message-Id: <20210314032324.45142-2-ma.mandourr@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210314032324.45142-1-ma.mandourr@gmail.com> References: <20210314032324.45142-1-ma.mandourr@gmail.com> MIME-Version: 1.0 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=2a00:1450:4864:20::32e; envelope-from=ma.mandourr@gmail.com; helo=mail-wm1-x32e.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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: , Cc: Mahmoud Mandour Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @gmail.com) Content-Type: text/plain; charset="utf-8" Replaced the calls to malloc(), realloc(), and free() to their equivalents in GLib's allocation functions in various places. Signed-off-by: Mahmoud Mandour --- bsd-user/elfload.c | 74 +++++++++++++++++++++++----------------------- 1 file changed, 37 insertions(+), 37 deletions(-) diff --git a/bsd-user/elfload.c b/bsd-user/elfload.c index 5f4d824d78..7b0793693b 100644 --- a/bsd-user/elfload.c +++ b/bsd-user/elfload.c @@ -867,8 +867,7 @@ static abi_ulong load_elf_interp(struct elfhdr * interp= _elf_ex, if (sizeof(struct elf_phdr) * interp_elf_ex->e_phnum > TARGET_PAGE= _SIZE) return ~(abi_ulong)0UL; =20 - elf_phdata =3D (struct elf_phdr *) - malloc(sizeof(struct elf_phdr) * interp_elf_ex->e_phnum); + elf_phdata =3D g_try_new(struct elf_phdr, interp_elf_ex->ephnum) =20 if (!elf_phdata) return ~((abi_ulong)0UL); @@ -878,7 +877,7 @@ static abi_ulong load_elf_interp(struct elfhdr * interp= _elf_ex, * we will be doing the wrong thing. */ if (interp_elf_ex->e_phentsize !=3D sizeof(struct elf_phdr)) { - free(elf_phdata); + g_free(elf_phdata); return ~((abi_ulong)0UL); } =20 @@ -891,7 +890,7 @@ static abi_ulong load_elf_interp(struct elfhdr * interp= _elf_ex, if (retval < 0) { perror("load_elf_interp"); exit(-1); - free (elf_phdata); + g_free(elf_phdata); return retval; } #ifdef BSWAP_NEEDED @@ -940,7 +939,7 @@ static abi_ulong load_elf_interp(struct elfhdr * interp= _elf_ex, if (error =3D=3D -1) { /* Real error */ close(interpreter_fd); - free(elf_phdata); + g_free(elf_phdata); return ~((abi_ulong)0UL); } =20 @@ -983,7 +982,7 @@ static abi_ulong load_elf_interp(struct elfhdr * interp= _elf_ex, PROT_READ|PROT_WRITE|PROT_EXEC, MAP_FIXED|MAP_PRIVATE|MAP_ANON, -1, 0); } - free(elf_phdata); + g_free(elf_phdata); =20 *interp_load_addr =3D load_addr; return ((abi_ulong) interp_elf_ex->e_entry) + load_addr; @@ -1064,24 +1063,24 @@ static void load_symbols(struct elfhdr *hdr, int fd) =20 found: /* Now know where the strtab and symtab are. Snarf them. */ - s =3D malloc(sizeof(*s)); - syms =3D malloc(symtab.sh_size); + s =3D g_try_malloc(sizeof(*s)); + syms =3D g_try_malloc(symtab.sh_size); if (!syms) { - free(s); + g_free(s); return; } - s->disas_strtab =3D strings =3D malloc(strtab.sh_size); + s->disas_strtab =3D strings =3D g_malloc(strtab.sh_size); if (!s->disas_strtab) { - free(s); - free(syms); + g_free(s); + g_free(syms); return; } =20 lseek(fd, symtab.sh_offset, SEEK_SET); if (read(fd, syms, symtab.sh_size) !=3D symtab.sh_size) { - free(s); - free(syms); - free(strings); + g_free(s); + g_free(syms); + g_free(strings); return; } =20 @@ -1113,11 +1112,11 @@ static void load_symbols(struct elfhdr *hdr, int fd) that we threw away. Whether or not this has any effect on the memory allocation depends on the malloc implementation and how many symbols we managed to discard. */ - new_syms =3D realloc(syms, nsyms * sizeof(*syms)); + new_syms =3D g_try_realloc(syms, nsyms * sizeof(*syms)); if (new_syms =3D=3D NULL) { - free(s); - free(syms); - free(strings); + g_free(s); + g_free(syms); + g_free(strings); return; } syms =3D new_syms; @@ -1126,9 +1125,9 @@ static void load_symbols(struct elfhdr *hdr, int fd) =20 lseek(fd, strtab.sh_offset, SEEK_SET); if (read(fd, strings, strtab.sh_size) !=3D strtab.sh_size) { - free(s); - free(syms); - free(strings); + g_free(s); + g_free(syms); + g_free(strings); return; } s->disas_num_syms =3D nsyms; @@ -1190,7 +1189,8 @@ int load_elf_binary(struct linux_binprm * bprm, struc= t target_pt_regs * regs, } =20 /* Now read in all of the header information */ - elf_phdata =3D (struct elf_phdr *)malloc(elf_ex.e_phentsize*elf_ex.e_p= hnum); + elf_phdata =3D + (struct elf_phdr *)g_try_malloc(elf_ex.e_phentsizei * elf_ex.e_phn= um); if (elf_phdata =3D=3D NULL) { return -ENOMEM; } @@ -1204,7 +1204,7 @@ int load_elf_binary(struct linux_binprm * bprm, struc= t target_pt_regs * regs, if (retval < 0) { perror("load_elf_binary"); exit(-1); - free (elf_phdata); + g_free(elf_phdata); return -errno; } =20 @@ -1231,8 +1231,8 @@ int load_elf_binary(struct linux_binprm * bprm, struc= t target_pt_regs * regs, if (elf_ppnt->p_type =3D=3D PT_INTERP) { if ( elf_interpreter !=3D NULL ) { - free (elf_phdata); - free(elf_interpreter); + g_free(elf_phdata); + g_free(elf_interpreter); close(bprm->fd); return -EINVAL; } @@ -1242,10 +1242,10 @@ int load_elf_binary(struct linux_binprm * bprm, str= uct target_pt_regs * regs, * is an a.out format binary */ =20 - elf_interpreter =3D (char *)malloc(elf_ppnt->p_filesz); + elf_interpreter =3D (char *)g_try_malloc(elf_ppnt->p_filesz); =20 if (elf_interpreter =3D=3D NULL) { - free (elf_phdata); + g_free(elf_phdata); close(bprm->fd); return -ENOMEM; } @@ -1298,8 +1298,8 @@ int load_elf_binary(struct linux_binprm * bprm, struc= t target_pt_regs * regs, if (retval < 0) { perror("load_elf_binary3"); exit(-1); - free (elf_phdata); - free(elf_interpreter); + g_free(elf_phdata); + g_free(elf_interpreter); close(bprm->fd); return retval; } @@ -1323,8 +1323,8 @@ int load_elf_binary(struct linux_binprm * bprm, struc= t target_pt_regs * regs, } =20 if (!interpreter_type) { - free(elf_interpreter); - free(elf_phdata); + g_free(elf_interpreter); + g_free(elf_phdata); close(bprm->fd); return -ELIBBAD; } @@ -1346,8 +1346,8 @@ int load_elf_binary(struct linux_binprm * bprm, struc= t target_pt_regs * regs, } } if (!bprm->p) { - free(elf_interpreter); - free (elf_phdata); + g_free(elf_interpreter); + g_free(elf_phdata); close(bprm->fd); return -E2BIG; } @@ -1486,17 +1486,17 @@ int load_elf_binary(struct linux_binprm * bprm, str= uct target_pt_regs * regs, reloc_func_desc =3D interp_load_addr; =20 close(interpreter_fd); - free(elf_interpreter); + g_free(elf_interpreter); =20 if (elf_entry =3D=3D ~((abi_ulong)0UL)) { printf("Unable to load interpreter\n"); - free(elf_phdata); + g_free(elf_phdata); exit(-1); return 0; } } =20 - free(elf_phdata); + g_free(elf_phdata); =20 if (qemu_log_enabled()) load_symbols(&elf_ex, bprm->fd); --=20 2.25.1 From nobody Sun May 19 02:38:12 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=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1615692416; cv=none; d=zohomail.com; s=zohoarc; b=bauB2xkbWHLqBpY5euR+f/pmPoabnxIAi9PquIRnT4bQSYRRL3L56jy0YjQRHnHz07OvcZAhrQMuCmOoxTFzmRbCBLQyTB5pquDGqPQ/HTMUp6A6zHq+HY/d5cspOZmZ+tM3xXMd7wMyi3vDQF+Kmw4KHD2Fnwuro3gLYJ1wmkA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1615692416; h=Content-Transfer-Encoding:Cc: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=jsATIuWJVLsP36O5rVx7aqqo47uS9kRVIQPzX/a9z+M=; b=HJOFj6trjWbniqA9hi3UMA/cTf/wWCHM5K7tBAU/Ehemv77rUFvLqDVW/04vcsdjFRAksW+rEhPDieg6Pds2anQI1tYyuleyuMPXcYnRcFL/G4hQeD1v5+6oIg5T287jIsChVBwoqhBRsb24yzMoj6XhbA7rn4714PcK9aoRNxE= 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 161569241685660.4006332275643; Sat, 13 Mar 2021 19:26:56 -0800 (PST) Received: from localhost ([::1]:43142 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lLHP5-0000mz-St for importer@patchew.org; Sat, 13 Mar 2021 22:26:55 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:48920) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lLHMH-0006V2-6C for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:01 -0500 Received: from mail-wm1-x331.google.com ([2a00:1450:4864:20::331]:37658) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lLHMF-0004Qa-8F for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:00 -0500 Received: by mail-wm1-x331.google.com with SMTP id f22-20020a7bc8d60000b029010c024a1407so18142586wml.2 for ; Sat, 13 Mar 2021 19:23:58 -0800 (PST) Received: from localhost.localdomain ([197.61.164.30]) by smtp.googlemail.com with ESMTPSA id e1sm14922249wrd.44.2021.03.13.19.23.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 13 Mar 2021 19:23:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=jsATIuWJVLsP36O5rVx7aqqo47uS9kRVIQPzX/a9z+M=; b=h47gePfWpVLTlLPeqZkKtPpprzcttqNm4iqD709FU3AmUNx5hZF7Ma5GzOUvsuXdE0 x4mwtwFfcDugY+XLj22iWPLYPll3qqoyQw46nZzht2aqWaYbUoelLvO0hvlNB6u9LXh7 74GU9z+9h4A1Vf/fYlj0nb6hIFvaqSN45IBa8eJzXtitgwW2GkRMCW5lUiKPt7ffFeRH LcXqbDX3TQyRXAjIe3841bnbXBV2SUwfq0S44Ul+Rq6cl/QlDGNRkwXMN0MuCbIW77w/ 6thnQtdpZ2tR3XigZTxAOhpUloTSj69BmJIG03aJbVHkXSLk2nOEwzaCvBRUpRPfCIkq nmQg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=jsATIuWJVLsP36O5rVx7aqqo47uS9kRVIQPzX/a9z+M=; b=DXILvivAyzEG6VRgb0B49H0FDgX0sfMc4ZpqxaS9OfEUHbLUq8IJmhIzWF1CmGq7ih 1kYVaDYd2xCDmFQuMa7rmts2Ll4a0rTEX2w6LzkSHz/rAXcoeokwROtMlyQ/SV3ImCdo xGpuv2793aoJg174aNtJtknZ0ArY5JxctyYB5u0ck1qBeopjvFYQA2def21RsLw6uGmu uT8CqCLZewoIRMlOOFKWv2RoQv32I6zTiab1HwkAfsW6bVDfzMqHkBYxWoZVXr1WM13R cKDabf4nlUFLPX9y/k7onndKQGKKByKsC+knzJjpSdBa92rqwl3x0cJcomSiN1o0ZBOi FTWw== X-Gm-Message-State: AOAM532ymiv6+L7UOfo1o2mGqHntN5zofqPXuLQoULbF2wpeZZ2iDB+S 6SJ+l7AGAlCU47HU9+gx4fFJRjgAbJg= X-Google-Smtp-Source: ABdhPJzqIDzR+OfYBW1hXWgNXuwHYHW1Ydm/6WN4sD2new6S/6A72iUdabFtAqRUo+oDaMfAx5rTfQ== X-Received: by 2002:a1c:3954:: with SMTP id g81mr17364503wma.170.1615692236853; Sat, 13 Mar 2021 19:23:56 -0800 (PST) From: Mahmoud Mandour To: qemu-devel@nongnu.org Subject: [PATCH 2/8] hw/audio/fmopl.c: Fixing some style errors Date: Sun, 14 Mar 2021 05:23:18 +0200 Message-Id: <20210314032324.45142-3-ma.mandourr@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210314032324.45142-1-ma.mandourr@gmail.com> References: <20210314032324.45142-1-ma.mandourr@gmail.com> MIME-Version: 1.0 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=2a00:1450:4864:20::331; envelope-from=ma.mandourr@gmail.com; helo=mail-wm1-x331.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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: , Cc: Mahmoud Mandour , Gerd Hoffmann Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @gmail.com) Content-Type: text/plain; charset="utf-8" Fixed style errors on the relevant lines in which I will introduce changes. Signed-off-by: Mahmoud Mandour --- hw/audio/fmopl.c | 58 ++++++++++++++++++++++++++---------------------- 1 file changed, 32 insertions(+), 26 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index 8a71a569fa..51b773695a 100644 --- a/hw/audio/fmopl.c +++ b/hw/audio/fmopl.c @@ -607,26 +607,32 @@ static int OPLOpenTable( void ) double pom; =20 /* allocate dynamic tables */ - if( (TL_TABLE =3D malloc(TL_MAX*2*sizeof(int32_t))) =3D=3D NULL) - return 0; - if( (SIN_TABLE =3D malloc(SIN_ENT*4 *sizeof(int32_t *))) =3D=3D NULL) - { - free(TL_TABLE); - return 0; - } - if( (AMS_TABLE =3D malloc(AMS_ENT*2 *sizeof(int32_t))) =3D=3D NULL) - { - free(TL_TABLE); - free(SIN_TABLE); - return 0; - } - if( (VIB_TABLE =3D malloc(VIB_ENT*2 *sizeof(int32_t))) =3D=3D NULL) - { - free(TL_TABLE); - free(SIN_TABLE); - free(AMS_TABLE); - return 0; - } + TL_TABLE =3D malloc(TL_MAX * 2 * sizeof(int32_t)); + if (TL_TABLE =3D=3D NULL) { + return 0; + } + + SIN_TABLE =3D malloc(SIN_ENT * 4 * sizeof(int32_t *)); + if (SIN_TABLE =3D=3D NULL) { + free(TL_TABLE); + return 0; + } + + AMS_TABLE =3D malloc(AMS_ENT * 2 * sizeof(int32_t)); + if (AMS_TABLE =3D=3D NULL) { + free(TL_TABLE); + free(SIN_TABLE); + return 0; + } + + VIB_TABLE =3D malloc(VIB_ENT * 2 * sizeof(int32_t)); + if (VIB_TABLE =3D=3D NULL) { + free(TL_TABLE); + free(SIN_TABLE); + free(AMS_TABLE); + return 0; + } + ENV_CURVE =3D g_new(int32_t, 2 * EG_ENT + 1); /* make total level table */ for (t =3D 0;t < EG_ENT-1 ;t++){ @@ -696,10 +702,10 @@ static int OPLOpenTable( void ) static void OPLCloseTable( void ) { g_free(ENV_CURVE); - free(TL_TABLE); - free(SIN_TABLE); - free(AMS_TABLE); - free(VIB_TABLE); + free(TL_TABLE); + free(SIN_TABLE); + free(AMS_TABLE); + free(VIB_TABLE); } =20 /* CSM Key Control */ @@ -1082,7 +1088,7 @@ FM_OPL *OPLCreate(int clock, int rate) state_size =3D sizeof(FM_OPL); state_size +=3D sizeof(OPL_CH)*max_ch; /* allocate memory block */ - ptr =3D malloc(state_size); + ptr =3D malloc(state_size); if(ptr=3D=3DNULL) return NULL; /* clear */ memset(ptr,0,state_size); @@ -1128,7 +1134,7 @@ void OPLDestroy(FM_OPL *OPL) } #endif OPL_UnLockTable(); - free(OPL); + free(OPL); } =20 /* ---------- Option handlers ---------- */ --=20 2.25.1 From nobody Sun May 19 02:38:12 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=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1615692379; cv=none; d=zohomail.com; s=zohoarc; b=jOOpkygEMFppi21aHq7nlmKzSAOgEQP5YIRg1Iooi4hjgUTmKypeNxNm4eNsBFaUiOafJLqZ5hKPBZ7MCuTSZq6A8Q0dtUrgo/LGmLKfrbPljK9XXX7uSCYAq5a2rtLUHwPdqrxOOkkpz8iyGWwSZ4KTuvWKnPjgRe4fGEZMeK4= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1615692379; h=Content-Transfer-Encoding:Cc: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=BajUyfK396v7XdGJs5VK12t0TqFac+fXAWp2W7CZvi4=; b=ly64V1zE8X9/v3F+vWvOyC14CkNdTe/vJ78/fWJloGkb6Np7pv16EGbO2Vy4P/+DDCECSeSllFyc+a6sUy7Df8QcDRkvtPkz9a9F7Suu2Iekb2ogu28PzzGBiAAeq3MtweZRFMnaQkDj3gTUayZ6FXIb3Tox9VJiblznQTICLqA= 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 1615692379923925.3235187763694; Sat, 13 Mar 2021 19:26:19 -0800 (PST) Received: from localhost ([::1]:41816 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lLHOU-0000EL-Sa for importer@patchew.org; Sat, 13 Mar 2021 22:26:18 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:48922) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lLHMH-0006Vc-H6 for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:02 -0500 Received: from mail-wr1-x42c.google.com ([2a00:1450:4864:20::42c]:37220) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lLHMF-0004RU-C3 for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:01 -0500 Received: by mail-wr1-x42c.google.com with SMTP id v15so6907456wrx.4 for ; Sat, 13 Mar 2021 19:23:58 -0800 (PST) Received: from localhost.localdomain ([197.61.164.30]) by smtp.googlemail.com with ESMTPSA id e1sm14922249wrd.44.2021.03.13.19.23.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 13 Mar 2021 19:23:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=BajUyfK396v7XdGJs5VK12t0TqFac+fXAWp2W7CZvi4=; b=NYw4mxvsGbWYnB32qoBuO35zU5yhFGs3LLivuPZslQd9Ddqw9I36e/l8MFygj0xOcc yZU1ppsm1zuQXYzieL6JuI4UzS7HyBVSbtFgWXhxGIR21XSLcMsyBleWtooI5nsarfEv BHSUd03doGa9h/X6DTnFyN4hnFewx8V871ycyPZRZp1D/F9BNV5qzWHjWz+MnMIeKOlO fqG1wB9pvEcsfuP3hn34hySwb4JMLnymABvwX/hX4aaZ47K8iCol+UM4WFH0ZLBfXPyV x5Ey7E8qf91NekepLqPgRcBkG7j/viiKOwhQVbzydIXR3MUfnorpaSxoqgKSVur7h1e9 1yaA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=BajUyfK396v7XdGJs5VK12t0TqFac+fXAWp2W7CZvi4=; b=Twu7ytU/oZWS29eXmkiDbddhDk1Dt4QyAs0e5hamRaknfRvIqoWrt7h6erK6QhJSRG y3Ly5Uxhk6HdOVXg40QOhc/HOIapyJu1sjq/5yZO0MvBhA0y3JfHEX+6540gJearPLty LhcsywkzbdHeXyYMoKgwq8zGVn0gEJJoAZMpM/bHSxOyW0x+Lw2gF/6RvEPyReWhxUKg 4tIN9a9yK0M+spsCIZNksXnK8rXaOMb/vimQhm3gdEdtl9NH3ZNASufHH86Y84jiJWm7 Ilx6cjNlOkfD55iPwhfSHUA/v/RfnKY+/+kuyliHNHAONi6yjDNRA0y78s5gZENXNgpV fkgw== X-Gm-Message-State: AOAM532eMna3y1Cqtrr6LAYlEvcUqOrY0AJUAWKC+4DR9VXktUKBEbSg NSUF330qIpK9XRmpyQpL1eohHfFKOSc= X-Google-Smtp-Source: ABdhPJyQiBG3C3mbSJVJ6DguNxzXLhDF91H+STCd7xv8rLaUhXvLS4A96xNipLEbjQ59LqNoemRiBw== X-Received: by 2002:a5d:570b:: with SMTP id a11mr21299224wrv.281.1615692237926; Sat, 13 Mar 2021 19:23:57 -0800 (PST) From: Mahmoud Mandour To: qemu-devel@nongnu.org Subject: [PATCH 3/8] hw/audio/fmopl.c: Replaced calls to malloc with GLib's variants Date: Sun, 14 Mar 2021 05:23:19 +0200 Message-Id: <20210314032324.45142-4-ma.mandourr@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210314032324.45142-1-ma.mandourr@gmail.com> References: <20210314032324.45142-1-ma.mandourr@gmail.com> MIME-Version: 1.0 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=2a00:1450:4864:20::42c; envelope-from=ma.mandourr@gmail.com; helo=mail-wr1-x42c.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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: , Cc: Mahmoud Mandour , Gerd Hoffmann Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @gmail.com) Content-Type: text/plain; charset="utf-8" Replaced calls to malloc(), and free() to their equivalent allocation functions from GLib. Also added checking for null after ENV_CURVE allocation following the same pattern of checking on preceeding table allocations. Signed-off-by: Mahmoud Mandour --- hw/audio/fmopl.c | 42 +++++++++++++++++++++++++----------------- 1 file changed, 25 insertions(+), 17 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index 51b773695a..795c7a23dc 100644 --- a/hw/audio/fmopl.c +++ b/hw/audio/fmopl.c @@ -607,33 +607,41 @@ static int OPLOpenTable( void ) double pom; =20 /* allocate dynamic tables */ - TL_TABLE =3D malloc(TL_MAX * 2 * sizeof(int32_t)); + TL_TABLE =3D g_try_new(int32_t, TL_MAX * 2); if (TL_TABLE =3D=3D NULL) { return 0; } =20 - SIN_TABLE =3D malloc(SIN_ENT * 4 * sizeof(int32_t *)); + SIN_TABLE =3D g_try_new(int32_t *, SIN_ENT * 4); if (SIN_TABLE =3D=3D NULL) { - free(TL_TABLE); + g_free(TL_TABLE); return 0; } =20 - AMS_TABLE =3D malloc(AMS_ENT * 2 * sizeof(int32_t)); + AMS_TABLE =3D g_try_new(int32_t, AMS_ENT * 2); if (AMS_TABLE =3D=3D NULL) { - free(TL_TABLE); - free(SIN_TABLE); + g_free(TL_TABLE); + g_free(SIN_TABLE); return 0; } =20 - VIB_TABLE =3D malloc(VIB_ENT * 2 * sizeof(int32_t)); + VIB_TABLE =3D g_try_new(int32_t, VIB_ENT * 2); if (VIB_TABLE =3D=3D NULL) { - free(TL_TABLE); - free(SIN_TABLE); - free(AMS_TABLE); + g_free(TL_TABLE); + g_free(SIN_TABLE); + g_free(AMS_TABLE); + return 0; + } + + ENV_CURVE =3D g_try_new(int32_t, 2 * EG_ENT + 1); + if (ENV_CURVE =3D=3D NULL) { + g_free(TL_TABLE); + g_free(SIN_TABLE); + g_free(AMS_TABLE); + g_free(VIB_TABLE); return 0; } =20 - ENV_CURVE =3D g_new(int32_t, 2 * EG_ENT + 1); /* make total level table */ for (t =3D 0;t < EG_ENT-1 ;t++){ rate =3D ((1< voltage */ @@ -702,10 +710,10 @@ static int OPLOpenTable( void ) static void OPLCloseTable( void ) { g_free(ENV_CURVE); - free(TL_TABLE); - free(SIN_TABLE); - free(AMS_TABLE); - free(VIB_TABLE); + g_free(TL_TABLE); + g_free(SIN_TABLE); + g_free(AMS_TABLE); + g_free(VIB_TABLE); } =20 /* CSM Key Control */ @@ -1088,7 +1096,7 @@ FM_OPL *OPLCreate(int clock, int rate) state_size =3D sizeof(FM_OPL); state_size +=3D sizeof(OPL_CH)*max_ch; /* allocate memory block */ - ptr =3D malloc(state_size); + ptr =3D g_try_malloc(state_size); if(ptr=3D=3DNULL) return NULL; /* clear */ memset(ptr,0,state_size); @@ -1134,7 +1142,7 @@ void OPLDestroy(FM_OPL *OPL) } #endif OPL_UnLockTable(); - free(OPL); + g_free(OPL); } =20 /* ---------- Option handlers ---------- */ --=20 2.25.1 From nobody Sun May 19 02:38:12 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; 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=fail(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1615692493; cv=none; d=zohomail.com; s=zohoarc; b=SAnT3whTHL+Ju+dgxu/bZbH2yVe+Y23tfMfpQhQxdMeKz8PwAx1k0YWHP0880nwPxNRtrqoP/TwL7eV150NPbaaqopA4gpHRlOalZg2XPsXPclmKu6EEg+Nv3oxpBl24fCzS1Nx2ZYXwRBeqSZCqt7dRMrkVjfu6QfzdRZMIu1E= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1615692493; h=Content-Transfer-Encoding:Cc: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=03j/gt08iEmB/YpVMydgNMGaK1/oqVMzCnaDLB+XL7M=; b=gPD/bvHsDT4pgOR4CK/sewZfl3Dbh8QqYWe3x8Qn5xC7Fx+jt0hKpq6ZsfYZORoXL1VUxujzPymE2Pevgbd4Ht+NXuC9Yz/B+qotvghrbKipquxZz5XBoNp9eOwDfx1itu5w3BFJAyIxaNHT6xF9AlId5nqEMi0Kx6xg097fHpo= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; 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=fail 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 1615692493967279.3422703806025; Sat, 13 Mar 2021 19:28:13 -0800 (PST) Received: from localhost ([::1]:47414 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lLHQK-0002Xq-Vv for importer@patchew.org; Sat, 13 Mar 2021 22:28:13 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:48944) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lLHMJ-0006Vt-5W for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:04 -0500 Received: from mail-wr1-x434.google.com ([2a00:1450:4864:20::434]:34473) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lLHMG-0004RY-Tt for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:02 -0500 Received: by mail-wr1-x434.google.com with SMTP id j7so3901968wrd.1 for ; Sat, 13 Mar 2021 19:24:00 -0800 (PST) Received: from localhost.localdomain ([197.61.164.30]) by smtp.googlemail.com with ESMTPSA id e1sm14922249wrd.44.2021.03.13.19.23.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 13 Mar 2021 19:23:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=03j/gt08iEmB/YpVMydgNMGaK1/oqVMzCnaDLB+XL7M=; b=Ii/byVoiSpzMkxd5JVFe+mKtH7ToN1dOHdaNLTKmYLx9njMUCC2Z7dinJf/eUcHotO mAWmLX4kSV5zmWeWjyZTzd199D2xnGybLV1qZK64u47Xa6VZu0B6X9iG2eoe4hKdyICb qk4pZLffEfmWMCk1LfoCgp/ml3ssWSmriTrsFGKFv0ZH5lALxNtLkmVR21MlU055hi5b dWXXj340OU/lEv/KBjzzgklVHdRkUHpwHkt7jzR6U7guWU15v3YdmTDOjqKGnSM4tBFs uvY9ZjPPDKfB81tg+e9BpXoXidnMulHzfvBF5SYWTHkGyQjHpuBvniU6I10M3ksTV3Ej 4i+w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=03j/gt08iEmB/YpVMydgNMGaK1/oqVMzCnaDLB+XL7M=; b=moon2JVmukL192HcFy4lwvkbRFgo7VnA2hZe7cS4YT0WlVh00fGWpoKiRAkkwkxnUT o+1mN9Z1UYy+O0WPZqEQknSikvD3smwKKKp9kCXKiNSFhqWgOf58qjpwORX9E7KGP+Fu XbMoFTwXi6KVc2ePT6f+ir2JXT2I0LCKVx2UPbGOjsDlTcTGtqj8Y50jR9ZLeiHf8/Qn Mb8UQbeWQ8Qk/LBpkX2zJBkr2gaqfdW6QhDIKr5txUSoswoRrwXL4jVj+mD1FTBDPWpz TiEHx9L4jRECJRuHoPL+4vdTod2iVNVhrfl+psJppaEhSIknO+WExqaD0Lr1JBaHnd9U SBKQ== X-Gm-Message-State: AOAM53288aV25LNQqUMxiwy6SVZnJRth2feGVpqibVOpoX4Wcdi94pdU y5fCxvGWqz1s+SUPPm9r9GdCgVt76Ms= X-Google-Smtp-Source: ABdhPJxCIdAjkcprtUcVribnivMJvBU3iHeU1ToyMfX/M18v1lHWN9FWNazHtLWSfV2adr1VBTzppw== X-Received: by 2002:a5d:6b50:: with SMTP id x16mr5420011wrw.379.1615692239020; Sat, 13 Mar 2021 19:23:59 -0800 (PST) From: Mahmoud Mandour To: qemu-devel@nongnu.org Subject: [PATCH 4/8] target/xtensa: Replaced malloc/free with GLib's variants Date: Sun, 14 Mar 2021 05:23:20 +0200 Message-Id: <20210314032324.45142-5-ma.mandourr@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210314032324.45142-1-ma.mandourr@gmail.com> References: <20210314032324.45142-1-ma.mandourr@gmail.com> MIME-Version: 1.0 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=2a00:1450:4864:20::434; envelope-from=ma.mandourr@gmail.com; helo=mail-wr1-x434.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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: , Cc: Max Filippov , Mahmoud Mandour Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" Replaced the calls to malloc() and their respective calls to free() with GLib's allocation and deallocation functions. Removed null checking before calling g_free() because it's not necessary and generates style errors. Signed-off-by: Mahmoud Mandour Acked-by: Max Filippov --- target/xtensa/xtensa-isa.c | 53 +++++++++++++++----------------------- 1 file changed, 21 insertions(+), 32 deletions(-) diff --git a/target/xtensa/xtensa-isa.c b/target/xtensa/xtensa-isa.c index 630b4f9da1..5afdba77aa 100644 --- a/target/xtensa/xtensa-isa.c +++ b/target/xtensa/xtensa-isa.c @@ -79,7 +79,7 @@ int xtensa_insnbuf_size(xtensa_isa isa) xtensa_insnbuf xtensa_insnbuf_alloc(xtensa_isa isa) { xtensa_insnbuf result =3D (xtensa_insnbuf) - malloc(xtensa_insnbuf_size(isa) * sizeof(xtensa_insnbuf_word)); + g_try_malloc(xtensa_insnbuf_size(isa) * sizeof(xtensa_insnbuf_word= )); =20 CHECK_ALLOC(result, 0); return result; @@ -89,7 +89,7 @@ xtensa_insnbuf xtensa_insnbuf_alloc(xtensa_isa isa) void xtensa_insnbuf_free(xtensa_isa isa __attribute__ ((unused)), xtensa_insnbuf buf) { - free(buf); + g_free(buf); } =20 =20 @@ -237,7 +237,7 @@ xtensa_isa xtensa_isa_init(void *xtensa_modules, xtensa= _isa_status *errno_p, =20 /* Set up the opcode name lookup table. */ isa->opname_lookup_table =3D - malloc(isa->num_opcodes * sizeof(xtensa_lookup_entry)); + g_try_new(xtensa_lookup_entry, isa->num_opcodes); CHECK_ALLOC_FOR_INIT(isa->opname_lookup_table, NULL, errno_p, error_ms= g_p); for (n =3D 0; n < isa->num_opcodes; n++) { isa->opname_lookup_table[n].key =3D isa->opcodes[n].name; @@ -248,7 +248,7 @@ xtensa_isa xtensa_isa_init(void *xtensa_modules, xtensa= _isa_status *errno_p, =20 /* Set up the state name lookup table. */ isa->state_lookup_table =3D - malloc(isa->num_states * sizeof(xtensa_lookup_entry)); + g_try_new(xtensa_lookup_entry, isa->num_states); CHECK_ALLOC_FOR_INIT(isa->state_lookup_table, NULL, errno_p, error_msg= _p); for (n =3D 0; n < isa->num_states; n++) { isa->state_lookup_table[n].key =3D isa->states[n].name; @@ -259,7 +259,7 @@ xtensa_isa xtensa_isa_init(void *xtensa_modules, xtensa= _isa_status *errno_p, =20 /* Set up the sysreg name lookup table. */ isa->sysreg_lookup_table =3D - malloc(isa->num_sysregs * sizeof(xtensa_lookup_entry)); + g_try_new(xtensa_lookup_entry, isa->num_sysregs); CHECK_ALLOC_FOR_INIT(isa->sysreg_lookup_table, NULL, errno_p, error_ms= g_p); for (n =3D 0; n < isa->num_sysregs; n++) { isa->sysreg_lookup_table[n].key =3D isa->sysregs[n].name; @@ -271,7 +271,7 @@ xtensa_isa xtensa_isa_init(void *xtensa_modules, xtensa= _isa_status *errno_p, /* Set up the user & system sysreg number tables. */ for (is_user =3D 0; is_user < 2; is_user++) { isa->sysreg_table[is_user] =3D - malloc((isa->max_sysreg_num[is_user] + 1) * sizeof(xtensa_sysr= eg)); + g_try_new(xtensa_sysreg, isa->max_sysreg_num[is_user] + 1); CHECK_ALLOC_FOR_INIT(isa->sysreg_table[is_user], NULL, errno_p, error_msg_p); =20 @@ -290,7 +290,7 @@ xtensa_isa xtensa_isa_init(void *xtensa_modules, xtensa= _isa_status *errno_p, =20 /* Set up the interface lookup table. */ isa->interface_lookup_table =3D - malloc(isa->num_interfaces * sizeof(xtensa_lookup_entry)); + g_try_new(xtensa_lookup_entry, isa->num_interfaces); CHECK_ALLOC_FOR_INIT(isa->interface_lookup_table, NULL, errno_p, error_msg_p); for (n =3D 0; n < isa->num_interfaces; n++) { @@ -302,7 +302,7 @@ xtensa_isa xtensa_isa_init(void *xtensa_modules, xtensa= _isa_status *errno_p, =20 /* Set up the funcUnit lookup table. */ isa->funcUnit_lookup_table =3D - malloc(isa->num_funcUnits * sizeof(xtensa_lookup_entry)); + g_try_new(xtensa_lookup_entry, isa->num_funcUnits); CHECK_ALLOC_FOR_INIT(isa->funcUnit_lookup_table, NULL, errno_p, error_msg_p); for (n =3D 0; n < isa->num_funcUnits; n++) { @@ -332,36 +332,25 @@ void xtensa_isa_free(xtensa_isa isa) * structure to its initial state. */ =20 - if (intisa->opname_lookup_table) { - free(intisa->opname_lookup_table); - intisa->opname_lookup_table =3D 0; - } + g_free(intisa->opname_lookup_table); + intisa->opname_lookup_table =3D 0; =20 - if (intisa->state_lookup_table) { - free(intisa->state_lookup_table); - intisa->state_lookup_table =3D 0; - } + g_free(intisa->state_lookup_table); + intisa->state_lookup_table =3D 0; + + g_free(intisa->sysreg_lookup_table); + intisa->sysreg_lookup_table =3D 0; =20 - if (intisa->sysreg_lookup_table) { - free(intisa->sysreg_lookup_table); - intisa->sysreg_lookup_table =3D 0; - } for (n =3D 0; n < 2; n++) { - if (intisa->sysreg_table[n]) { - free(intisa->sysreg_table[n]); - intisa->sysreg_table[n] =3D 0; - } + g_free(intisa->sysreg_table[n]); + intisa->sysreg_table[n] =3D 0; } =20 - if (intisa->interface_lookup_table) { - free(intisa->interface_lookup_table); - intisa->interface_lookup_table =3D 0; - } + g_free(intisa->interface_lookup_table); + intisa->interface_lookup_table =3D 0; =20 - if (intisa->funcUnit_lookup_table) { - free(intisa->funcUnit_lookup_table); - intisa->funcUnit_lookup_table =3D 0; - } + g_free(intisa->funcUnit_lookup_table); + intisa->funcUnit_lookup_table =3D 0; } =20 =20 --=20 2.25.1 From nobody Sun May 19 02:38:12 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=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1615692459; cv=none; d=zohomail.com; s=zohoarc; b=OmL63/V02LtnxG5P3WeZvU//3CR9HnnIXCL+Zaccul0PJMdqd4tpOk+3eC3T884hcy4dN0J//s4p6v6CxGyusL6hFemaJnsqwu3tXvbIPkw1dd5sQpyEQbdbRqAXfRuD7GBB92eqJNEcvBYuUTgXMLa/Yb6oyi1ZaDJxN62POJ4= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1615692459; h=Content-Transfer-Encoding:Cc: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=ULb0qQJUebEUkzQmRP5LxPxlhhbE/qPxwsfl/mciK6w=; b=ACnSuYUFSt7d1e63es5+NF36EyI8wvyeJ8sYpZ/TJWaSEQpUxL/YRZbqFBzuAI1WhsUii3ABEBgTD1KJAivEWfT22ZDdhjCJRkVNIfb72OeO0xpD8PwLOd2jsKOTIaeC9QG3o+y7lPg1xthkhuv2rDccqKs1U2xVmz4mZLBGW5w= 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 1615692459783638.4326289733225; Sat, 13 Mar 2021 19:27:39 -0800 (PST) Received: from localhost ([::1]:46016 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lLHPm-0001y4-Pb for importer@patchew.org; Sat, 13 Mar 2021 22:27:38 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:48956) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lLHMK-0006XF-8K for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:04 -0500 Received: from mail-wm1-x335.google.com ([2a00:1450:4864:20::335]:52897) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lLHMH-0004SU-Hq for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:04 -0500 Received: by mail-wm1-x335.google.com with SMTP id d191so6054668wmd.2 for ; Sat, 13 Mar 2021 19:24:01 -0800 (PST) Received: from localhost.localdomain ([197.61.164.30]) by smtp.googlemail.com with ESMTPSA id e1sm14922249wrd.44.2021.03.13.19.23.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 13 Mar 2021 19:23:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=ULb0qQJUebEUkzQmRP5LxPxlhhbE/qPxwsfl/mciK6w=; b=IZpnYbQCRmVXF5uhIjhIZcymB+m/+BcS4dI9REG29zDjOwpKgx+T+rEFslPGY0i/9j N8RrqqWTUQM8/1Ys5VcJPjmHDpWh1CW0pkcT5j+sMtqmJQIQkk+VFfl7EAvapUMy0AeA 1QcauFJGg3PG/k3YSG7g88qUV+2dDmGTeTRpkpSKVuC9dniD34m0lsQd4eUPKfud8H8V HlWc8ouwQ27JNKfB8mveApEGJSDNVwaQ79i7J7BnR0tBhHDjMcHPdpPYGkH6IhLiSdcg YD2QmiS0ugYZUh7x5m8QKoYvGNRHjPn5/AoPpg/kyldYADkPu5Ayigsbl4atFTyOrB9x 1Tcw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=ULb0qQJUebEUkzQmRP5LxPxlhhbE/qPxwsfl/mciK6w=; b=P+YL4EeFZ8eQa0CAk9Ozd/m/FHUWoD5DWYKHI2HbENsqdKmYc+4YzZ6ZY3m9gM3dkv 0YHIOZLM7YZnkCp8LogscWzRerLYyYPalwUb0SmW7hDFPoQkANxhf1a4i1eqZ7trFmm4 +GXI5EJTzdtH/6i66+dlkuyAbmwCGS3hmb9WdeK2ZV+kVKBMG9Bda8C0VwYzJeU2Or78 D44Xa+JT7d3aEcUg5/2rhBAvTRdme9AO1E1O/IhiBIMhXp4wIWyora/AIE8BDvx3nQ0d P9nTFsZLlDDAUShXJOgFQRE3kQ3SLeFgsiQ9cRiiEsy07cc4n9O16eAEi+sLIeVS9ixz DOrQ== X-Gm-Message-State: AOAM533HclR3HwHK5r5ThWeVFtW/DHX82sv4xVeww18ZfApPn1g1fx2H Qkdw4WOp9J6ub1bvPe54CRjSJFBW7pg= X-Google-Smtp-Source: ABdhPJzkO11Wgy1Gb1TPHHEW8R2fRpofdOGJ1FFdHVLl9IHD1TVoNdkRGTN5Jj88I9KH6hdW1Uw2Ug== X-Received: by 2002:a1c:bc56:: with SMTP id m83mr20210266wmf.174.1615692240129; Sat, 13 Mar 2021 19:24:00 -0800 (PST) From: Mahmoud Mandour To: qemu-devel@nongnu.org Subject: [PATCH 5/8] util/compatfd.c: Replaced a malloc with GLib's variant Date: Sun, 14 Mar 2021 05:23:21 +0200 Message-Id: <20210314032324.45142-6-ma.mandourr@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210314032324.45142-1-ma.mandourr@gmail.com> References: <20210314032324.45142-1-ma.mandourr@gmail.com> MIME-Version: 1.0 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=2a00:1450:4864:20::335; envelope-from=ma.mandourr@gmail.com; helo=mail-wm1-x335.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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: , Cc: Mahmoud Mandour Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @gmail.com) Content-Type: text/plain; charset="utf-8" Replaced a malloc() call and its respective free() call with GLib's g_try_malloc() and g_free(). Also, did slight styling changes that were producing style errors when using the checkpatch.pl script against the file. Signed-off-by: Mahmoud Mandour --- util/compatfd.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/util/compatfd.c b/util/compatfd.c index ee47dd8089..834ddd0573 100644 --- a/util/compatfd.c +++ b/util/compatfd.c @@ -20,8 +20,7 @@ #include #endif =20 -struct sigfd_compat_info -{ +struct sigfd_compat_info { sigset_t mask; int fd; }; @@ -53,8 +52,9 @@ static void *sigwait_compat(void *opaque) =20 len =3D write(info->fd, (char *)&buffer + offset, sizeof(buffer) - offset); - if (len =3D=3D -1 && errno =3D=3D EINTR) + if (len =3D=3D -1 && errno =3D=3D EINTR) { continue; + } =20 if (len <=3D 0) { return NULL; @@ -72,14 +72,14 @@ static int qemu_signalfd_compat(const sigset_t *mask) QemuThread thread; int fds[2]; =20 - info =3D malloc(sizeof(*info)); + info =3D g_try_malloc(sizeof(*info)); if (info =3D=3D NULL) { errno =3D ENOMEM; return -1; } =20 if (pipe(fds) =3D=3D -1) { - free(info); + g_free(info); return -1; } =20 --=20 2.25.1 From nobody Sun May 19 02:38:12 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=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1615692557; cv=none; d=zohomail.com; s=zohoarc; b=TAJzRS1M10AQVajgkrnZx8Ir+z6O2upVszcJBWNJpdIp/P+m1JxSy0fqHjLRKQ5xVrgSUyzREyDi232alr8ml2LmC+hvPfqawkLFYbwEgnKI1q4+SE3QE4PilMZNqswc2CLxOSliOvs1yCKuqVo1zhVm3trVXN2lbj5uDn+sUvU= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1615692557; h=Content-Transfer-Encoding:Cc: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=FBWETjBx7yR8JhCne1xrs11sERKEjTujz/IcxswUR6c=; b=DP+tuzoORCvcZInPsrr7z1h62DrGjO4qG1TyUZ9lKj8NgI+ekIMg6piXjFc+jYmHVDw/L4+wp57wFs3exFOFCiFvIzXG7PxLlgWtrarXalrf35PKmLgHfb4rLtoTbH5OEt2vU8V8N9LHT9XmxJDY8ro8oU9oJTNVJxGiBzQySKY= 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 1615692557623828.284523331955; Sat, 13 Mar 2021 19:29:17 -0800 (PST) Received: from localhost ([::1]:51686 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lLHRM-0004Ha-MD for importer@patchew.org; Sat, 13 Mar 2021 22:29:16 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:48966) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lLHMK-0006XP-Vt for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:06 -0500 Received: from mail-wm1-x336.google.com ([2a00:1450:4864:20::336]:35664) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lLHMI-0004Sx-Tm for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:04 -0500 Received: by mail-wm1-x336.google.com with SMTP id c76-20020a1c9a4f0000b029010c94499aedso18133291wme.0 for ; Sat, 13 Mar 2021 19:24:02 -0800 (PST) Received: from localhost.localdomain ([197.61.164.30]) by smtp.googlemail.com with ESMTPSA id e1sm14922249wrd.44.2021.03.13.19.24.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 13 Mar 2021 19:24:00 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=FBWETjBx7yR8JhCne1xrs11sERKEjTujz/IcxswUR6c=; b=PeQlarZHA8Q8SVrhCYuFtqbkvRlPrf7SMWwPYTOJ5qrLNlJiREAVqglAgKh4BGLLke mDnMmofpYYpzJi2WIf640C8ZW+gLZjuga6n4422KlQ+HC1R8Q+bPfYPJCMTmIqS2K6A6 F6TlLkRQ2ROJQp1Dya54MF44IL6J428c/SNG+shu0v3omt/WLB/K/Ube9w4Wq4qYbk5J MwD3tU4Mv+EitCFRI+ZyiFT+yMQF73L/LJw19HmMOWIbSypRiD9Mfgs7XyKtpZvCtMCp quDMD8pQXWGKMpS4AL5NIg3OvRD4eYsc1opn/UyBkhG/gYxXcsQVOWZCxB+R5eCoDd0S 4gxg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=FBWETjBx7yR8JhCne1xrs11sERKEjTujz/IcxswUR6c=; b=PxBuSz+rHvvXFmOGL9ccsTbYdE3cj7BGgEzy7Z8etFz2l06lvL38xzVsorc9cieetS QrzStqG9IRWAgb1mV7Nq0hsK8NdjjIqw7hMN7kEwd71GZpgxybeEjy1fqOLluSexStNr oZs55nd07AahvdkTzyjjhwrL7eR1VUHU740FtNcyKKVbEymyOShw+vkQ4U+WwthHrsCl ocqrSDq2mXd5IUZ0GmC8Xm4MnlpJwZvPthnfENPtkOgna4lXmSd62nlk9zOv+zZuozI+ jSw9H4iHNxwz5+/qOgBH0uivHAyxf8fjWMZ0E+WZ9u/YcEK1FA1klK+s/Xiwkbo3V3+Y sc1A== X-Gm-Message-State: AOAM531asKALMpTD/nxNXcEkiSDjcinBoh9JEheTaQBmfAIgdvc7yocW W50vyEXYmM57Ee+748oQpEuOGXwFQkw= X-Google-Smtp-Source: ABdhPJxdIXS+YrhlhQAMGO2tdAUOEpd7VE+G4qViNr/uNDCzhPVD9kOL0KJCgnN8zxSi5wP0E0eOpg== X-Received: by 2002:a05:600c:2282:: with SMTP id 2mr20082767wmf.93.1615692241253; Sat, 13 Mar 2021 19:24:01 -0800 (PST) From: Mahmoud Mandour To: qemu-devel@nongnu.org Subject: [PATCH 6/8] tools/virtiofsd/buffer.c: replaced a calloc call with GLib's g_try_new0 Date: Sun, 14 Mar 2021 05:23:22 +0200 Message-Id: <20210314032324.45142-7-ma.mandourr@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210314032324.45142-1-ma.mandourr@gmail.com> References: <20210314032324.45142-1-ma.mandourr@gmail.com> MIME-Version: 1.0 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=2a00:1450:4864:20::336; envelope-from=ma.mandourr@gmail.com; helo=mail-wm1-x336.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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: , Cc: Mahmoud Mandour , "Dr. David Alan Gilbert" , Stefan Hajnoczi Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @gmail.com) Content-Type: text/plain; charset="utf-8" Replaced a call to calloc() and its respective free() call with GLib's g_try_new0() and g_free() calls. Signed-off-by: Mahmoud Mandour Reviewed-by: Stefan Hajnoczi --- tools/virtiofsd/buffer.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/virtiofsd/buffer.c b/tools/virtiofsd/buffer.c index 874f01c488..b5f04be356 100644 --- a/tools/virtiofsd/buffer.c +++ b/tools/virtiofsd/buffer.c @@ -37,7 +37,7 @@ static ssize_t fuse_buf_writev(struct fuse_buf *out_buf, struct iovec *iov; int fd =3D out_buf->fd; =20 - iov =3D calloc(iovcnt, sizeof(struct iovec)); + iov =3D g_try_new0(struct iovec, iovcnt); if (!iov) { return -ENOMEM; } @@ -61,7 +61,7 @@ static ssize_t fuse_buf_writev(struct fuse_buf *out_buf, res =3D -errno; } =20 - free(iov); + g_free(iov); return res; } =20 --=20 2.25.1 From nobody Sun May 19 02:38:12 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=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1615692536; cv=none; d=zohomail.com; s=zohoarc; b=CIihDbiLJZPV9F1liCdk+C+gldUCzUO2qiL3S9wcV1PbuGWDB41JHjAkSG9rdT+MRxdUXRidLob6s42TRfFqHO+rec0qAhnKFyUNCLAjC0h4VTajG2dgC06P6NJZkNOH7bwJNr8a6aqZfpySOQRW6vfEcZg1ortfpwfk8I2nTFM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1615692536; h=Content-Transfer-Encoding:Cc: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=oCmAvra+9pdnWt5HPQ7qb167aeUA8Zrq2LTIBRXk/CU=; b=RG9LX+mWgVsnhPX+hUPWfc1JMcn/qIJxegS80gPWeZC9DtDz84dkqctqqEdGVvjTvwXAs0OWFfhcHSIvkAWTVTTMRActtrY1Bm1xN2SsaGfT4i2eymnd5YkZMf/elxLGK/1oUY4/vNexJijSwPlMGOL2NS2R9w9clMbflfuewMs= 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 161569253651016.40402743391303; Sat, 13 Mar 2021 19:28:56 -0800 (PST) Received: from localhost ([::1]:50204 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lLHR0-0003gO-HI for importer@patchew.org; Sat, 13 Mar 2021 22:28:54 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:48970) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lLHMM-0006Xp-Eh for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:06 -0500 Received: from mail-wr1-x431.google.com ([2a00:1450:4864:20::431]:37225) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lLHMK-0004T3-0l for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:06 -0500 Received: by mail-wr1-x431.google.com with SMTP id v15so6907495wrx.4 for ; Sat, 13 Mar 2021 19:24:03 -0800 (PST) Received: from localhost.localdomain ([197.61.164.30]) by smtp.googlemail.com with ESMTPSA id e1sm14922249wrd.44.2021.03.13.19.24.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 13 Mar 2021 19:24:02 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=oCmAvra+9pdnWt5HPQ7qb167aeUA8Zrq2LTIBRXk/CU=; b=C3fsQZjWINmodDF5RjZxzhEd867S9ad8EMBpBOK2Q7+q0isd3NK5dpbHqqq/9X3AYn CcZEsBitKYwCAgZFnErz4cGLCGU4VSgU1Yl2Ed2aUYr2kHpCvSsS86o3UHUTOfsP52sJ eylXltFmu1TprHKNng/SHnUnq+evSNxJ8HDmKs8zG/TPTBKdUe+UkW5aH/oWZay2YDMj HWjKjptNKXdYLWWee94gbQ/mwkncslEz2hoZiHVBaggIyVrctQ1mb+pVEi4m+qgx8+Nr htws7uf58LEKJnmFgmX7pHkKsDvE1ZfdDEkb5nrOn1gegQ7TZDWFcojxWWFJyz9micG1 cFTw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=oCmAvra+9pdnWt5HPQ7qb167aeUA8Zrq2LTIBRXk/CU=; b=KbEo6JjTOpFXDVo8inZFWphGWs+q/tAKelHnq7JScHmRJzL3EE17iA6l8oXeMbq8Te CGvkoLRU85BX4z4NujboE8P7S/5QmLdsZaW3SsWr+4GLCSwEOH4scbV9yo4u3RHnyC+U gWEWGoDFsxC4Fu4bENECiwGW63Ej29QKa8wRRXRvxQJDZYpqCETWRI13fzATHOZOUd9W 82Of7kHk4F9D2SKm3L04FI54Dlz0s+a8HtJTawXKXOdBJIYOrJT6qaPTXU7PRMSAf/KK Wz9a0pEP7h4kfKB34g5J36v4Od4juOPb0L7XVaTNf6kZEHzFvWbtsDkcAXOhauTfXgpp 3JFA== X-Gm-Message-State: AOAM530wvd6nc0pMroqhTtjEyX8VPo/e4REWBpIBIjUJAkTeSwDOHb32 ZNmFq5UAZhebpA6dv2axAxN0xr5e3ac= X-Google-Smtp-Source: ABdhPJwNvu/QQE13zP/lB4aBNUJaPdXgPka47jUDe77Ix3n87gJ/PuI3hmAVV2WdMnMfqOuU1A62sw== X-Received: by 2002:a5d:6290:: with SMTP id k16mr20354363wru.177.1615692242460; Sat, 13 Mar 2021 19:24:02 -0800 (PST) From: Mahmoud Mandour To: qemu-devel@nongnu.org Subject: [PATCH 7/8] tools/virtiofsd/fuse_opt.c: Replaced a malloc with GLib's g_try_malloc Date: Sun, 14 Mar 2021 05:23:23 +0200 Message-Id: <20210314032324.45142-8-ma.mandourr@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210314032324.45142-1-ma.mandourr@gmail.com> References: <20210314032324.45142-1-ma.mandourr@gmail.com> MIME-Version: 1.0 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=2a00:1450:4864:20::431; envelope-from=ma.mandourr@gmail.com; helo=mail-wr1-x431.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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: , Cc: Mahmoud Mandour , "Dr. David Alan Gilbert" , Stefan Hajnoczi Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @gmail.com) Content-Type: text/plain; charset="utf-8" Replaced a malloc() call and its respective free() with GLib's g_try_malloc() and g_free() calls. Signed-off-by: Mahmoud Mandour Reviewed-by: Stefan Hajnoczi --- tools/virtiofsd/fuse_opt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/virtiofsd/fuse_opt.c b/tools/virtiofsd/fuse_opt.c index f0ab8d22f4..9d371448e9 100644 --- a/tools/virtiofsd/fuse_opt.c +++ b/tools/virtiofsd/fuse_opt.c @@ -272,7 +272,7 @@ static int process_opt_sep_arg(struct fuse_opt_context = *ctx, } =20 param =3D ctx->argv[ctx->argctr]; - newarg =3D malloc(sep + strlen(param) + 1); + newarg =3D g_try_malloc(sep + strlen(param) + 1); if (!newarg) { return alloc_failed(); } @@ -280,7 +280,7 @@ static int process_opt_sep_arg(struct fuse_opt_context = *ctx, memcpy(newarg, arg, sep); strcpy(newarg + sep, param); res =3D process_opt(ctx, opt, sep, newarg, iso); - free(newarg); + g_free(newarg); =20 return res; } --=20 2.25.1 From nobody Sun May 19 02:38:12 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=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1615692597; cv=none; d=zohomail.com; s=zohoarc; b=HujaWb4gsBnmRd7URcao8pHYuNk4aNsJVygxJinc2y9DOlUBI/MePRt+HilccLDBBuWJ9rId6uvT+21hicBdx0wIjm+/SjeXYeUlhVRXb+KvNIX6bRMOV7STBJXmdbZek3tI10ZWb1vgWlP7pvyB4qXH7dK0O3JuiO9bXWrZX3M= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1615692597; h=Content-Transfer-Encoding:Cc: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=mktypJvu/1F6oyY228R0OtNNmqjKKSzXTpTB6Y2kEiY=; b=aTxg4vN/MtZbtEFDX823gYi2hy0Nsc320/cUSAlXUO+GM4az3Cnrn1KVy373tE7u9IIvvbYFbjZJXmNG0skbPAdOL8y3ecn0f+OQH3DP0ftwZGnf3PBi1t2O9PqcWyw/yCa2RRYsEkrHPPepsoWo+BljgmkpBWeOol7MeL7qcKo= 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 1615692597293205.12718205288036; Sat, 13 Mar 2021 19:29:57 -0800 (PST) Received: from localhost ([::1]:54178 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lLHRz-0005JE-3J for importer@patchew.org; Sat, 13 Mar 2021 22:29:55 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:48980) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lLHMO-0006YS-QT for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:09 -0500 Received: from mail-wm1-x330.google.com ([2a00:1450:4864:20::330]:39644) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lLHMM-0004TE-78 for qemu-devel@nongnu.org; Sat, 13 Mar 2021 22:24:08 -0500 Received: by mail-wm1-x330.google.com with SMTP id 124-20020a1c00820000b029010b871409cfso18126360wma.4 for ; Sat, 13 Mar 2021 19:24:05 -0800 (PST) Received: from localhost.localdomain ([197.61.164.30]) by smtp.googlemail.com with ESMTPSA id e1sm14922249wrd.44.2021.03.13.19.24.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 13 Mar 2021 19:24:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=mktypJvu/1F6oyY228R0OtNNmqjKKSzXTpTB6Y2kEiY=; b=lJ+JDP2UnJpvyBNe0cBpfBUBwhyPhZHBnqHAfQ9vRxnseAU7MSxw8j6FVvFGyQyrQo vLC65EQJ6JAj7C6kLA7AYwV7j+IIMYM7WzssHHRNt/yNK/bKfjgEYiqtEknIm6itLnn+ QDe+HeiGYIMQ5Sa3avjqmls8TFvxpEMzIZiTzTLYApVqri+1NLRFUAOok3uWmq/KN8Bo KSYrqKbU+8VzINyDiPajDftO9a/jp7W8ByVYfQS2TKtF6QmHxN11BCDO433hN9DJtPrZ 3tWR/+s3rdDGAxdhyKwQD7sZFmvAIEpQRqC6DZFzmuOUyeg317p7FZR0P9DKUlOfXSj1 A8Xg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=mktypJvu/1F6oyY228R0OtNNmqjKKSzXTpTB6Y2kEiY=; b=TwTq8lcux6CImm5L9fz0Jek27j0xhZjaxGQeOJiBuM3qcN4g+ykvsuJlDqNkDsyi82 vMGvP/CuZ5CsI3ldPZI5rJRNJM+fDsAy+AIkgs+W0EZehHu3WPk6qujx+uO78lV2nKqC EnGbeIRYKTOiq2Xl97ojCJ0BR/zhafIFna00m2NY7pT4nGFry/kJbJ7WFJGOO9c8bbY+ xXryvYWSfN1su6RPeFOJ8WoDpw2xTJ4nDqQOx6ujANg36mYwsax0TlrAzmnbghDfm1R7 YCM0rQP+2J9G8YlQllRxf5u0qrZLW58u+ZlSkQ0MxDL1kODnVAoz3fgg/Rrc5vAc9ld4 zEZA== X-Gm-Message-State: AOAM530NudQ65uYwd1D/UTre4ev/qa9Xdtc6KbI2d/GB+e2uASYTfRy6 c909eoazJDQK6zIs9FlORRGteust6sw= X-Google-Smtp-Source: ABdhPJy016qykM4zKN9Qi6JfK3qVDPwP9o/T39Ms+MDkUkUfokzes1Qf5zLlxXbld2c1dZQT0uzWYA== X-Received: by 2002:a1c:4e0e:: with SMTP id g14mr19484034wmh.160.1615692243866; Sat, 13 Mar 2021 19:24:03 -0800 (PST) From: Mahmoud Mandour To: qemu-devel@nongnu.org Subject: [PATCH 8/8] tools/virtiofsd: Replacing malloc-like calls with GLib's variants Date: Sun, 14 Mar 2021 05:23:24 +0200 Message-Id: <20210314032324.45142-9-ma.mandourr@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210314032324.45142-1-ma.mandourr@gmail.com> References: <20210314032324.45142-1-ma.mandourr@gmail.com> MIME-Version: 1.0 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=2a00:1450:4864:20::330; envelope-from=ma.mandourr@gmail.com; helo=mail-wm1-x330.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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: , Cc: Mahmoud Mandour , "Dr. David Alan Gilbert" , Stefan Hajnoczi Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @gmail.com) Content-Type: text/plain; charset="utf-8" Changed calls to malloc(), calloc(), and realloc() with their equivalent allocation functions in GLib, and replaced their respective free() calls with g_free(). Allocation and deallocation of fuse_req structs, fuse_pollhandle structs, fuse_session structs and many local variables are now established through GLib's functions. Signed-off-by: Mahmoud Mandour --- tools/virtiofsd/fuse_lowlevel.c | 30 ++++++++++++++-------------- tools/virtiofsd/fuse_virtio.c | 34 ++++++++++++++++---------------- tools/virtiofsd/passthrough_ll.c | 32 +++++++++++++++--------------- 3 files changed, 48 insertions(+), 48 deletions(-) diff --git a/tools/virtiofsd/fuse_lowlevel.c b/tools/virtiofsd/fuse_lowleve= l.c index 1aa26c6333..5e188f8d8f 100644 --- a/tools/virtiofsd/fuse_lowlevel.c +++ b/tools/virtiofsd/fuse_lowlevel.c @@ -106,7 +106,7 @@ static void list_add_req(struct fuse_req *req, struct f= use_req *next) static void destroy_req(fuse_req_t req) { pthread_mutex_destroy(&req->lock); - free(req); + g_free(req); } =20 void fuse_free_req(fuse_req_t req) @@ -130,7 +130,7 @@ static struct fuse_req *fuse_ll_alloc_req(struct fuse_s= ession *se) { struct fuse_req *req; =20 - req =3D (struct fuse_req *)calloc(1, sizeof(struct fuse_req)); + req =3D g_try_new(struct fuse_req, 1); if (req =3D=3D NULL) { fuse_log(FUSE_LOG_ERR, "fuse: failed to allocate request\n"); } else { @@ -219,7 +219,7 @@ int fuse_reply_iov(fuse_req_t req, const struct iovec *= iov, int count) int res; struct iovec *padded_iov; =20 - padded_iov =3D malloc((count + 1) * sizeof(struct iovec)); + padded_iov =3D g_try_new(struct iovec, count + 1); if (padded_iov =3D=3D NULL) { return fuse_reply_err(req, ENOMEM); } @@ -228,7 +228,7 @@ int fuse_reply_iov(fuse_req_t req, const struct iovec *= iov, int count) count++; =20 res =3D send_reply_iov(req, 0, padded_iov, count); - free(padded_iov); + g_free(padded_iov); =20 return res; } @@ -568,7 +568,7 @@ static struct fuse_ioctl_iovec *fuse_ioctl_iovec_copy(c= onst struct iovec *iov, struct fuse_ioctl_iovec *fiov; size_t i; =20 - fiov =3D malloc(sizeof(fiov[0]) * count); + fiov =3D g_try_new(struct fuse_ioctl_iovec, count); if (!fiov) { return NULL; } @@ -629,8 +629,8 @@ int fuse_reply_ioctl_retry(fuse_req_t req, const struct= iovec *in_iov, =20 res =3D send_reply_iov(req, 0, iov, count); out: - free(in_fiov); - free(out_fiov); + g_free(in_fiov); + g_free(out_fiov); =20 return res; =20 @@ -667,7 +667,7 @@ int fuse_reply_ioctl_iov(fuse_req_t req, int result, co= nst struct iovec *iov, struct fuse_ioctl_out arg; int res; =20 - padded_iov =3D malloc((count + 2) * sizeof(struct iovec)); + padded_iov =3D g_try_new(struct iovec, count + 2); if (padded_iov =3D=3D NULL) { return fuse_reply_err(req, ENOMEM); } @@ -680,7 +680,7 @@ int fuse_reply_ioctl_iov(fuse_req_t req, int result, co= nst struct iovec *iov, memcpy(&padded_iov[2], iov, count * sizeof(struct iovec)); =20 res =3D send_reply_iov(req, 0, padded_iov, count + 2); - free(padded_iov); + g_free(padded_iov); =20 return res; } @@ -1684,7 +1684,7 @@ static struct fuse_req *check_interrupt(struct fuse_s= ession *se, if (curr->u.i.unique =3D=3D req->unique) { req->interrupted =3D 1; list_del_req(curr); - free(curr); + g_free(curr); return NULL; } } @@ -1760,7 +1760,7 @@ static void do_ioctl(fuse_req_t req, fuse_ino_t nodei= d, =20 void fuse_pollhandle_destroy(struct fuse_pollhandle *ph) { - free(ph); + g_free(ph); } =20 static void do_poll(fuse_req_t req, fuse_ino_t nodeid, @@ -1783,7 +1783,7 @@ static void do_poll(fuse_req_t req, fuse_ino_t nodeid, struct fuse_pollhandle *ph =3D NULL; =20 if (arg->flags & FUSE_POLL_SCHEDULE_NOTIFY) { - ph =3D malloc(sizeof(struct fuse_pollhandle)); + ph =3D g_try_new(struct fuse_pollhandle, 1); if (ph =3D=3D NULL) { fuse_reply_err(req, ENOMEM); return; @@ -2476,7 +2476,7 @@ void fuse_session_destroy(struct fuse_session *se) free(se->vu_socket_path); se->vu_socket_path =3D NULL; =20 - free(se); + g_free(se); } =20 =20 @@ -2499,7 +2499,7 @@ struct fuse_session *fuse_session_new(struct fuse_arg= s *args, return NULL; } =20 - se =3D (struct fuse_session *)calloc(1, sizeof(struct fuse_session)); + se =3D g_try_new0(struct fuse_session, 1); if (se =3D=3D NULL) { fuse_log(FUSE_LOG_ERR, "fuse: failed to allocate fuse object\n"); goto out1; @@ -2559,7 +2559,7 @@ struct fuse_session *fuse_session_new(struct fuse_arg= s *args, out4: fuse_opt_free_args(args); out2: - free(se); + g_free(se); out1: return NULL; } diff --git a/tools/virtiofsd/fuse_virtio.c b/tools/virtiofsd/fuse_virtio.c index 523ee64fb7..598c97db1f 100644 --- a/tools/virtiofsd/fuse_virtio.c +++ b/tools/virtiofsd/fuse_virtio.c @@ -347,7 +347,7 @@ int virtio_send_data_iov(struct fuse_session *se, struc= t fuse_chan *ch, * Build a copy of the the in_sg iov so we can skip bits in it, * including changing the offsets */ - struct iovec *in_sg_cpy =3D calloc(sizeof(struct iovec), in_num); + struct iovec *in_sg_cpy =3D g_try_new0(struct iovec, in_num); assert(in_sg_cpy); memcpy(in_sg_cpy, in_sg, sizeof(struct iovec) * in_num); /* These get updated as we skip */ @@ -386,7 +386,7 @@ int virtio_send_data_iov(struct fuse_session *se, struc= t fuse_chan *ch, ret =3D errno; fuse_log(FUSE_LOG_DEBUG, "%s: preadv failed (%m) len=3D%zd\n", __func__, len); - free(in_sg_cpy); + g_free(in_sg_cpy); goto err; } fuse_log(FUSE_LOG_DEBUG, "%s: preadv ret=3D%d len=3D%zd\n", __func= __, @@ -410,13 +410,13 @@ int virtio_send_data_iov(struct fuse_session *se, str= uct fuse_chan *ch, if (ret !=3D len) { fuse_log(FUSE_LOG_DEBUG, "%s: ret!=3Dlen\n", __func__); ret =3D EIO; - free(in_sg_cpy); + g_free(in_sg_cpy); goto err; } in_sg_left -=3D ret; len -=3D ret; } while (in_sg_left); - free(in_sg_cpy); + g_free(in_sg_cpy); =20 /* Need to fix out->len on EOF */ if (len) { @@ -476,7 +476,7 @@ static void fv_queue_worker(gpointer data, gpointer use= r_data) * They're spread over multiple descriptors in a scatter/gather set * and we can't trust the guest to keep them still; so copy in/out. */ - fbuf.mem =3D malloc(se->bufsize); + fbuf.mem =3D g_try_malloc(se->bufsize); assert(fbuf.mem); =20 fuse_mutex_init(&req->ch.lock); @@ -528,10 +528,10 @@ static void fv_queue_worker(gpointer data, gpointer u= ser_data) fbuf.size =3D out_sg[0].iov_len + out_sg[1].iov_len; =20 /* Allocate the bufv, with space for the rest of the iov */ - pbufv =3D malloc(sizeof(struct fuse_bufvec) + + pbufv =3D g_try_malloc(sizeof(struct fuse_bufvec) + sizeof(struct fuse_buf) * (out_num - 2)); if (!pbufv) { - fuse_log(FUSE_LOG_ERR, "%s: pbufv malloc failed\n", + fuse_log(FUSE_LOG_ERR, "%s: pbufv g_try_malloc failed\n", __func__); goto out; } @@ -573,7 +573,7 @@ static void fv_queue_worker(gpointer data, gpointer use= r_data) =20 out: if (allocated_bufv) { - free(pbufv); + g_free(pbufv); } =20 /* If the request has no reply, still recycle the virtqueue element */ @@ -592,8 +592,8 @@ out: } =20 pthread_mutex_destroy(&req->ch.lock); - free(fbuf.mem); - free(req); + g_free(fbuf.mem); + g_free(req); } =20 /* Thread function for individual queues, created when a queue is 'started= ' */ @@ -733,7 +733,7 @@ static void fv_queue_cleanup_thread(struct fv_VuDev *vu= d, int qidx) pthread_mutex_destroy(&ourqi->vq_lock); close(ourqi->kill_fd); ourqi->kick_fd =3D -1; - free(vud->qi[qidx]); + g_free(vud->qi[qidx]); vud->qi[qidx] =3D NULL; } =20 @@ -764,14 +764,14 @@ static void fv_queue_set_started(VuDev *dev, int qidx= , bool started) if (started) { /* Fire up a thread to watch this queue */ if (qidx >=3D vud->nqueues) { - vud->qi =3D realloc(vud->qi, (qidx + 1) * sizeof(vud->qi[0])); + vud->qi =3D g_try_realloc_n(vud->qi, (qidx + 1), sizeof(vud->q= i[0])); assert(vud->qi); memset(vud->qi + vud->nqueues, 0, sizeof(vud->qi[0]) * (1 + (qidx - vud->nqueues))); vud->nqueues =3D qidx + 1; } if (!vud->qi[qidx]) { - vud->qi[qidx] =3D calloc(sizeof(struct fv_QueueInfo), 1); + vud->qi[qidx] =3D g_try_new0(struct fv_QueueInfo, 1); assert(vud->qi[qidx]); vud->qi[qidx]->virtio_dev =3D vud; vud->qi[qidx]->qidx =3D qidx; @@ -1032,9 +1032,9 @@ int virtio_session_mount(struct fuse_session *se) __func__); =20 /* TODO: Some cleanup/deallocation! */ - se->virtio_dev =3D calloc(sizeof(struct fv_VuDev), 1); + se->virtio_dev =3D g_try_new0(struct fv_VuDev, 1); if (!se->virtio_dev) { - fuse_log(FUSE_LOG_ERR, "%s: virtio_dev calloc failed\n", __func__); + fuse_log(FUSE_LOG_ERR, "%s: virtio_dev g_try_new0 failed\n", __fun= c__); close(data_sock); return -1; } @@ -1059,8 +1059,8 @@ void virtio_session_close(struct fuse_session *se) return; } =20 - free(se->virtio_dev->qi); + g_free(se->virtio_dev->qi); pthread_rwlock_destroy(&se->virtio_dev->vu_dispatch_rwlock); - free(se->virtio_dev); + g_free(se->virtio_dev); se->virtio_dev =3D NULL; } diff --git a/tools/virtiofsd/passthrough_ll.c b/tools/virtiofsd/passthrough= _ll.c index fc7e1b1e8e..5c475a30af 100644 --- a/tools/virtiofsd/passthrough_ll.c +++ b/tools/virtiofsd/passthrough_ll.c @@ -399,7 +399,7 @@ static void lo_map_init(struct lo_map *map) =20 static void lo_map_destroy(struct lo_map *map) { - free(map->elems); + g_free(map->elems); } =20 static int lo_map_grow(struct lo_map *map, size_t new_nelems) @@ -411,7 +411,7 @@ static int lo_map_grow(struct lo_map *map, size_t new_n= elems) return 1; } =20 - new_elems =3D realloc(map->elems, sizeof(map->elems[0]) * new_nelems); + new_elems =3D g_realloc_n(map->elems, new_nelems, sizeof(map->elems[0]= )); if (!new_elems) { return 0; } @@ -549,7 +549,7 @@ static void lo_inode_put(struct lo_data *lo, struct lo_= inode **inodep) =20 if (g_atomic_int_dec_and_test(&inode->refcount)) { close(inode->fd); - free(inode); + g_free(inode); } } =20 @@ -904,7 +904,7 @@ static void posix_locks_value_destroy(gpointer data) * closing this fd should release all OFD locks. */ close(plock->fd); - free(plock); + g_free(plock); } =20 static int do_statx(struct lo_data *lo, int dirfd, const char *pathname, @@ -1020,7 +1020,7 @@ static int lo_do_lookup(fuse_req_t req, fuse_ino_t pa= rent, const char *name, if (inode) { close(newfd); } else { - inode =3D calloc(1, sizeof(struct lo_inode)); + inode =3D g_try_new0(struct lo_inode, 1); if (!inode) { goto out_err; } @@ -1532,7 +1532,7 @@ static void lo_dirp_put(struct lo_dirp **dp) =20 if (g_atomic_int_dec_and_test(&d->refcount)) { closedir(d->dp); - free(d); + g_free(d); } } =20 @@ -1564,7 +1564,7 @@ static void lo_opendir(fuse_req_t req, fuse_ino_t ino, int fd; ssize_t fh; =20 - d =3D calloc(1, sizeof(struct lo_dirp)); + d =3D g_try_new0(struct lo_dirp, 1); if (d =3D=3D NULL) { goto out_err; } @@ -1606,7 +1606,7 @@ out_err: } else if (fd !=3D -1) { close(fd); } - free(d); + g_free(d); } fuse_reply_err(req, error); } @@ -1633,7 +1633,7 @@ static void lo_do_readdir(fuse_req_t req, fuse_ino_t = ino, size_t size, } =20 err =3D ENOMEM; - buf =3D calloc(1, size); + buf =3D g_try_malloc0(size); if (!buf) { goto error; } @@ -1719,7 +1719,7 @@ error: } else { fuse_reply_buf(req, buf, size - rem); } - free(buf); + g_free(buf); } =20 static void lo_readdir(fuse_req_t req, fuse_ino_t ino, size_t size, @@ -1943,7 +1943,7 @@ static struct lo_inode_plock *lookup_create_plock_ctx= (struct lo_data *lo, return plock; } =20 - plock =3D malloc(sizeof(struct lo_inode_plock)); + plock =3D g_try_new(struct lo_inode_plock, 1); if (!plock) { *err =3D ENOMEM; return NULL; @@ -1954,7 +1954,7 @@ static struct lo_inode_plock *lookup_create_plock_ctx= (struct lo_data *lo, fd =3D lo_inode_open(lo, inode, O_RDWR); if (fd < 0) { *err =3D -fd; - free(plock); + g_free(plock); return NULL; } =20 @@ -2731,7 +2731,7 @@ static void lo_getxattr(fuse_req_t req, fuse_ino_t in= o, const char *in_name, ino, name, size); =20 if (size) { - value =3D malloc(size); + value =3D g_try_malloc(size); if (!value) { goto out_err; } @@ -2770,7 +2770,7 @@ static void lo_getxattr(fuse_req_t req, fuse_ino_t in= o, const char *in_name, fuse_reply_xattr(req, ret); } out_free: - free(value); + g_free(value); =20 if (fd >=3D 0) { close(fd); @@ -2812,7 +2812,7 @@ static void lo_listxattr(fuse_req_t req, fuse_ino_t i= no, size_t size) size); =20 if (size) { - value =3D malloc(size); + value =3D g_try_malloc(size); if (!value) { goto out_err; } @@ -2897,7 +2897,7 @@ static void lo_listxattr(fuse_req_t req, fuse_ino_t i= no, size_t size) fuse_reply_xattr(req, ret); } out_free: - free(value); + g_free(value); =20 if (fd >=3D 0) { close(fd); --=20 2.25.1