[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 05/10] util/path: Use the GLib memory allocat
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH v5 05/10] util/path: Use the GLib memory allocation routines |
Date: |
Mon, 11 Aug 2014 10:32:45 +0100 |
zhanghailiang writes:
> In this file, we don't check the return value of malloc/strdup/realloc which
> may fail.
> Instead of using these APIs, we use the GLib memory APIs
> g_malloc/g_strdup/g_realloc.
> They will exit on allocation failure, so there is no need to test for failure,
> which would be fine for setup.
>
> Signed-off-by: zhanghailiang <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
> ---
> util/path.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/util/path.c b/util/path.c
> index 5c59d9f..e152f2a 100644
> --- a/util/path.c
> +++ b/util/path.c
> @@ -45,8 +45,8 @@ static struct pathelem *new_entry(const char *root,
> struct pathelem *parent,
> const char *name)
> {
> - struct pathelem *new = malloc(sizeof(*new));
> - new->name = strdup(name);
> + struct pathelem *new = g_malloc(sizeof(*new));
> + new->name = g_strdup(name);
> new->pathname = g_strdup_printf("%s/%s", root, name);
> new->num_entries = 0;
> return new;
> @@ -88,7 +88,7 @@ static struct pathelem *add_entry(struct pathelem *root,
> const char *name,
>
> root->num_entries++;
>
> - root = realloc(root, sizeof(*root)
> + root = g_realloc(root, sizeof(*root)
> + sizeof(root->entries[0])*root->num_entries);
> e = &root->entries[root->num_entries-1];
--
Alex Bennée
- [Qemu-devel] [PATCH v5 00/10] fix three bugs about use-after-free and several api abuse, zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 04/10] ivshmem: check the value returned by fstat(), zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 03/10] virtio-blk: fix reference a pointer which might be freed, zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 06/10] slirp/misc: Use g_malloc() instead of malloc(), zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 05/10] util/path: Use the GLib memory allocation routines, zhanghailiang, 2014/08/11
- Re: [Qemu-devel] [PATCH v5 05/10] util/path: Use the GLib memory allocation routines,
Alex Bennée <=
- [Qemu-devel] [PATCH v5 08/10] tests/bios-tables-test: check the value returned by fopen(), zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 02/10] monitor: fix access freed memory, zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 01/10] l2cap: fix access freed memory, zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 10/10] block/vvfat: fix setbuf stream parameter may be NULL, zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 09/10] tcg: check return value of fopen(), zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 07/10] linux-user: check return value of malloc(), zhanghailiang, 2014/08/11