test: Free the memory, not the pointer to the memory

In read_file(), we see:

    *data = malloc (*len);
    ...
    if (fread(*data, *len, 1, fp) != 1) {
      free(data);
      ...

The free call needs to be free(*data), to match the malloc call.

Matthias Clasen found this via Coverity and proposed the fix.

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=91381

1 files changed, 1 insertions, 1 deletions

diff --git a/test/pdf-mime-data.c b/test/pdf-mime-data.c
index fd5af1f7f..b49a1d18d 100644
--- a/test/pdf-mime-data.c
+++ b/test/pdf-mime-data.c
@@ -82,7 +82,7 @@ read_file (const cairo_test_context_t *ctx,
     }
 
     if (fread(*data, *len, 1, fp) != 1) {
-	free (data);
+	free (*data);
 	fclose(fp);
 	cairo_test_log (ctx, "Could not read data from file %s\n", file);
 	return CAIRO_TEST_FAILURE;