Blame SOURCES/autofs-5.1.7-check-for-offset-with-no-mount-location.patch

9a499a
autofs-5.1.7 - check for offset with no mount location
9a499a
9a499a
From: Ian Kent <raven@themaw.net>
9a499a
9a499a
Offsets need to have a mount location, check for it.
9a499a
9a499a
Signed-off-by: Ian Kent <raven@themaw.net>
9a499a
---
9a499a
 CHANGELOG           |    1 +
9a499a
 modules/parse_sun.c |   15 ++++++++++++++-
9a499a
 2 files changed, 15 insertions(+), 1 deletion(-)
9a499a
9a499a
--- autofs-5.1.4.orig/CHANGELOG
9a499a
+++ autofs-5.1.4/CHANGELOG
9a499a
@@ -47,6 +47,7 @@
9a499a
 - pass root length to mount_fullpath().
9a499a
 - remove unused function master_submount_list_empty().
9a499a
 - move amd mounts removal into lib/mounts.c.
9a499a
+- check for offset with no mount location.
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
@@ -803,7 +803,20 @@ update_offset_entry(struct autofs_point
9a499a
 
9a499a
 	memset(m_mapent, 0, MAPENT_MAX_LEN + 1);
9a499a
 
9a499a
-	/* Internal hosts map may have loc == NULL */
9a499a
+	if (!loc || !*loc) {
9a499a
+		const char *type = ap->entry->maps->type;
9a499a
+
9a499a
+		/* If it's not the internal hosts map it must have a
9a499a
+		 * mount location.
9a499a
+		 */
9a499a
+		if (!type || strcmp(type, "hosts")) {
9a499a
+			error(ap->logopt,
9a499a
+			      MODPREFIX "syntax error in offset %s -> %s",
9a499a
+			      m_offset, loc);
9a499a
+			return CHE_FAIL;
9a499a
+		}
9a499a
+	}
9a499a
+
9a499a
 	if (!*m_offset) {
9a499a
 		error(ap->logopt,
9a499a
 		      MODPREFIX "syntax error in offset %s -> %s", m_offset, loc);