* [Qemu-trivial] [PATCH] hw/smbus_eeprom.c : fix memory leak
@ 2011-12-02 3:23 Li Zhi Hui
2011-12-02 3:47 ` [Qemu-trivial] [Qemu-devel] " Mark Wu
0 siblings, 1 reply; 2+ messages in thread
From: Li Zhi Hui @ 2011-12-02 3:23 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-trivial, Li Zhi Hui, stefanha
Signed-off-by: Li Zhi Hui <zhihuili@linux.vnet.ibm.com>
---
hw/smbus_eeprom.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/hw/smbus_eeprom.c b/hw/smbus_eeprom.c
index 5d080ab..d66fbbc 100644
--- a/hw/smbus_eeprom.c
+++ b/hw/smbus_eeprom.c
@@ -142,4 +142,5 @@ void smbus_eeprom_init(i2c_bus *smbus, int nb_eeprom,
qdev_prop_set_ptr(eeprom, "data", eeprom_buf + (i * 256));
qdev_init_nofail(eeprom);
}
+ g_free(eeprom_buf);
}
--
1.7.4.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [Qemu-trivial] [Qemu-devel] [PATCH] hw/smbus_eeprom.c : fix memory leak
2011-12-02 3:23 [Qemu-trivial] [PATCH] hw/smbus_eeprom.c : fix memory leak Li Zhi Hui
@ 2011-12-02 3:47 ` Mark Wu
0 siblings, 0 replies; 2+ messages in thread
From: Mark Wu @ 2011-12-02 3:47 UTC (permalink / raw)
To: Li Zhi Hui; +Cc: qemu-trivial, qemu-devel, stefanha
On 12/02/2011 11:23 AM, Li Zhi Hui wrote:
> Signed-off-by: Li Zhi Hui<zhihuili@linux.vnet.ibm.com>
> ---
> hw/smbus_eeprom.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/hw/smbus_eeprom.c b/hw/smbus_eeprom.c
> index 5d080ab..d66fbbc 100644
> --- a/hw/smbus_eeprom.c
> +++ b/hw/smbus_eeprom.c
> @@ -142,4 +142,5 @@ void smbus_eeprom_init(i2c_bus *smbus, int nb_eeprom,
> qdev_prop_set_ptr(eeprom, "data", eeprom_buf + (i * 256));
> qdev_init_nofail(eeprom);
> }
> + g_free(eeprom_buf);
> }
The memory pointed by eeprom_buf is used as 'data' buf of 8
SMBusEEPROMDevice after initialization. And it calls qdev_init_nofail,
which will exit on failure. it should be ok to leave it there. Anyway,
you shouldn't free it in the initialization.
Mark.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-12-02 3:47 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-02 3:23 [Qemu-trivial] [PATCH] hw/smbus_eeprom.c : fix memory leak Li Zhi Hui
2011-12-02 3:47 ` [Qemu-trivial] [Qemu-devel] " Mark Wu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).