[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 31/31] eepro100: replace g_malloc()+memcpy() wit
From: |
Stefan Weil |
Subject: |
Re: [Qemu-devel] [PATCH 31/31] eepro100: replace g_malloc()+memcpy() with g_memdup() |
Date: |
Thu, 22 Jun 2017 15:09:53 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
Am 22.06.2017 um 14:42 schrieb Marc-André Lureau:
> I found these pattern via grepping the source tree. I don't have a
> coccinelle script for it!
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> hw/net/eepro100.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/hw/net/eepro100.c b/hw/net/eepro100.c
> index 4bf71f2d85..7d3b2e52c7 100644
> --- a/hw/net/eepro100.c
> +++ b/hw/net/eepro100.c
> @@ -1894,8 +1894,7 @@ static void e100_nic_realize(PCIDevice *pci_dev, Error
> **errp)
>
> qemu_register_reset(nic_reset, s);
>
> - s->vmstate = g_malloc(sizeof(vmstate_eepro100));
> - memcpy(s->vmstate, &vmstate_eepro100, sizeof(vmstate_eepro100));
> + s->vmstate = g_memdup(&vmstate_eepro100, sizeof(vmstate_eepro100));
> s->vmstate->name = qemu_get_queue(s->nic)->model;
> vmstate_register(&pci_dev->qdev, -1, s->vmstate, s);
> }
>
Reviewed-by: Stefan Weil <address@hidden>
- [Qemu-devel] [PATCH 23/31] ppc: use DIV_ROUND_UP, (continued)
- [Qemu-devel] [PATCH 23/31] ppc: use DIV_ROUND_UP, Marc-André Lureau, 2017/06/22
- [Qemu-devel] [PATCH 24/31] i386/dump: use DIV_ROUND_UP, Marc-André Lureau, 2017/06/22
- [Qemu-devel] [PATCH 25/31] kvm: use DIV_ROUND_UP, Marc-André Lureau, 2017/06/22
- [Qemu-devel] [PATCH 26/31] decnumber: use DIV_ROUND_UP, Marc-André Lureau, 2017/06/22
- [Qemu-devel] [PATCH 27/31] i386: introduce ELF_NOTE_SIZE macro, Marc-André Lureau, 2017/06/22
- [Qemu-devel] [PATCH 28/31] 9pfs: replace g_malloc()+memcpy() with g_memdup(), Marc-André Lureau, 2017/06/22
- [Qemu-devel] [PATCH 29/31] i386: replace g_malloc()+memcpy() with g_memdup(), Marc-André Lureau, 2017/06/22
- [Qemu-devel] [PATCH 30/31] test-iov: replace g_malloc()+memcpy() with g_memdup(), Marc-André Lureau, 2017/06/22
- [Qemu-devel] [PATCH 31/31] eepro100: replace g_malloc()+memcpy() with g_memdup(), Marc-André Lureau, 2017/06/22
- Re: [Qemu-devel] [PATCH 31/31] eepro100: replace g_malloc()+memcpy() with g_memdup(),
Stefan Weil <=
- Re: [Qemu-devel] [PATCH 00/31] Refactoring with clang-tidy, Peter Maydell, 2017/06/22
- Re: [Qemu-devel] [PATCH 00/31] Refactoring with clang-tidy, no-reply, 2017/06/22