Ian Kent 0e5314
autofs-5.1.0 - fix out of order clearing of options buffer
Ian Kent 0e5314
Ian Kent 0e5314
From: Ian Kent <ikent@redhat.com>
Ian Kent 0e5314
Ian Kent 0e5314
When setting the result of parsing an options string the options buffer
Ian Kent 0e5314
was being cleared before use.
Ian Kent 0e5314
---
Ian Kent 0e5314
 CHANGELOG           |    1 +
Ian Kent 0e5314
 modules/amd_parse.y |    2 +-
Ian Kent 0e5314
 2 files changed, 2 insertions(+), 1 deletion(-)
Ian Kent 0e5314
Ian Kent 0e5314
diff --git a/CHANGELOG b/CHANGELOG
Ian Kent 0e5314
index 439a51f..51c935d 100644
Ian Kent 0e5314
--- a/CHANGELOG
Ian Kent 0e5314
+++ b/CHANGELOG
Ian Kent 0e5314
@@ -26,6 +26,7 @@
Ian Kent 0e5314
 - fix memory leak in create_client().
Ian Kent 0e5314
 - fix memory leak in get_exports().
Ian Kent 0e5314
 - fix memory leak in get_defaults_entry().
Ian Kent 0e5314
+- fix out of order clearing of options buffer.
Ian Kent 0e5314
 
Ian Kent 0e5314
 04/06/2014 autofs-5.1.0
Ian Kent 0e5314
 =======================
Ian Kent 0e5314
diff --git a/modules/amd_parse.y b/modules/amd_parse.y
Ian Kent 0e5314
index f65ab7a..a7f38f8 100644
Ian Kent 0e5314
--- a/modules/amd_parse.y
Ian Kent 0e5314
+++ b/modules/amd_parse.y
Ian Kent 0e5314
@@ -396,7 +396,6 @@ option_assignment: MAP_OPTION OPTION_ASSIGN FS_TYPE
Ian Kent 0e5314
 	}
Ian Kent 0e5314
 	| MNT_OPTION OPTION_ASSIGN options
Ian Kent 0e5314
 	{
Ian Kent 0e5314
-		memset(opts, 0, sizeof(opts));
Ian Kent 0e5314
 		if (!strcmp($1, "opts"))
Ian Kent 0e5314
 			entry.opts = amd_strdup(opts);
Ian Kent 0e5314
 		else if (!strcmp($1, "addopts"))
Ian Kent 0e5314
@@ -407,6 +406,7 @@ option_assignment: MAP_OPTION OPTION_ASSIGN FS_TYPE
Ian Kent 0e5314
 			amd_notify($1);
Ian Kent 0e5314
 			YYABORT;
Ian Kent 0e5314
 		}
Ian Kent 0e5314
+		memset(opts, 0, sizeof(opts));
Ian Kent 0e5314
 	}
Ian Kent 0e5314
 	| MNT_OPTION OPTION_ASSIGN
Ian Kent 0e5314
 	{