commit a208ea6301b3a7141f800d21407b0befa8ad0a56
author Elliott Hughes <enh@google.com> Wed Sep 18 23:35:01 2013 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Wed Sep 18 23:35:01 2013 +0000
tree 6428e56f7dd1460b42dd50473ef672aa5c30e4d8
parent 810ff69609c6f6e0a562c8f05912d4d785fd8bd2
parent d431b8d7f1396fe984b946f55e480ead99219484

Merge "Fix fstab memory leak"

fs_mgr/fs_mgr.c

diff --git a/fs_mgr/fs_mgr.c b/fs_mgr/fs_mgr.c
index 85ce397..4f11fbb 100644
--- a/fs_mgr/fs_mgr.c
+++ b/fs_mgr/fs_mgr.c
@@ -222,7 +222,7 @@
     char *line = NULL;
     const char *delim = " \t";
     char *save_ptr, *p;
-    struct fstab *fstab;
+    struct fstab *fstab = NULL;
     struct fstab_rec *recs;
     char *key_loc;
     long long part_length;
@@ -345,7 +345,10 @@
     return fstab;
 
 err:
+    fclose(fstab_file);
     free(line);
+    if (fstab)
+        fs_mgr_free_fstab(fstab);
     return NULL;
 }
 
@@ -361,7 +364,6 @@
         free(fstab->recs[i].fs_options);
         free(fstab->recs[i].key_loc);
         free(fstab->recs[i].label);
-        i++;
     }
 
     /* Free the fstab_recs array created by calloc(3) */