Blame SOURCES/autofs-5.1.7-fix-double-free-in-parse_mapent.patch

9a499a
autofs-5.1.7 - fix double free in parse_mapent()
9a499a
9a499a
From: Ian Kent <raven@themaw.net>
9a499a
9a499a
Coverity:
9a499a
in parse_mapent(): double_free: Calling "free" frees pointer "newopt"
9a499a
		   which has already been freed.
9a499a
9a499a
Signed-off-by: Ian Kent <raven@themaw.net>
9a499a
---
9a499a
 CHANGELOG           |    1 +
9a499a
 modules/parse_sun.c |    2 --
9a499a
 2 files changed, 1 insertion(+), 2 deletions(-)
9a499a
9a499a
--- autofs-5.1.4.orig/CHANGELOG
9a499a
+++ autofs-5.1.4/CHANGELOG
9a499a
@@ -62,6 +62,7 @@
9a499a
 - fix dead code in mnts_add_mount().
9a499a
 - fix arg not used in error print.
9a499a
 - fix missing lock release in mount_subtree().
9a499a
+- fix double free in parse_mapent().
9a499a
 
9a499a
 xx/xx/2018 autofs-5.1.5
9a499a
 - fix flag file permission.
9a499a
--- autofs-5.1.4.orig/modules/parse_sun.c
9a499a
+++ autofs-5.1.4/modules/parse_sun.c
9a499a
@@ -976,8 +976,6 @@ static int parse_mapent(const char *ent,
9a499a
 					estr = strerror_r(errno, buf, MAX_ERR_BUF);
9a499a
 					error(logopt, MODPREFIX
9a499a
 					      "concat_options: %s", estr);
9a499a
-					if (newopt)
9a499a
-						free(newopt);
9a499a
 					free(myoptions);
9a499a
 					return 0;
9a499a
 				}