Blame SOURCES/autofs-5.1.7-fix-possible-memory-leak-in-master_parse.patch

96dc52
autofs-5.1.7 - fix possible memory leak in master_parse()
96dc52
96dc52
From: Ian Kent <raven@themaw.net>
96dc52
96dc52
Coverity: Overwriting "path" in "path = master_strdup(yyvsp[-1].strtype)"
96dc52
	  leaks the storage that "path" points to.
96dc52
96dc52
Signed-off-by: Ian Kent <raven@themaw.net>
96dc52
---
96dc52
 CHANGELOG             |    1 +
96dc52
 daemon/master_parse.y |    2 ++
96dc52
 2 files changed, 3 insertions(+)
96dc52
96dc52
diff --git a/CHANGELOG b/CHANGELOG
96dc52
index 2186cbe3..b797f6dc 100644
96dc52
--- a/CHANGELOG
96dc52
+++ b/CHANGELOG
96dc52
@@ -52,6 +52,7 @@
96dc52
 - remove unused variable from get_exports().
96dc52
 - add missing free in handle_mounts().
96dc52
 - remove redundant if check.
96dc52
+- fix possible memory leak in master_parse().
96dc52
 
96dc52
 25/01/2021 autofs-5.1.7
96dc52
 - make bind mounts propagation slave by default.
96dc52
diff --git a/daemon/master_parse.y b/daemon/master_parse.y
96dc52
index 08e44b57..7480c36a 100644
96dc52
--- a/daemon/master_parse.y
96dc52
+++ b/daemon/master_parse.y
96dc52
@@ -155,6 +155,8 @@ file: {
96dc52
 line:
96dc52
 	| PATH mapspec
96dc52
 	{
96dc52
+		if (path)
96dc52
+			free(path);
96dc52
 		path = master_strdup($1);
96dc52
 		if (!path) {
96dc52
 			local_free_vars();