Blame SOURCES/autofs-5.1.3-revert-fix-argc-off-by-one-in-mount_autofs_c.patch

306fa1
autofs-5.1.3 - revert fix argc off by one in mount_autofs.c
306fa1
306fa1
From: Ian Kent <raven@themaw.net>
306fa1
306fa1
Commit 6a44f715cf to fix an off by one error in the arguments calculation
306fa1
when mounting an autofs submount actually introduced an off by one error
306fa1
instead of fixing one.
306fa1
306fa1
Revert the change.
306fa1
306fa1
Signed-off-by: Ian Kent <raven@themaw.net>
306fa1
---
306fa1
 CHANGELOG              |    1 +
306fa1
 modules/mount_autofs.c |    8 ++++----
306fa1
 2 files changed, 5 insertions(+), 4 deletions(-)
306fa1
306fa1
--- autofs-5.0.7.orig/CHANGELOG
306fa1
+++ autofs-5.0.7/CHANGELOG
306fa1
@@ -256,6 +256,7 @@
306fa1
 - fix typos in autofs man pages.
306fa1
 - fix some man page problems.
306fa1
 - allow dot in OPTIONSTR value lexer pattern.
306fa1
+- revert fix argc off by one in mount_autofs.c.
306fa1
 
306fa1
 25/07/2012 autofs-5.0.7
306fa1
 =======================
306fa1
--- autofs-5.0.7.orig/modules/mount_autofs.c
306fa1
+++ autofs-5.0.7/modules/mount_autofs.c
306fa1
@@ -181,11 +181,11 @@ int mount_mount(struct autofs_point *ap,
306fa1
 
306fa1
 	if (options) {
306fa1
 		char *t = options;
306fa1
-		while ((t = strchr(t, ',')) != NULL) {
306fa1
+		do {
306fa1
 			argc++;
306fa1
 			if (*t == ',')
306fa1
 				t++;
306fa1
-		}
306fa1
+		} while ((t = strchr(t, ',')) != NULL);
306fa1
 	}
306fa1
 	argv = (const char **) alloca((argc + 1) * sizeof(char *));
306fa1
 
306fa1
@@ -213,13 +213,13 @@ int mount_mount(struct autofs_point *ap,
306fa1
 
306fa1
 	if (options) {
306fa1
 		p = options;
306fa1
-		while ((p = strchr(p, ',')) != NULL) {
306fa1
+		do {
306fa1
 			if (*p == ',') {
306fa1
 				*p = '\0';
306fa1
 				p++;
306fa1
 			}
306fa1
 			argv[argc++] = p;
306fa1
-		}
306fa1
+		} while ((p = strchr(p, ',')) != NULL);
306fa1
 	}
306fa1
 	argv[argc] = NULL;
306fa1