[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH 0/4] Pair g_malloc() with g_free(
From: |
Eric Blake |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH 0/4] Pair g_malloc() with g_free(), not free() |
Date: |
Wed, 28 Jan 2015 08:13:48 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 |
On 01/28/2015 07:54 AM, Markus Armbruster wrote:
> I'm routing these patches through qemu-trivial, even though some of
> them touch actively maintained code, and could go through the
> respective tree instead:
>
> * PATCH 3 sPAPR (Alex)
>
> * PATCH 4 USB (Gerd)
>
> If you want me to reroute any of them, let me know.
Reviewed-by: Eric Blake <address@hidden>
[for patches like these, sometimes I like to use git format-patch -U20
or whatever it takes to make the patch show full context of the
allocation a few lines earlier that matches with the actual g_free fix,
so that reviewers don't have to open the file themselves. It makes for
larger emails, but potentially faster reviews. But it's not the end of
the world to stick with the default 3-line context]
>
> Markus Armbruster (4):
> qemu-option: Replace pointless use of g_malloc0() by g_malloc()
> qemu-option: Pair g_malloc() with g_free(), not free()
> spapr_vio: Pair g_malloc() with g_free(), not free()
> usb: Pair g_malloc() with g_free(), not free()
>
> hw/ppc/spapr_vio.c | 2 +-
> hw/usb/desc-msos.c | 2 +-
> util/qemu-option.c | 8 ++++----
> 3 files changed, 6 insertions(+), 6 deletions(-)
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-trivial] [PATCH 0/4] Pair g_malloc() with g_free(), not free(), Markus Armbruster, 2015/01/28
- [Qemu-trivial] [PATCH 2/4] qemu-option: Pair g_malloc() with g_free(), not free(), Markus Armbruster, 2015/01/28
- [Qemu-trivial] [PATCH 3/4] spapr_vio: Pair g_malloc() with g_free(), not free(), Markus Armbruster, 2015/01/28
- [Qemu-trivial] [PATCH 1/4] qemu-option: Replace pointless use of g_malloc0() by g_malloc(), Markus Armbruster, 2015/01/28
- [Qemu-trivial] [PATCH 4/4] usb: Pair g_malloc() with g_free(), not free(), Markus Armbruster, 2015/01/28
- Re: [Qemu-trivial] [Qemu-devel] [PATCH 0/4] Pair g_malloc() with g_free(), not free(),
Eric Blake <=