Blame SOURCES/autofs-5.1.0-fix-memory-leak-in-get_defaults_entry.patch

306fa1
autofs-5.1.0 - fix memory leak in get_defaults_entry()
306fa1
306fa1
From: Ian Kent <ikent@redhat.com>
306fa1
306fa1
In get_defaults_entry(), if parsing of the expanded defaults string fails
306fa1
the expanded string used for the parse is not freed.
306fa1
---
306fa1
 CHANGELOG           |    1 +
306fa1
 modules/parse_amd.c |    4 +++-
306fa1
 2 files changed, 4 insertions(+), 1 deletion(-)
306fa1
306fa1
--- autofs-5.0.7.orig/CHANGELOG
306fa1
+++ autofs-5.0.7/CHANGELOG
306fa1
@@ -150,6 +150,7 @@
306fa1
 - fix hosts map options check in lookup_amd_instance().
306fa1
 - fix memory leak in create_client().
306fa1
 - fix memory leak in get_exports().
306fa1
+- fix memory leak in get_defaults_entry().
306fa1
 
306fa1
 25/07/2012 autofs-5.0.7
306fa1
 =======================
306fa1
--- autofs-5.0.7.orig/modules/parse_amd.c
306fa1
+++ autofs-5.0.7/modules/parse_amd.c
306fa1
@@ -1797,8 +1797,10 @@ static struct amd_entry *get_defaults_en
306fa1
 		char *expand;
306fa1
 		if (!expand_selectors(ap, defaults, &expand, sv))
306fa1
 			goto out;
306fa1
-		if (amd_parse_list(ap, expand, &dflts, &sv))
306fa1
+		if (amd_parse_list(ap, expand, &dflts, &sv)) {
306fa1
+			free(expand);
306fa1
 			goto out;
306fa1
+		}
306fa1
 		entry = select_default_entry(ap, &dflts, sv);
306fa1
 		if (!entry->map_type) {
306fa1
 			/*