[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] elfload: use g_new instead of malloc
From: |
Thomas Huth |
Subject: |
Re: [PATCH] elfload: use g_new instead of malloc |
Date: |
Fri, 2 Oct 2020 07:18:06 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0 |
On 02/10/2020 07.05, Markus Armbruster wrote:
> Elena Afanasova <eafanasova@gmail.com> writes:
>
>> Signed-off-by: Elena Afanasova <eafanasova@gmail.com>
>> ---
>> bsd-user/elfload.c | 92 +++++++++++++++-------------------------------
>> 1 file changed, 30 insertions(+), 62 deletions(-)
>>
>> diff --git a/bsd-user/elfload.c b/bsd-user/elfload.c
>> index 32378af7b2..e10ca54eb7 100644
>> --- a/bsd-user/elfload.c
>> +++ b/bsd-user/elfload.c
>> @@ -867,18 +867,14 @@ 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;
>>
>> - elf_phdata = (struct elf_phdr *)
>> - malloc(sizeof(struct elf_phdr) * interp_elf_ex->e_phnum);
>> -
>> - if (!elf_phdata)
>> - return ~((abi_ulong)0UL);
>> + elf_phdata = g_new(struct elf_phdr, interp_elf_ex->e_phnum);
>>
>> /*
>> * If the size of this structure has changed, then punt, since
>> * we will be doing the wrong thing.
>> */
>> if (interp_elf_ex->e_phentsize != sizeof(struct elf_phdr)) {
>> - free(elf_phdata);
>> + g_free(elf_phdata);
>> return ~((abi_ulong)0UL);
>> }
>>
>> @@ -890,9 +886,8 @@ static abi_ulong load_elf_interp(struct elfhdr *
>> interp_elf_ex,
>> }
>> if (retval < 0) {
>> perror("load_elf_interp");
>> + g_free(elf_phdata);
>> exit(-1);
>> - free (elf_phdata);
>> - return retval;
>
> Deleting return looks wrong.
Why? There is an exit(-1) right in front of it, so this is dead code...
well, maybe that should be done in a separate patch, or at least
mentioned in the patch description, though.
Thomas
- [PATCH] elfload: use g_new instead of malloc, Elena Afanasova, 2020/10/01
- Re: [PATCH] elfload: use g_new instead of malloc, Thomas Huth, 2020/10/01
- Re: [PATCH] elfload: use g_new instead of malloc, Markus Armbruster, 2020/10/02
- Re: [PATCH] elfload: use g_new instead of malloc,
Thomas Huth <=
- Re: [PATCH] elfload: use g_new instead of malloc, Markus Armbruster, 2020/10/02
- Re: [PATCH] elfload: use g_new instead of malloc, Eric Blake, 2020/10/02
- Re: [PATCH v2] elfload: use g_new/g_malloc and g_autofree, Elena Afanasova, 2020/10/04
- Re: [PATCH v2] elfload: use g_new/g_malloc and g_autofree, Markus Armbruster, 2020/10/05
- Re: [PATCH v2] elfload: use g_new/g_malloc and g_autofree, Peter Maydell, 2020/10/05
- Re: [PATCH v2] elfload: use g_new/g_malloc and g_autofree, Elena Afanasova, 2020/10/06