diff options
author | Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> | 2012-09-22 22:56:09 +0000 |
---|---|---|
committer | Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> | 2012-09-22 22:56:09 +0000 |
commit | e0ff1652a60e3fdcd82e3def8c9d13b071a4d05d (patch) | |
tree | 71c48f27dadf37c78b9e9da5defba243ea3a2be3 | |
parent | 5609f9d408645657ef0dd2ca986ada5aaad4c875 (diff) | |
download | flashrom-e0ff1652a60e3fdcd82e3def8c9d13b071a4d05d.tar.gz flashrom-e0ff1652a60e3fdcd82e3def8c9d13b071a4d05d.tar.bz2 flashrom-e0ff1652a60e3fdcd82e3def8c9d13b071a4d05d.zip |
Fix a memleak in the dummyflasher
Emu_persistent_image was not freed correctly.
This bug was found thanks to valgrind.
Corresponding to flashrom svn r1603.
Signed-off-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
-rw-r--r-- | dummyflasher.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/dummyflasher.c b/dummyflasher.c index 2803f76f..655b6781 100644 --- a/dummyflasher.c +++ b/dummyflasher.c @@ -150,8 +150,9 @@ static int dummy_shutdown(void *data) if (emu_chip != EMULATE_NONE) { if (emu_persistent_image) { msg_pdbg("Writing %s\n", emu_persistent_image); - write_buf_to_file(flashchip_contents, emu_chip_size, - emu_persistent_image); + write_buf_to_file(flashchip_contents, emu_chip_size, emu_persistent_image); + free(emu_persistent_image); + emu_persistent_image = NULL; } free(flashchip_contents); } |