[autofs/f16] - update fix-improve-mount-location-error-reporting patch to current patch.

Ian Kent iankent at fedoraproject.org
Sun Jan 22 23:38:31 UTC 2012


commit abe8e4b83856ded2dd8561fe81b05a1a665bcdc4
Author: Ian Kent <raven at themaw.net>
Date:   Mon Jan 23 07:37:16 2012 +0800

    - update fix-improve-mount-location-error-reporting patch to current patch.

 ....0.6-fix-fix-wait-for-master-source-mutex.patch |   10 +++-----
 ...ix-improve-mount-location-error-reporting.patch |   23 +++++++++----------
 2 files changed, 15 insertions(+), 18 deletions(-)
---
diff --git a/autofs-5.0.6-fix-fix-wait-for-master-source-mutex.patch b/autofs-5.0.6-fix-fix-wait-for-master-source-mutex.patch
index 9e6d74c..5d0036d 100644
--- a/autofs-5.0.6-fix-fix-wait-for-master-source-mutex.patch
+++ b/autofs-5.0.6-fix-fix-wait-for-master-source-mutex.patch
@@ -12,14 +12,12 @@ avoid false positive fails.
 ---
 
  lib/master.c |    2 +-
- 1 files changed, 1 insertions(+), 1 deletions(-)
+ 1 file changed, 1 insertion(+), 1 deletion(-)
 
 
-diff --git a/lib/master.c b/lib/master.c
-index 87d1269..aad9087 100644
---- a/lib/master.c
-+++ b/lib/master.c
-@@ -552,7 +552,7 @@ void master_source_writelock(struct master_mapent *entry)
+--- autofs-5.0.6.orig/lib/master.c
++++ autofs-5.0.6/lib/master.c
+@@ -552,7 +552,7 @@ void master_source_writelock(struct mast
  
  void master_source_readlock(struct master_mapent *entry)
  {
diff --git a/autofs-5.0.6-fix-improve-mount-location-error-reporting.patch b/autofs-5.0.6-fix-improve-mount-location-error-reporting.patch
index 4e58b9e..22ac8fc 100644
--- a/autofs-5.0.6-fix-improve-mount-location-error-reporting.patch
+++ b/autofs-5.0.6-fix-improve-mount-location-error-reporting.patch
@@ -5,13 +5,13 @@ From: Ian Kent <raven at themaw.net>
 The validate_location() function is meant to check for a small subset
 of map location errors only but the "improve mount location error
 reporting" patch inverted a logic test which has made the scope of
-the testing greater causing false positive fails. So add a check for
-those special cases.
+the test greater causing false positive fails. So add a check for
+those special cases and return success instead.
 ---
 
  CHANGELOG           |    1 +
- modules/parse_sun.c |   15 +++++++++++++++
- 2 files changed, 16 insertions(+)
+ modules/parse_sun.c |   14 ++++++++++++++
+ 2 files changed, 15 insertions(+)
 
 
 --- autofs-5.0.6.orig/CHANGELOG
@@ -26,10 +26,11 @@ those special cases.
  -----------------------
 --- autofs-5.0.6.orig/modules/parse_sun.c
 +++ autofs-5.0.6/modules/parse_sun.c
-@@ -868,6 +868,21 @@ static int validate_location(unsigned in
+@@ -868,6 +868,20 @@ static int validate_location(unsigned in
  	 * have ":", "[" and "]".
  	 */
  	if (!check_colon(ptr)) {
++		char *esc;
 +		/*
 +		 * Don't forget cases where a colon is present but
 +		 * not followed by a "/" or, if there is no colon at
@@ -37,13 +38,11 @@ those special cases.
 +		 * it may be a map name by itself, for example.
 +		 */
 +		if (!strchr(ptr, ':') ||
-+		    !strncmp(name, "file:", 5) ||
-+		    !strncmp(name, "yp:", 3) ||
-+		    !strncmp(name, "nis:", 4) ||
-+		    !strncmp(name, "nisplus:", 8) ||
-+		    !strncmp(name, "ldap:", 5) ||
-+		    !strncmp(name, "ldaps:", 6) ||
-+		    !strncmp(name, "dir:", 4))
++		    ((esc = strchr(ptr, '\\')) && *(esc + 1) == ':') ||
++		    !strncmp(ptr, "file:", 5) || !strncmp(ptr, "yp:", 3) ||
++		    !strncmp(ptr, "nis:", 4) || !strncmp(ptr, "nisplus:", 8) ||
++		    !strncmp(ptr, "ldap:", 5) || !strncmp(ptr, "ldaps:", 6) ||
++		    !strncmp(ptr, "dir:", 4))
 +			return 1;
  		error(logopt,
  		      "expected colon delimeter not found in location %s",


More information about the scm-commits mailing list