lib/bitmap: Fix a potential memory leak (#3004)

map is initinialized but not free'd Co-authored-by: Loïc Bartoletti <lbartoletti@qgis_dev.Oslandia.com>
OSGeo · Aug 16, 2023 · f51ceee · f51ceee
1 parent e353c3c
commit f51ceee
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/lib/bitmap/bitmap.c b/lib/bitmap/bitmap.c
@@ -315,8 +315,11 @@ struct BM *BM_file_read(FILE *fp)
     if (NULL == (map = (struct BM *)malloc(sizeof(struct BM))))
         return (NULL);
 
-    if (fread(&c, sizeof(char), sizeof(char), fp) != sizeof(char))
+    if (fread(&c, sizeof(char), sizeof(char), fp) != sizeof(char)) {
+        free(map);
         return NULL;
+    }
+
     if (c != BM_MAGIC) {
         free(map);
         return NULL;