commit af781c2692c20c954f650505faa1d2ba36b643f6
author Elliott Hughes <enh@google.com> Wed Sep 18 16:42:12 2013 -0700
committer Android Git Automerger <android-git-automerger@android.com> Wed Sep 18 16:42:12 2013 -0700
tree dfc0e083879f6074f7bd0db238983f1a70db9c02
parent 7fdd7913bd54adcc7ef911d3ed9697bf142c1af9
parent a2a93250f625b776886d3c5d41599abdaddb2e50

am a2a93250: am c67bd8d7: am a208ea63: Merge "Fix fstab memory leak"

* commit 'a2a93250f625b776886d3c5d41599abdaddb2e50':
  Fix fstab memory leak

fs_mgr/fs_mgr.c

diff --git a/fs_mgr/fs_mgr.c b/fs_mgr/fs_mgr.c
index dabbc27..9a84a4e 100644
--- a/fs_mgr/fs_mgr.c
+++ b/fs_mgr/fs_mgr.c
@@ -244,7 +244,7 @@
     char *line = NULL;
     const char *delim = " \t";
     char *save_ptr, *p;
-    struct fstab *fstab;
+    struct fstab *fstab = NULL;
     struct fstab_rec *recs;
     struct fs_mgr_flag_values flag_vals;
 #define FS_OPTIONS_LEN 1024
@@ -363,7 +363,10 @@
     return fstab;
 
 err:
+    fclose(fstab_file);
     free(line);
+    if (fstab)
+        fs_mgr_free_fstab(fstab);
     return NULL;
 }
 
@@ -379,7 +382,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) */