Blame SOURCES/autofs-5.1.7-rename-path-to-m_offset-in-update_offset_entry.patch

9a499a
autofs-5.1.7 - rename path to m_offset in update_offset_entry()
9a499a
9a499a
From: Ian Kent <raven@themaw.net>
9a499a
9a499a
Rename local variable from path to m_offset in update_offset_entry() to
9a499a
make the meaning of this variable clear.
9a499a
9a499a
Signed-off-by: Ian Kent <raven@themaw.net>
9a499a
---
9a499a
 CHANGELOG           |    1 +
9a499a
 modules/parse_sun.c |   46 +++++++++++++++++++++++-----------------------
9a499a
 2 files changed, 24 insertions(+), 23 deletions(-)
9a499a
9a499a
--- autofs-5.1.4.orig/CHANGELOG
9a499a
+++ autofs-5.1.4/CHANGELOG
9a499a
@@ -24,6 +24,7 @@
9a499a
 - don't add offset mounts to mounted mounts table.
9a499a
 - reduce umount EBUSY check delay.
9a499a
 - cleanup cache_delete() a little.
9a499a
+- rename path to m_offset in update_offset_entry().
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
@@ -798,36 +798,36 @@ static int
9a499a
 update_offset_entry(struct autofs_point *ap,
9a499a
 		    struct mapent_cache *mc, const char *name,
9a499a
 		    const char *m_root, int m_root_len,
9a499a
-		    const char *path, const char *myoptions,
9a499a
+		    const char *m_offset, const char *myoptions,
9a499a
 		    const char *loc, time_t age)
9a499a
 {
9a499a
 	char m_key[PATH_MAX + 1];
9a499a
 	char m_mapent[MAPENT_MAX_LEN + 1];
9a499a
-	int p_len, m_key_len, m_options_len, m_mapent_len;
9a499a
+	int o_len, m_key_len, m_options_len, m_mapent_len;
9a499a
 	int ret;
9a499a
 
9a499a
 	memset(m_mapent, 0, MAPENT_MAX_LEN + 1);
9a499a
 
9a499a
 	/* Internal hosts map may have loc == NULL */
9a499a
-	if (!*path) {
9a499a
+	if (!*m_offset) {
9a499a
 		error(ap->logopt,
9a499a
-		      MODPREFIX "syntax error in offset %s -> %s", path, loc);
9a499a
+		      MODPREFIX "syntax error in offset %s -> %s", m_offset, loc);
9a499a
 		return CHE_FAIL;
9a499a
 	}
9a499a
 
9a499a
-	p_len = strlen(path);
9a499a
+	o_len = strlen(m_offset);
9a499a
 	/* Trailing '/' causes us pain */
9a499a
-	if (p_len > 1) {
9a499a
-		while (p_len > 1 && path[p_len - 1] == '/')
9a499a
-			p_len--;
9a499a
+	if (o_len > 1) {
9a499a
+		while (o_len > 1 && m_offset[o_len - 1] == '/')
9a499a
+			o_len--;
9a499a
 	}
9a499a
-	m_key_len = m_root_len + p_len;
9a499a
+	m_key_len = m_root_len + o_len;
9a499a
 	if (m_key_len > PATH_MAX) {
9a499a
 		error(ap->logopt, MODPREFIX "multi mount key too long");
9a499a
 		return CHE_FAIL;
9a499a
 	}
9a499a
 	strcpy(m_key, m_root);
9a499a
-	strncat(m_key, path, p_len);
9a499a
+	strncat(m_key, m_offset, o_len);
9a499a
 	m_key[m_key_len] = '\0';
9a499a
 
9a499a
 	m_options_len = 0;
9a499a
@@ -862,15 +862,15 @@ update_offset_entry(struct autofs_point
9a499a
 
9a499a
 	if (ret == CHE_DUPLICATE) {
9a499a
 		warn(ap->logopt, MODPREFIX
9a499a
-		     "syntax error or duplicate offset %s -> %s", path, loc);
9a499a
+		     "syntax error or duplicate offset %s -> %s", m_offset, loc);
9a499a
 		ret = CHE_OK;
9a499a
 	} else if (ret == CHE_FAIL)
9a499a
 		debug(ap->logopt, MODPREFIX
9a499a
-		      "failed to update multi-mount offset %s -> %s", path, m_mapent);
9a499a
+		      "failed to update multi-mount offset %s -> %s", m_offset, m_mapent);
9a499a
 	else {
9a499a
 		ret = CHE_OK;
9a499a
 		debug(ap->logopt, MODPREFIX
9a499a
-		      "updated multi-mount offset %s -> %s", path, m_mapent);
9a499a
+		      "updated multi-mount offset %s -> %s", m_offset, m_mapent);
9a499a
 	}
9a499a
 
9a499a
 	return ret;
9a499a
@@ -1540,22 +1540,22 @@ dont_expand:
9a499a
 
9a499a
 		/* It's a multi-mount; deal with it */
9a499a
 		do {
9a499a
-			char *path, *myoptions, *loc;
9a499a
+			char *m_offset, *myoptions, *loc;
9a499a
 			int status;
9a499a
 
9a499a
 			if ((*p == '"' && *(p + 1) != '/') || (*p != '"' && *p != '/')) {
9a499a
 				l = 0;
9a499a
-				path = dequote("/", 1, ap->logopt);
9a499a
+				m_offset = dequote("/", 1, ap->logopt);
9a499a
 				debug(ap->logopt,
9a499a
-				      MODPREFIX "dequote(\"/\") -> %s", path);
9a499a
+				      MODPREFIX "dequote(\"/\") -> %s", m_offset);
9a499a
 			} else {
9a499a
 				l = span_space(p, mapent_len - (p - pmapent));
9a499a
-				path = sanitize_path(p, l, LKP_MULTI, ap->logopt);
9a499a
+				m_offset = sanitize_path(p, l, LKP_MULTI, ap->logopt);
9a499a
 				debug(ap->logopt, MODPREFIX
9a499a
-				      "dequote(\"%.*s\") -> %s", l, p, path);
9a499a
+				      "dequote(\"%.*s\") -> %s", l, p, m_offset);
9a499a
 			}
9a499a
 
9a499a
-			if (!path) {
9a499a
+			if (!m_offset) {
9a499a
 				warn(ap->logopt, MODPREFIX "null path or out of memory");
9a499a
 				cache_writelock(mc);
9a499a
 				cache_delete_offset_list(mc, name);
9a499a
@@ -1577,7 +1577,7 @@ dont_expand:
9a499a
 				cache_writelock(mc);
9a499a
 				cache_delete_offset_list(mc, name);
9a499a
 				cache_unlock(mc);
9a499a
-				free(path);
9a499a
+				free(m_offset);
9a499a
 				free(options);
9a499a
 				free(pmapent);
9a499a
 				pthread_setcancelstate(cur_state, NULL);
9a499a
@@ -1589,14 +1589,14 @@ dont_expand:
9a499a
 
9a499a
 			status = update_offset_entry(ap, mc,
9a499a
 						     name, m_root, m_root_len,
9a499a
-						     path, myoptions, loc, age);
9a499a
+						     m_offset, myoptions, loc, age);
9a499a
 
9a499a
 			if (status != CHE_OK) {
9a499a
 				warn(ap->logopt, MODPREFIX "error adding multi-mount");
9a499a
 				cache_writelock(mc);
9a499a
 				cache_delete_offset_list(mc, name);
9a499a
 				cache_unlock(mc);
9a499a
-				free(path);
9a499a
+				free(m_offset);
9a499a
 				free(options);
9a499a
 				free(pmapent);
9a499a
 				free(myoptions);
9a499a
@@ -1608,7 +1608,7 @@ dont_expand:
9a499a
 
9a499a
 			if (loc)
9a499a
 				free(loc);
9a499a
-			free(path);
9a499a
+			free(m_offset);
9a499a
 			free(myoptions);
9a499a
 		} while (*p == '/' || (*p == '"' && *(p + 1) == '/'));
9a499a