Nico Huber submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved Edward O'Callaghan: Looks good to me, approved
dummyflasher.c: Fix memory leak on shutdown

emu_data *data is allocated in init function and needs
to be freed in shutdown function.

BUG=b:181803212
TEST=builds and ninja test

Change-Id: I36f76d84d3547d081c64857e06da23ee63cc5594
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/52557
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
---
M dummyflasher.c
1 file changed, 1 insertion(+), 0 deletions(-)

diff --git a/dummyflasher.c b/dummyflasher.c
index 5190282..ec5060d 100644
--- a/dummyflasher.c
+++ b/dummyflasher.c
@@ -650,6 +650,7 @@
free(flashchip_contents);
}
#endif
+ free(data);
return 0;
}


To view, visit change 52557. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I36f76d84d3547d081c64857e06da23ee63cc5594
Gerrit-Change-Number: 52557
Gerrit-PatchSet: 2
Gerrit-Owner: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-MessageType: merged