53faf3
diff -upr libqb-1.0.1.orig/configure.ac libqb-1.0.1/configure.ac
53faf3
--- libqb-1.0.1.orig/configure.ac	2016-11-08 11:15:16.000000000 +0000
53faf3
+++ libqb-1.0.1/configure.ac	2017-05-15 10:47:39.042145452 +0100
53faf3
@@ -482,6 +482,11 @@ AC_ARG_WITH([socket-dir],
53faf3
 	[ SOCKETDIR="$withval" ],
53faf3
 	[ SOCKETDIR="$localstatedir/run" ])
53faf3
 
53faf3
+AC_ARG_WITH([force-sockets-config-file],
53faf3
+  [AS_HELP_STRING([--with-force-sockets-config-file=FILE],[config file to force IPC to use filesystem sockets (Linux & Cygwin only) @<:@SYSCONFDIR/libqb/force-filesystem-sockets@:>@])],
53faf3
+	[ FORCESOCKETSFILE="$withval" ],
53faf3
+	[ FORCESOCKETSFILE="$sysconfdir/libqb/force-filesystem-sockets" ])
53faf3
+
53faf3
 AC_SUBST(CP)
53faf3
 # *FLAGS handling goes here
53faf3
 
53faf3
@@ -643,11 +648,14 @@ AM_CONDITIONAL([HAVE_DICT_WORDS], [test
53faf3
 # substitute what we need:
53faf3
 AC_SUBST([SOCKETDIR])
53faf3
 AC_SUBST([LINT_FLAGS])
53faf3
+AC_SUBST([FORCESOCKETSFILE])
53faf3
 
53faf3
 AC_DEFINE_UNQUOTED([SOCKETDIR], "$(eval echo ${SOCKETDIR})", [Socket directory])
53faf3
 AC_DEFINE_UNQUOTED([LOCALSTATEDIR], "$(eval echo ${localstatedir})", [localstate directory])
53faf3
 AC_DEFINE_UNQUOTED([PACKAGE_FEATURES], "${PACKAGE_FEATURES}", [quarterback built-in features])
53faf3
 
53faf3
+AC_DEFINE_UNQUOTED([FORCESOCKETSFILE], "$(eval echo ${FORCESOCKETSFILE})", [for sockets config file])
53faf3
+
53faf3
 # version parsing (for qbconfig.h)
53faf3
 AC_DEFINE_UNQUOTED([QB_VER_MAJOR],
53faf3
                    [$(echo "${VERSION}" \
53faf3
Only in libqb-1.0.1: configure.ac.orig
53faf3
diff -upr libqb-1.0.1.orig/docs/mainpage.h libqb-1.0.1/docs/mainpage.h
53faf3
--- libqb-1.0.1.orig/docs/mainpage.h	2016-11-08 10:10:23.000000000 +0000
53faf3
+++ libqb-1.0.1/docs/mainpage.h	2017-05-15 10:47:39.042145452 +0100
53faf3
@@ -101,6 +101,19 @@
53faf3
  * a single one pushed throughout its lifecycle just with a single thread;
53faf3
  * anything else would likely warrant external synchronization enforcement.
53faf3
  *
53faf3
+ * @par IPC sockets (Linux only)
53faf3
+ * On Linux IPC, abstract (non-filesystem) sockets are used by default. If you
53faf3
+ * need to override this (say in a net=host container) and use sockets that reside
53faf3
+ * in the filesystem, then create a file called /etc/libqb/force-filesystem-sockets
53faf3
+ * - this is the default name and can be changed at ./configure time.
53faf3
+ * The file need contain no text, it's not a configuration file as such, just its
53faf3
+ * presence will activate the feature.
53faf3
+ *
53faf3
+ * Note that this is a global option and read each time a new IPC connection
53faf3
+ * (client or server) is created. So, to avoid having clients that cannot
53faf3
+ * connect to running servers it is STRONGLY recommended to only create or remove
53faf3
+ * this file prior to a system reboot or container restart.
53faf3
+ *
53faf3
  * @par Client API
53faf3
  * @copydoc qbipcc.h
53faf3
  * @see qbipcc.h
53faf3
Only in libqb-1.0.1/docs: mainpage.h.orig
53faf3
diff -upr libqb-1.0.1.orig/lib/ipc_int.h libqb-1.0.1/lib/ipc_int.h
53faf3
--- libqb-1.0.1.orig/lib/ipc_int.h	2016-02-22 16:01:51.000000000 +0000
53faf3
+++ libqb-1.0.1/lib/ipc_int.h	2017-05-15 10:47:39.042145452 +0100
53faf3
@@ -205,4 +205,6 @@ int32_t qb_ipcs_process_request(struct q
53faf3
 
53faf3
 int32_t qb_ipc_us_sock_error_is_disconnected(int err);
53faf3
 
53faf3
+int use_filesystem_sockets(void);
53faf3
+
53faf3
 #endif /* QB_IPC_INT_H_DEFINED */
53faf3
diff -upr libqb-1.0.1.orig/lib/ipc_setup.c libqb-1.0.1/lib/ipc_setup.c
53faf3
--- libqb-1.0.1.orig/lib/ipc_setup.c	2016-11-08 10:10:23.000000000 +0000
53faf3
+++ libqb-1.0.1/lib/ipc_setup.c	2017-05-15 10:47:39.042145452 +0100
53faf3
@@ -69,7 +69,6 @@ struct ipc_auth_data {
53faf3
 
53faf3
 };
53faf3
 
53faf3
-
53faf3
 static int32_t qb_ipcs_us_connection_acceptor(int fd, int revent, void *data);
53faf3
 
53faf3
 ssize_t
53faf3
@@ -286,12 +285,13 @@ qb_ipcc_stream_sock_connect(const char *
53faf3
 	address.sun_len = QB_SUN_LEN(&address);
53faf3
 #endif
53faf3
 
53faf3
-#if defined(QB_LINUX) || defined(QB_CYGWIN)
53faf3
-	snprintf(address.sun_path + 1, UNIX_PATH_MAX - 1, "%s", socket_name);
53faf3
-#else
53faf3
-	snprintf(address.sun_path, sizeof(address.sun_path), "%s/%s", SOCKETDIR,
53faf3
-		 socket_name);
53faf3
-#endif
53faf3
+	if (!use_filesystem_sockets()) {
53faf3
+		snprintf(address.sun_path + 1, UNIX_PATH_MAX - 1, "%s", socket_name);
53faf3
+	} else {
53faf3
+		snprintf(address.sun_path, sizeof(address.sun_path), "%s/%s", SOCKETDIR,
53faf3
+			 socket_name);
53faf3
+	}
53faf3
+
53faf3
 	if (connect(request_fd, (struct sockaddr *)&address,
53faf3
 		    QB_SUN_LEN(&address)) == -1) {
53faf3
 		res = -errno;
53faf3
@@ -535,10 +535,11 @@ qb_ipcs_us_publish(struct qb_ipcs_servic
53faf3
 #endif
53faf3
 
53faf3
 	qb_util_log(LOG_INFO, "server name: %s", s->name);
53faf3
-#if defined(QB_LINUX) || defined(QB_CYGWIN)
53faf3
-	snprintf(un_addr.sun_path + 1, UNIX_PATH_MAX - 1, "%s", s->name);
53faf3
-#else
53faf3
-	{
53faf3
+
53faf3
+	if (!use_filesystem_sockets()) {
53faf3
+		snprintf(un_addr.sun_path + 1, UNIX_PATH_MAX - 1, "%s", s->name);
53faf3
+	}
53faf3
+	else {
53faf3
 		struct stat stat_out;
53faf3
 		res = stat(SOCKETDIR, &stat_out);
53faf3
 		if (res == -1 || (res == 0 && !S_ISDIR(stat_out.st_mode))) {
53faf3
@@ -552,7 +553,6 @@ qb_ipcs_us_publish(struct qb_ipcs_servic
53faf3
 			 s->name);
53faf3
 		unlink(un_addr.sun_path);
53faf3
 	}
53faf3
-#endif
53faf3
 
53faf3
 	res = bind(s->server_sock, (struct sockaddr *)&un_addr,
53faf3
 		   QB_SUN_LEN(&un_addr));
53faf3
@@ -561,15 +561,15 @@ qb_ipcs_us_publish(struct qb_ipcs_servic
53faf3
 		qb_util_perror(LOG_ERR, "Could not bind AF_UNIX (%s)",
53faf3
 			       un_addr.sun_path);
53faf3
 		goto error_close;
53faf3
-	}
53faf3
+        }
53faf3
 
53faf3
 	/*
53faf3
 	 * Allow everyone to write to the socket since the IPC layer handles
53faf3
 	 * security automatically
53faf3
 	 */
53faf3
-#if !defined(QB_LINUX) && !defined(QB_CYGWIN)
53faf3
-	res = chmod(un_addr.sun_path, S_IRWXU | S_IRWXG | S_IRWXO);
53faf3
-#endif
53faf3
+	if (use_filesystem_sockets()) {
53faf3
+	        res = chmod(un_addr.sun_path, S_IRWXU | S_IRWXG | S_IRWXO);
53faf3
+        }
53faf3
 #ifdef SO_PASSCRED
53faf3
 	setsockopt(s->server_sock, SOL_SOCKET, SO_PASSCRED, &on, sizeof(on));
53faf3
 #endif
53faf3
@@ -593,6 +593,16 @@ qb_ipcs_us_withdraw(struct qb_ipcs_servi
53faf3
 	qb_util_log(LOG_INFO, "withdrawing server sockets");
53faf3
 	(void)s->poll_fns.dispatch_del(s->server_sock);
53faf3
 	shutdown(s->server_sock, SHUT_RDWR);
53faf3
+
53faf3
+	if (use_filesystem_sockets()) {
53faf3
+		struct sockaddr_un sockname;
53faf3
+		socklen_t socklen = sizeof(sockname);
53faf3
+		if ((getsockname(s->server_sock, (struct sockaddr *)&sockname, &socklen) == 0) &&
53faf3
+		    sockname.sun_family == AF_UNIX) {
53faf3
+			unlink(sockname.sun_path);
53faf3
+		}
53faf3
+	}
53faf3
+
53faf3
 	close(s->server_sock);
53faf3
 	s->server_sock = -1;
53faf3
 	return 0;
53faf3
diff -upr libqb-1.0.1.orig/lib/ipc_socket.c libqb-1.0.1/lib/ipc_socket.c
53faf3
--- libqb-1.0.1.orig/lib/ipc_socket.c	2016-11-08 10:10:23.000000000 +0000
53faf3
+++ libqb-1.0.1/lib/ipc_socket.c	2017-05-15 10:47:39.043145455 +0100
53faf3
@@ -42,6 +42,26 @@ struct ipc_us_control {
53faf3
 };
53faf3
 #define SHM_CONTROL_SIZE (3 * sizeof(struct ipc_us_control))
53faf3
 
53faf3
+int use_filesystem_sockets(void)
53faf3
+{
53faf3
+	static int need_init = 1;
53faf3
+	static int filesystem_sockets = 0;
53faf3
+
53faf3
+	if (need_init) {
53faf3
+		struct stat buf;
53faf3
+
53faf3
+		need_init = 0;
53faf3
+#if defined(QB_LINUX) || defined(QB_CYGWIN)
53faf3
+		if (stat(FORCESOCKETSFILE, &buf) == 0) {
53faf3
+			filesystem_sockets = 1;
53faf3
+		}
53faf3
+#else
53faf3
+		filesystem_sockets = 1;
53faf3
+#endif
53faf3
+	}
53faf3
+	return filesystem_sockets;
53faf3
+}
53faf3
+
53faf3
 static void
53faf3
 set_sock_addr(struct sockaddr_un *address, const char *socket_name)
53faf3
 {
53faf3
@@ -51,12 +71,12 @@ set_sock_addr(struct sockaddr_un *addres
53faf3
 	address->sun_len = QB_SUN_LEN(address);
53faf3
 #endif
53faf3
 
53faf3
-#if defined(QB_LINUX) || defined(QB_CYGWIN)
53faf3
-	snprintf(address->sun_path + 1, UNIX_PATH_MAX - 1, "%s", socket_name);
53faf3
-#else
53faf3
-	snprintf(address->sun_path, sizeof(address->sun_path), "%s/%s", SOCKETDIR,
53faf3
-		 socket_name);
53faf3
-#endif
53faf3
+	if (!use_filesystem_sockets()) {
53faf3
+		snprintf(address->sun_path + 1, UNIX_PATH_MAX - 1, "%s", socket_name);
53faf3
+	} else {
53faf3
+		snprintf(address->sun_path, sizeof(address->sun_path), "%s/%s", SOCKETDIR,
53faf3
+			 socket_name);
53faf3
+	}
53faf3
 }
53faf3
 
53faf3
 static int32_t
53faf3
@@ -81,15 +101,16 @@ qb_ipc_dgram_sock_setup(const char *base
53faf3
 	}
53faf3
 	snprintf(sock_path, PATH_MAX, "%s-%s", base_name, service_name);
53faf3
 	set_sock_addr(&local_address, sock_path);
53faf3
-#if !(defined(QB_LINUX) || defined(QB_CYGWIN))
53faf3
-	res = unlink(local_address.sun_path);
53faf3
-#endif
53faf3
+	if (use_filesystem_sockets()) {
53faf3
+		res = unlink(local_address.sun_path);
53faf3
+	}
53faf3
 	res = bind(request_fd, (struct sockaddr *)&local_address,
53faf3
 		   sizeof(local_address));
53faf3
-#if !(defined(QB_LINUX) || defined(QB_CYGWIN))
53faf3
-	chmod(local_address.sun_path, 0660);
53faf3
-	chown(local_address.sun_path, -1, gid);
53faf3
-#endif
53faf3
+
53faf3
+	if (use_filesystem_sockets()) {
53faf3
+		chmod(local_address.sun_path, 0660);
53faf3
+		chown(local_address.sun_path, -1, gid);
53faf3
+	}
53faf3
 	if (res < 0) {
53faf3
 		goto error_connect;
53faf3
 	}
53faf3
@@ -316,36 +337,33 @@ _finish_connecting(struct qb_ipc_one_way
53faf3
 static void
53faf3
 qb_ipcc_us_disconnect(struct qb_ipcc_connection *c)
53faf3
 {
53faf3
-#if !(defined(QB_LINUX) || defined(QB_CYGWIN))
53faf3
-  struct sockaddr_un un_addr;
53faf3
-  socklen_t un_addr_len = sizeof(struct sockaddr_un);
53faf3
-  char *base_name;
53faf3
-  char sock_name[PATH_MAX];
53faf3
-  size_t length;
53faf3
-#endif
53faf3
-
53faf3
 	munmap(c->request.u.us.shared_data, SHM_CONTROL_SIZE);
53faf3
 	unlink(c->request.u.us.shared_file_name);
53faf3
 
53faf3
-#if !(defined(QB_LINUX) || defined(QB_CYGWIN))
53faf3
-    if (getsockname(c->response.u.us.sock, (struct sockaddr *)&un_addr, &un_addr_len) == 0) {
53faf3
-      length = strlen(un_addr.sun_path);
53faf3
-      base_name = strndup(un_addr.sun_path,length-9);
53faf3
-      qb_util_log(LOG_DEBUG, "unlinking socket bound files with base_name=%s length=%d",base_name,length);
53faf3
-      snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"request");
53faf3
-      qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
-      unlink(sock_name);
53faf3
-      snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"event");
53faf3
-      qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
-      unlink(sock_name);
53faf3
-      snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"event-tx");
53faf3
-      qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
-      unlink(sock_name);
53faf3
-      snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"response");
53faf3
-      qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
-      unlink(sock_name);
53faf3
-    }
53faf3
-#endif
53faf3
+	if (use_filesystem_sockets()) {
53faf3
+		struct sockaddr_un un_addr;
53faf3
+		socklen_t un_addr_len = sizeof(struct sockaddr_un);
53faf3
+		char *base_name;
53faf3
+		char sock_name[PATH_MAX];
53faf3
+		size_t length;
53faf3
+		if (getsockname(c->response.u.us.sock, (struct sockaddr *)&un_addr, &un_addr_len) == 0) {
53faf3
+			length = strlen(un_addr.sun_path);
53faf3
+			base_name = strndup(un_addr.sun_path,length-9);
53faf3
+			qb_util_log(LOG_DEBUG, "unlinking socket bound files with base_name=%s length=%d",base_name,length);
53faf3
+			snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"request");
53faf3
+			qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
+			unlink(sock_name);
53faf3
+			snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"event");
53faf3
+			qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
+			unlink(sock_name);
53faf3
+			snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"event-tx");
53faf3
+			qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
+			unlink(sock_name);
53faf3
+			snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"response");
53faf3
+			qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
+			unlink(sock_name);
53faf3
+		}
53faf3
+	}
53faf3
 	qb_ipcc_us_sock_close(c->event.u.us.sock);
53faf3
 	qb_ipcc_us_sock_close(c->request.u.us.sock);
53faf3
 	qb_ipcc_us_sock_close(c->setup.u.us.sock);
53faf3
@@ -451,11 +469,11 @@ retry_peek:
53faf3
 
53faf3
 		if (errno != EAGAIN) {
53faf3
 			final_rc = -errno;
53faf3
-#if !(defined(QB_LINUX) || defined(QB_CYGWIN))
53faf3
-			if (errno == ECONNRESET || errno == EPIPE) {
53faf3
-				final_rc = -ENOTCONN;
53faf3
+			if (use_filesystem_sockets()) {
53faf3
+				if (errno == ECONNRESET || errno == EPIPE) {
53faf3
+					final_rc = -ENOTCONN;
53faf3
+				}
53faf3
 			}
53faf3
-#endif
53faf3
 			goto cleanup_sigpipe;
53faf3
 		}
53faf3
 
53faf3
@@ -686,38 +704,36 @@ _sock_rm_from_mainloop(struct qb_ipcs_co
53faf3
 static void
53faf3
 qb_ipcs_us_disconnect(struct qb_ipcs_connection *c)
53faf3
 {
53faf3
-#if !(defined(QB_LINUX) || defined(QB_CYGWIN))
53faf3
-	struct sockaddr_un un_addr;
53faf3
-	socklen_t un_addr_len = sizeof(struct sockaddr_un);
53faf3
-	char *base_name;
53faf3
-	char sock_name[PATH_MAX];
53faf3
-	size_t length;
53faf3
-#endif
53faf3
 	qb_enter();
53faf3
 
53faf3
 	if (c->state == QB_IPCS_CONNECTION_ESTABLISHED ||
53faf3
 	    c->state == QB_IPCS_CONNECTION_ACTIVE) {
53faf3
 		_sock_rm_from_mainloop(c);
53faf3
 
53faf3
-#if !(defined(QB_LINUX) || defined(QB_CYGWIN))
53faf3
-		if (getsockname(c->response.u.us.sock, (struct sockaddr *)&un_addr, &un_addr_len) == 0) {
53faf3
-			length = strlen(un_addr.sun_path);
53faf3
-			base_name = strndup(un_addr.sun_path,length-8);
53faf3
-			qb_util_log(LOG_DEBUG, "unlinking socket bound files with base_name=%s length=%d",base_name,length);
53faf3
-			snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"request");
53faf3
-			qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
-			unlink(sock_name);
53faf3
-			snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"event");
53faf3
-			qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
-			unlink(sock_name);
53faf3
-			snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"event-tx");
53faf3
-			qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
-			unlink(sock_name);
53faf3
-			snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"response");
53faf3
-			qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
-			unlink(sock_name);
53faf3
+		if (use_filesystem_sockets()) {
53faf3
+			struct sockaddr_un un_addr;
53faf3
+			socklen_t un_addr_len = sizeof(struct sockaddr_un);
53faf3
+			char *base_name;
53faf3
+			char sock_name[PATH_MAX];
53faf3
+			size_t length;
53faf3
+			if (getsockname(c->response.u.us.sock, (struct sockaddr *)&un_addr, &un_addr_len) == 0) {
53faf3
+				length = strlen(un_addr.sun_path);
53faf3
+				base_name = strndup(un_addr.sun_path,length-8);
53faf3
+				qb_util_log(LOG_DEBUG, "unlinking socket bound files with base_name=%s length=%d",base_name,length);
53faf3
+				snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"request");
53faf3
+				qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
+				unlink(sock_name);
53faf3
+				snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"event");
53faf3
+				qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
+				unlink(sock_name);
53faf3
+				snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"event-tx");
53faf3
+				qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
+				unlink(sock_name);
53faf3
+				snprintf(sock_name,PATH_MAX,"%s-%s",base_name,"response");
53faf3
+				qb_util_log(LOG_DEBUG, "unlink sock_name=%s",sock_name);
53faf3
+				unlink(sock_name);
53faf3
+			}
53faf3
 		}
53faf3
-#endif
53faf3
 		qb_ipcc_us_sock_close(c->setup.u.us.sock);
53faf3
 		qb_ipcc_us_sock_close(c->request.u.us.sock);
53faf3
 		qb_ipcc_us_sock_close(c->event.u.us.sock);
53faf3
Only in libqb-1.0.1/lib: ipc_socket.c.orig
53faf3
diff -upr libqb-1.0.1.orig/tests/check_ipc.c libqb-1.0.1/tests/check_ipc.c
53faf3
--- libqb-1.0.1.orig/tests/check_ipc.c	2016-11-08 11:15:16.000000000 +0000
53faf3
+++ libqb-1.0.1/tests/check_ipc.c	2017-05-15 10:47:39.043145455 +0100
53faf3
@@ -1417,10 +1417,17 @@ END_TEST
53faf3
 #ifdef HAVE_FAILURE_INJECTION
53faf3
 START_TEST(test_ipcc_truncate_when_unlink_fails_shm)
53faf3
 {
53faf3
+	char sock_file[PATH_MAX];
53faf3
 	qb_enter();
53faf3
-	_fi_unlink_inject_failure = QB_TRUE;
53faf3
 	ipc_type = QB_IPC_SHM;
53faf3
 	set_ipc_name(__func__);
53faf3
+
53faf3
+	sprintf(sock_file, "%s/%s", SOCKETDIR, ipc_name);
53faf3
+	/* If there's an old socket left from a previous run this test will fail
53faf3
+	   unexpectedly, so try to remove it first */
53faf3
+	unlink(sock_file);
53faf3
+
53faf3
+	_fi_unlink_inject_failure = QB_TRUE;
53faf3
 	test_ipc_server_fail();
53faf3
 	_fi_unlink_inject_failure = QB_FALSE;
53faf3
 	qb_leave();
53faf3
Only in libqb-1.0.1/tests: check_ipc.c.orig