Blame SOURCES/autofs-5.1.6-remove-unused-function-tree_get_mnt_list.patch

9a499a
autofs-5.1.6 - remove unused function tree_get_mnt_list()
9a499a
9a499a
From: Ian Kent <raven@themaw.net>
9a499a
9a499a
Remove the now unused function tree_get_mnt_list() and friends.
9a499a
9a499a
Signed-off-by: Ian Kent <raven@themaw.net>
9a499a
---
9a499a
 CHANGELOG        |    1 
9a499a
 include/mounts.h |   17 ---
9a499a
 lib/mounts.c     |  299 -------------------------------------------------------
9a499a
 3 files changed, 1 insertion(+), 316 deletions(-)
9a499a
9a499a
--- autofs-5.1.4.orig/CHANGELOG
9a499a
+++ autofs-5.1.4/CHANGELOG
9a499a
@@ -130,6 +130,7 @@ xx/xx/2018 autofs-5.1.5
9a499a
 - fix stale offset directories disable mount.
9a499a
 - use struct mnt_list to track mounted mounts.
9a499a
 - use struct mnt_list mounted list for expire.
9a499a
+- remove unused function tree_get_mnt_list().
9a499a
 
9a499a
 19/12/2017 autofs-5.1.4
9a499a
 - fix spec file url.
9a499a
--- autofs-5.1.4.orig/include/mounts.h
9a499a
+++ autofs-5.1.4/include/mounts.h
9a499a
@@ -83,20 +83,8 @@ struct mnt_list {
9a499a
 	 * List operations ie. get_mnt_list.
9a499a
 	 */
9a499a
 	struct mnt_list *next;
9a499a
-
9a499a
-	/*
9a499a
-	 * Tree operations ie. tree_make_tree,
9a499a
-	 * tree_get_mnt_list etc.
9a499a
-	 */
9a499a
-	struct mnt_list *left;
9a499a
-	struct mnt_list *right;
9a499a
-	struct list_head self;
9a499a
-	struct list_head list;
9a499a
-	struct list_head entries;
9a499a
-	struct list_head sublist;
9a499a
 };
9a499a
 
9a499a
-
9a499a
 struct nfs_mount_vers {
9a499a
 	unsigned int major;
9a499a
 	unsigned int minor;
9a499a
@@ -143,11 +131,6 @@ void mnts_set_mounted_mount(struct autof
9a499a
 int unlink_mount_tree(struct autofs_point *ap, const char *mp);
9a499a
 void free_mnt_list(struct mnt_list *list);
9a499a
 int is_mounted(const char *mp, unsigned int type);
9a499a
-void tree_free_mnt_tree(struct mnt_list *tree);
9a499a
-struct mnt_list *tree_make_mnt_tree(const char *path);
9a499a
-int tree_get_mnt_list(struct mnt_list *mnts, struct list_head *list, const char *path, int include);
9a499a
-int tree_get_mnt_sublist(struct mnt_list *mnts, struct list_head *list, const char *path, int include);
9a499a
-int tree_find_mnt_ents(struct mnt_list *mnts, struct list_head *list, const char *path);
9a499a
 void set_tsd_user_vars(unsigned int, uid_t, gid_t);
9a499a
 const char *mount_type_str(unsigned int);
9a499a
 void set_exp_timeout(struct autofs_point *ap, struct map_source *source, time_t timeout);
9a499a
--- autofs-5.1.4.orig/lib/mounts.c
9a499a
+++ autofs-5.1.4/lib/mounts.c
9a499a
@@ -1685,305 +1685,6 @@ int is_mounted(const char *mp, unsigned
9a499a
 		return table_is_mounted(mp, type);
9a499a
 }
9a499a
 
9a499a
-/*
9a499a
- * Since we have to look at the entire mount tree for direct
9a499a
- * mounts (all mounts under "/") and we may have a large number
9a499a
- * of entries to traverse again and again we need to
9a499a
- * use a more efficient method than the routines above.
9a499a
- *
9a499a
- * Thre tree_... routines allow us to read the mount tree
9a499a
- * once and pass it to subsequent functions for use. Since
9a499a
- * it's a tree structure searching should be a low overhead
9a499a
- * operation.
9a499a
- */
9a499a
-void tree_free_mnt_tree(struct mnt_list *tree)
9a499a
-{
9a499a
-	struct list_head *head, *p;
9a499a
-
9a499a
-	if (!tree)
9a499a
-		return;
9a499a
-
9a499a
-	tree_free_mnt_tree(tree->left);
9a499a
-	tree_free_mnt_tree(tree->right);
9a499a
-
9a499a
-	head = &tree->self;
9a499a
-	p = head->next;
9a499a
-	while (p != head) {
9a499a
-		struct mnt_list *this;
9a499a
-
9a499a
-		this = list_entry(p, struct mnt_list, self);
9a499a
-
9a499a
-		p = p->next;
9a499a
-
9a499a
-		list_del(&this->self);
9a499a
-
9a499a
-		free(this->mp);
9a499a
-
9a499a
-		free(this);
9a499a
-	}
9a499a
-
9a499a
-	free(tree->mp);
9a499a
-	free(tree);
9a499a
-}
9a499a
-
9a499a
-/*
9a499a
- * Make tree of system mounts in /proc/mounts.
9a499a
- */
9a499a
-struct mnt_list *tree_make_mnt_tree(const char *path)
9a499a
-{
9a499a
-	FILE *tab;
9a499a
-	struct mntent mnt_wrk;
9a499a
-	char buf[PATH_MAX * 3];
9a499a
-	struct mntent *mnt;
9a499a
-	struct mnt_list *ent, *mptr;
9a499a
-	struct mnt_list *tree = NULL;
9a499a
-	size_t plen;
9a499a
-	int eq;
9a499a
-
9a499a
-	tab = open_fopen_r(_PROC_MOUNTS);
9a499a
-	if (!tab) {
9a499a
-		char *estr = strerror_r(errno, buf, PATH_MAX - 1);
9a499a
-		logerr("fopen: %s", estr);
9a499a
-		return NULL;
9a499a
-	}
9a499a
-
9a499a
-	plen = strlen(path);
9a499a
-
9a499a
-	while ((mnt = local_getmntent_r(tab, &mnt_wrk, buf, PATH_MAX * 3))) {
9a499a
-		size_t len = strlen(mnt->mnt_dir);
9a499a
-
9a499a
-		/* Not matching path */
9a499a
-		if (strncmp(mnt->mnt_dir, path, plen))
9a499a
-			continue;
9a499a
-
9a499a
-		/* Not a subdirectory of requested path */
9a499a
-		if (plen > 1 && len > plen && mnt->mnt_dir[plen] != '/')
9a499a
-			continue;
9a499a
-
9a499a
-		ent = malloc(sizeof(*ent));
9a499a
-		if (!ent) {
9a499a
-			endmntent(tab);
9a499a
-			tree_free_mnt_tree(tree);
9a499a
-			return NULL;
9a499a
-		}
9a499a
-		memset(ent, 0, sizeof(*ent));
9a499a
-
9a499a
-		INIT_LIST_HEAD(&ent->self);
9a499a
-		INIT_LIST_HEAD(&ent->list);
9a499a
-		INIT_LIST_HEAD(&ent->entries);
9a499a
-		INIT_LIST_HEAD(&ent->sublist);
9a499a
-
9a499a
-		ent->mp = malloc(len + 1);
9a499a
-		if (!ent->mp) {
9a499a
-			endmntent(tab);
9a499a
-			free(ent);
9a499a
-			tree_free_mnt_tree(tree);
9a499a
-			return NULL;
9a499a
-		}
9a499a
-		strcpy(ent->mp, mnt->mnt_dir);
9a499a
-
9a499a
-		if (!strcmp(mnt->mnt_type, "autofs"))
9a499a
-			ent->flags |= MNTS_AUTOFS;
9a499a
-
9a499a
-		if (ent->flags & MNTS_AUTOFS) {
9a499a
-			if (strstr(mnt->mnt_opts, "indirect"))
9a499a
-				ent->flags |= MNTS_INDIRECT;
9a499a
-			else if (strstr(mnt->mnt_opts, "direct"))
9a499a
-				ent->flags |= MNTS_DIRECT;
9a499a
-			else if (strstr(mnt->mnt_opts, "offset"))
9a499a
-				ent->flags |= MNTS_OFFSET;
9a499a
-		}
9a499a
-
9a499a
-		mptr = tree;
9a499a
-		while (mptr) {
9a499a
-			int elen = strlen(ent->mp);
9a499a
-			int mlen = strlen(mptr->mp);
9a499a
-
9a499a
-			if (elen < mlen) {
9a499a
-				if (mptr->left) {
9a499a
-					mptr = mptr->left;
9a499a
-					continue;
9a499a
-				} else {
9a499a
-					mptr->left = ent;
9a499a
-					break;
9a499a
-				}
9a499a
-			} else if (elen > mlen) {
9a499a
-				if (mptr->right) {
9a499a
-					mptr = mptr->right;
9a499a
-					continue;
9a499a
-				} else {
9a499a
-					mptr->right = ent;
9a499a
-					break;
9a499a
-				}
9a499a
-			}
9a499a
-
9a499a
-			eq = strcmp(ent->mp, mptr->mp);
9a499a
-			if (eq < 0) {
9a499a
-				if (mptr->left)
9a499a
-					mptr = mptr->left;
9a499a
-				else {
9a499a
-					mptr->left = ent;
9a499a
-					break;
9a499a
-				}
9a499a
-			} else if (eq > 0) {
9a499a
-				if (mptr->right)
9a499a
-					mptr = mptr->right;
9a499a
-				else {
9a499a
-					mptr->right = ent;
9a499a
-					break;
9a499a
-				}
9a499a
-			} else {
9a499a
-				list_add_tail(&ent->self, &mptr->self);
9a499a
-				break;
9a499a
-			}
9a499a
-		}
9a499a
-
9a499a
-		if (!tree)
9a499a
-			tree = ent;
9a499a
-	}
9a499a
-	fclose(tab);
9a499a
-
9a499a
-	return tree;
9a499a
-}
9a499a
-
9a499a
-/*
9a499a
- * Get list of mounts under "path" in longest->shortest order
9a499a
- */
9a499a
-int tree_get_mnt_list(struct mnt_list *mnts, struct list_head *list, const char *path, int include)
9a499a
-{
9a499a
-	size_t mlen, plen;
9a499a
-
9a499a
-	if (!mnts)
9a499a
-		return 0;
9a499a
-
9a499a
-	plen = strlen(path);
9a499a
-	mlen = strlen(mnts->mp);
9a499a
-	if (mlen < plen)
9a499a
-		return tree_get_mnt_list(mnts->right, list, path, include);
9a499a
-	else {
9a499a
-		struct list_head *self, *p;
9a499a
-
9a499a
-		tree_get_mnt_list(mnts->left, list, path, include);
9a499a
-
9a499a
-		if ((!include && mlen <= plen) ||
9a499a
-				strncmp(mnts->mp, path, plen))
9a499a
-			goto skip;
9a499a
-
9a499a
-		if (plen > 1 && mlen > plen && mnts->mp[plen] != '/')
9a499a
-			goto skip;
9a499a
-
9a499a
-		INIT_LIST_HEAD(&mnts->list);
9a499a
-		list_add(&mnts->list, list);
9a499a
-
9a499a
-		self = &mnts->self;
9a499a
-		list_for_each(p, self) {
9a499a
-			struct mnt_list *this;
9a499a
-
9a499a
-			this = list_entry(p, struct mnt_list, self);
9a499a
-			INIT_LIST_HEAD(&this->list);
9a499a
-			list_add(&this->list, list);
9a499a
-		}
9a499a
-skip:
9a499a
-		tree_get_mnt_list(mnts->right, list, path, include);
9a499a
-	}
9a499a
-
9a499a
-	if (list_empty(list))
9a499a
-		return 0;
9a499a
-
9a499a
-	return 1;
9a499a
-}
9a499a
-
9a499a
-/*
9a499a
- * Get list of mounts under "path" in longest->shortest order
9a499a
- */
9a499a
-int tree_get_mnt_sublist(struct mnt_list *mnts, struct list_head *list, const char *path, int include)
9a499a
-{
9a499a
-	size_t mlen, plen;
9a499a
-
9a499a
-	if (!mnts)
9a499a
-		return 0;
9a499a
-
9a499a
-	plen = strlen(path);
9a499a
-	mlen = strlen(mnts->mp);
9a499a
-	if (mlen < plen)
9a499a
-		return tree_get_mnt_sublist(mnts->right, list, path, include);
9a499a
-	else {
9a499a
-		struct list_head *self, *p;
9a499a
-
9a499a
-		tree_get_mnt_sublist(mnts->left, list, path, include);
9a499a
-
9a499a
-		if ((!include && mlen <= plen) ||
9a499a
-				strncmp(mnts->mp, path, plen))
9a499a
-			goto skip;
9a499a
-
9a499a
-		if (plen > 1 && mlen > plen && mnts->mp[plen] != '/')
9a499a
-			goto skip;
9a499a
-
9a499a
-		INIT_LIST_HEAD(&mnts->sublist);
9a499a
-		list_add(&mnts->sublist, list);
9a499a
-
9a499a
-		self = &mnts->self;
9a499a
-		list_for_each(p, self) {
9a499a
-			struct mnt_list *this;
9a499a
-
9a499a
-			this = list_entry(p, struct mnt_list, self);
9a499a
-			INIT_LIST_HEAD(&this->sublist);
9a499a
-			list_add(&this->sublist, list);
9a499a
-		}
9a499a
-skip:
9a499a
-		tree_get_mnt_sublist(mnts->right, list, path, include);
9a499a
-	}
9a499a
-
9a499a
-	if (list_empty(list))
9a499a
-		return 0;
9a499a
-
9a499a
-	return 1;
9a499a
-}
9a499a
-
9a499a
-int tree_find_mnt_ents(struct mnt_list *mnts, struct list_head *list, const char *path)
9a499a
-{
9a499a
-	int mlen, plen;
9a499a
-
9a499a
-	if (!mnts)
9a499a
-		return 0;
9a499a
-
9a499a
-	plen = strlen(path);
9a499a
-	mlen = strlen(mnts->mp);
9a499a
-	if (mlen < plen)
9a499a
-		return tree_find_mnt_ents(mnts->right, list, path);
9a499a
-	else if (mlen > plen)
9a499a
-		return tree_find_mnt_ents(mnts->left, list, path);
9a499a
-	else {
9a499a
-		struct list_head *self, *p;
9a499a
-
9a499a
-		tree_find_mnt_ents(mnts->left, list, path);
9a499a
-
9a499a
-		if (!strcmp(mnts->mp, path)) {
9a499a
-			INIT_LIST_HEAD(&mnts->entries);
9a499a
-			list_add(&mnts->entries, list);
9a499a
-		}
9a499a
-
9a499a
-		self = &mnts->self;
9a499a
-		list_for_each(p, self) {
9a499a
-			struct mnt_list *this;
9a499a
-
9a499a
-			this = list_entry(p, struct mnt_list, self);
9a499a
-
9a499a
-			if (!strcmp(this->mp, path)) {
9a499a
-				INIT_LIST_HEAD(&this->entries);
9a499a
-				list_add(&this->entries, list);
9a499a
-			}
9a499a
-		}
9a499a
-
9a499a
-		tree_find_mnt_ents(mnts->right, list, path);
9a499a
-
9a499a
-		if (!list_empty(list))
9a499a
-			return 1;
9a499a
-	}
9a499a
-
9a499a
-	return 0;
9a499a
-}
9a499a
-
9a499a
 void set_tsd_user_vars(unsigned int logopt, uid_t uid, gid_t gid)
9a499a
 {
9a499a
 	struct thread_stdenv_vars *tsv;