From nobody Sun May 19 08:30:36 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; 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=git.sr.ht Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1697539451559990.7023147305449; Tue, 17 Oct 2023 03:44:11 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qshXd-0003FP-HZ; Tue, 17 Oct 2023 06:43:13 -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 1qshXa-0003BH-58 for qemu-devel@nongnu.org; Tue, 17 Oct 2023 06:43:10 -0400 Received: from mail-b.sr.ht ([173.195.146.151]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qshXY-0006Ot-3I for qemu-devel@nongnu.org; Tue, 17 Oct 2023 06:43:09 -0400 Received: from git.sr.ht (unknown [173.195.146.142]) by mail-b.sr.ht (Postfix) with ESMTPSA id 01D6811F37E; Tue, 17 Oct 2023 10:43:05 +0000 (UTC) Authentication-Results: mail-b.sr.ht; dkim=none From: ~h0lyalg0rithm Date: Tue, 03 Oct 2023 14:45:14 +0200 Subject: [PATCH qemu v3 1/1] Switch memory management calls to new coding conventions Message-ID: <169753938460.23804.11418813007617535750-1@git.sr.ht> X-Mailer: git.sr.ht In-Reply-To: <169753938460.23804.11418813007617535750-0@git.sr.ht> To: qemu-devel@nongnu.org Cc: trivial@nongnu.org Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 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=173.195.146.151; envelope-from=outgoing@sr.ht; helo=mail-b.sr.ht X-Spam_score_int: 36 X-Spam_score: 3.6 X-Spam_bar: +++ X-Spam_report: (3.6 / 5.0 requ) BAYES_00=-1.9, DATE_IN_PAST_96_XX=3.405, FREEMAIL_FORGED_REPLYTO=2.095, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=no 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: , Reply-To: ~h0lyalg0rithm Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZM-MESSAGEID: 1697539454017100006 From: Suraj Shirvankar Signed-off-by: Suraj Shirvankar --- contrib/elf2dmp/addrspace.c | 7 ++----- contrib/elf2dmp/main.c | 6 +++--- contrib/elf2dmp/pdb.c | 16 ++++++++-------- contrib/elf2dmp/qemu_elf.c | 7 ++----- 4 files changed, 15 insertions(+), 21 deletions(-) diff --git a/contrib/elf2dmp/addrspace.c b/contrib/elf2dmp/addrspace.c index 64b5d680ad..6f608a517b 100644 --- a/contrib/elf2dmp/addrspace.c +++ b/contrib/elf2dmp/addrspace.c @@ -72,10 +72,7 @@ int pa_space_create(struct pa_space *ps, QEMU_Elf *qemu_= elf) } } =20 - ps->block =3D malloc(sizeof(*ps->block) * ps->block_nr); - if (!ps->block) { - return 1; - } + ps->block =3D g_new(struct pa_block, ps->block_nr); =20 for (i =3D 0; i < phdr_nr; i++) { if (phdr[i].p_type =3D=3D PT_LOAD) { @@ -97,7 +94,7 @@ int pa_space_create(struct pa_space *ps, QEMU_Elf *qemu_e= lf) void pa_space_destroy(struct pa_space *ps) { ps->block_nr =3D 0; - free(ps->block); + g_free(ps->block); } =20 void va_space_set_dtb(struct va_space *vs, uint64_t dtb) diff --git a/contrib/elf2dmp/main.c b/contrib/elf2dmp/main.c index 5db163bdbe..26a15a9d22 100644 --- a/contrib/elf2dmp/main.c +++ b/contrib/elf2dmp/main.c @@ -120,14 +120,14 @@ static KDDEBUGGER_DATA64 *get_kdbg(uint64_t KernBase,= struct pdb_reader *pdb, } } =20 - kdbg =3D malloc(kdbg_hdr.Size); + kdbg =3D g_malloc(kdbg_hdr.Size); if (!kdbg) { return NULL; } =20 if (va_space_rw(vs, KdDebuggerDataBlock, kdbg, kdbg_hdr.Size, 0)) { eprintf("Failed to extract entire KDBG\n"); - free(kdbg); + g_free(kdbg); return NULL; } =20 @@ -643,7 +643,7 @@ int main(int argc, char *argv[]) } =20 out_kdbg: - free(kdbg); + g_free(kdbg); out_pdb: pdb_exit(&pdb); out_pdb_file: diff --git a/contrib/elf2dmp/pdb.c b/contrib/elf2dmp/pdb.c index 6ca5086f02..2404f41404 100644 --- a/contrib/elf2dmp/pdb.c +++ b/contrib/elf2dmp/pdb.c @@ -90,18 +90,18 @@ uint64_t pdb_resolve(uint64_t img_base, struct pdb_read= er *r, const char *name) =20 static void pdb_reader_ds_exit(struct pdb_reader *r) { - free(r->ds.toc); + g_free(r->ds.toc); } =20 static void pdb_exit_symbols(struct pdb_reader *r) { - free(r->modimage); - free(r->symbols); + g_free(r->modimage); + g_free(r->symbols); } =20 static void pdb_exit_segments(struct pdb_reader *r) { - free(r->segs); + g_free(r->segs); } =20 static void *pdb_ds_read(const PDB_DS_HEADER *header, @@ -116,7 +116,7 @@ static void *pdb_ds_read(const PDB_DS_HEADER *header, =20 nBlocks =3D (size + header->block_size - 1) / header->block_size; =20 - buffer =3D malloc(nBlocks * header->block_size); + buffer =3D g_malloc(nBlocks * header->block_size); if (!buffer) { return NULL; } @@ -201,7 +201,7 @@ static int pdb_init_symbols(struct pdb_reader *r) return 0; =20 out_symbols: - free(symbols); + g_free(symbols); =20 return err; } @@ -258,7 +258,7 @@ static int pdb_reader_init(struct pdb_reader *r, void *= data) out_sym: pdb_exit_symbols(r); out_root: - free(r->ds.root); + g_free(r->ds.root); out_ds: pdb_reader_ds_exit(r); =20 @@ -269,7 +269,7 @@ static void pdb_reader_exit(struct pdb_reader *r) { pdb_exit_segments(r); pdb_exit_symbols(r); - free(r->ds.root); + g_free(r->ds.root); pdb_reader_ds_exit(r); } =20 diff --git a/contrib/elf2dmp/qemu_elf.c b/contrib/elf2dmp/qemu_elf.c index de6ad744c6..055e6f8792 100644 --- a/contrib/elf2dmp/qemu_elf.c +++ b/contrib/elf2dmp/qemu_elf.c @@ -94,10 +94,7 @@ static int init_states(QEMU_Elf *qe) =20 printf("%zu CPU states has been found\n", cpu_nr); =20 - qe->state =3D malloc(sizeof(*qe->state) * cpu_nr); - if (!qe->state) { - return 1; - } + qe->state =3D g_new(QEMUCPUState*, cpu_nr); =20 cpu_nr =3D 0; =20 @@ -115,7 +112,7 @@ static int init_states(QEMU_Elf *qe) =20 static void exit_states(QEMU_Elf *qe) { - free(qe->state); + g_free(qe->state); } =20 static bool check_ehdr(QEMU_Elf *qe) --=20 2.38.5