Message ID | 20170221141451.28305-13-marcandre.lureau@redhat.com |
---|---|
State | New |
Headers | show |
Marc-André Lureau <marcandre.lureau@redhat.com> writes: > The inside array should be free too. > Spotted by ASAN. > > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> > --- > tests/bios-tables-test.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c > index 54048050c0..d54018da73 100644 > --- a/tests/bios-tables-test.c > +++ b/tests/bios-tables-test.c > @@ -144,7 +144,7 @@ static void free_test_data(test_data *data) > g_free(temp->asl_file); > } > > - g_array_free(data->tables, false); > + g_array_free(data->tables, true); > } > > static uint8_t acpi_checksum(const uint8_t *data, int len) Reviewed-by: Markus Armbruster <armbru@redhat.com>
diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c index 54048050c0..d54018da73 100644 --- a/tests/bios-tables-test.c +++ b/tests/bios-tables-test.c @@ -144,7 +144,7 @@ static void free_test_data(test_data *data) g_free(temp->asl_file); } - g_array_free(data->tables, false); + g_array_free(data->tables, true); } static uint8_t acpi_checksum(const uint8_t *data, int len)
The inside array should be free too. Spotted by ASAN. Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> --- tests/bios-tables-test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)