03.10.2023 15:45, ~h0lyalg0rithm wrote:
From: Suraj Shirvankar <surajshirvan...@gmail.com>Signed-off-by: Suraj Shirvankar <surajshirvan...@gmail.com> --- contrib/elf2dmp/addrspace.c | 4 ++-- contrib/elf2dmp/main.c | 6 +++--- contrib/elf2dmp/pdb.c | 16 ++++++++-------- contrib/elf2dmp/qemu_elf.c | 4 ++-- 4 files changed, 15 insertions(+), 15 deletions(-) diff --git a/contrib/elf2dmp/addrspace.c b/contrib/elf2dmp/addrspace.c index 64b5d680ad..3bfbb5093c 100644 --- a/contrib/elf2dmp/addrspace.c +++ b/contrib/elf2dmp/addrspace.c @@ -72,7 +72,7 @@ int pa_space_create(struct pa_space *ps, QEMU_Elf *qemu_elf) } }- ps->block = malloc(sizeof(*ps->block) * ps->block_nr);+ ps->block = g_new(struct pa_block, ps->block_nr); if (!ps->block) { return 1; }
g_new(), unlike malloc(), does not return NULL (unless n_structs, the second argument, is 0). So it seems all checks for return value of g_new &Co should be dropped too, here and in all other places. /mjt
