Blame SOURCES/autofs-5.1.6-use-master_list_empty-for-list-empty-check.patch

9a499a
autofs-5.1.6 - use master_list_empty() for list empty check
9a499a
9a499a
From: Ian Kent <raven@themaw.net>
9a499a
9a499a
For consistency use the master_list_empty() function for list empty
9a499a
checks everywhere.
9a499a
9a499a
Signed-off-by: Ian Kent <raven@themaw.net>
9a499a
---
9a499a
 CHANGELOG          |    1 +
9a499a
 daemon/automount.c |    2 +-
9a499a
 daemon/master.c    |   24 +++++++++++++++++-------
9a499a
 include/master.h   |    1 +
9a499a
 4 files changed, 20 insertions(+), 8 deletions(-)
9a499a
9a499a
--- autofs-5.1.4.orig/CHANGELOG
9a499a
+++ autofs-5.1.4/CHANGELOG
9a499a
@@ -134,6 +134,7 @@ xx/xx/2018 autofs-5.1.5
9a499a
 - only add expre alarm for active mounts.
9a499a
 - move submount check into conditional_alarm_add().
9a499a
 - move lib/master.c to daemon/master.c.
9a499a
+- use master_list_empty() for list empty check.
9a499a
 
9a499a
 19/12/2017 autofs-5.1.4
9a499a
 - fix spec file url.
9a499a
--- autofs-5.1.4.orig/daemon/automount.c
9a499a
+++ autofs-5.1.4/daemon/automount.c
9a499a
@@ -1596,7 +1596,7 @@ static void *statemachine(void *arg)
9a499a
 		case SIGUSR2:
9a499a
 			master_mutex_lock();
9a499a
 			if (list_empty(&master_list->completed)) {
9a499a
-				if (list_empty(&master_list->mounts)) {
9a499a
+				if (__master_list_empty(master_list)) {
9a499a
 					master_mutex_unlock();
9a499a
 					return NULL;
9a499a
 				}
9a499a
--- autofs-5.1.4.orig/daemon/master.c
9a499a
+++ autofs-5.1.4/daemon/master.c
9a499a
@@ -1110,7 +1110,7 @@ int master_read_master(struct master *ma
9a499a
 			master_mount_mounts(master, age);
9a499a
 	}
9a499a
 
9a499a
-	if (list_empty(&master->mounts))
9a499a
+	if (__master_list_empty(master))
9a499a
 		warn(logopt, "no mounts in table");
9a499a
 
9a499a
 	master_mutex_unlock();
9a499a
@@ -1625,7 +1625,7 @@ int dump_map(struct master *master, cons
9a499a
 {
9a499a
 	struct list_head *p, *head;
9a499a
 
9a499a
-	if (list_empty(&master->mounts)) {
9a499a
+	if (__master_list_empty(master)) {
9a499a
 		printf("no master map entries found\n");
9a499a
 		return 1;
9a499a
 	}
9a499a
@@ -1743,7 +1743,7 @@ int master_show_mounts(struct master *ma
9a499a
 		printf("global options %s be appended to map entries\n", append);
9a499a
 	}
9a499a
 
9a499a
-	if (list_empty(&master->mounts)) {
9a499a
+	if (__master_list_empty(master)) {
9a499a
 		printf("no master map entries found\n\n");
9a499a
 		return 1;
9a499a
 	}
9a499a
@@ -1831,13 +1831,22 @@ int master_show_mounts(struct master *ma
9a499a
 	return 1;
9a499a
 }
9a499a
 
9a499a
-int master_list_empty(struct master *master)
9a499a
+int __master_list_empty(struct master *master)
9a499a
 {
9a499a
 	int res = 0;
9a499a
 
9a499a
-	master_mutex_lock();
9a499a
 	if (list_empty(&master->mounts))
9a499a
 		res = 1;
9a499a
+
9a499a
+	return res;
9a499a
+}
9a499a
+
9a499a
+int master_list_empty(struct master *master)
9a499a
+{
9a499a
+	int res;
9a499a
+
9a499a
+	master_mutex_lock();
9a499a
+	res = __master_list_empty(master);
9a499a
 	master_mutex_unlock();
9a499a
 
9a499a
 	return res;
9a499a
@@ -1859,7 +1868,8 @@ int master_done(struct master *master)
9a499a
 		master_free_mapent_sources(entry, 1);
9a499a
 		master_free_mapent(entry);
9a499a
 	}
9a499a
-	if (list_empty(&master->mounts))
9a499a
+
9a499a
+	if (__master_list_empty(master))
9a499a
 		res = 1;
9a499a
 
9a499a
 	return res;
9a499a
@@ -1872,7 +1882,7 @@ unsigned int master_get_logopt(void)
9a499a
 
9a499a
 int master_kill(struct master *master)
9a499a
 {
9a499a
-	if (!list_empty(&master->mounts))
9a499a
+	if (!master_list_empty(master))
9a499a
 		return 0;
9a499a
 
9a499a
 	if (master->name)
9a499a
--- autofs-5.1.4.orig/include/master.h
9a499a
+++ autofs-5.1.4/include/master.h
9a499a
@@ -123,6 +123,7 @@ int master_mount_mounts(struct master *,
9a499a
 int dump_map(struct master *, const char *, const char *);
9a499a
 int master_show_mounts(struct master *);
9a499a
 unsigned int master_get_logopt(void);
9a499a
+int __master_list_empty(struct master *);
9a499a
 int master_list_empty(struct master *);
9a499a
 int master_done(struct master *);
9a499a
 int master_kill(struct master *);