[Tarantool-patches] [PATCH 01/12] popen: allow to kill process group

Alexander Turenko alexander.turenko at tarantool.org
Tue Apr 14 14:38:10 MSK 2020


From: Cyrill Gorcunov <gorcunov at gmail.com>

As Alexander pointed out this might be useful
for running a pipe of programs inside shell
(i.e. popen.shell('foo | bar | baz', 'r')).

Part of #4031

Reported-by: Alexander Turenko <alexander.turenko at tarantool.org>
Acked-by: Alexander Turenko <alexander.turenko at tarantool.org>
Signed-off-by: Cyrill Gorcunov <gorcunov at gmail.com>
---
 src/lib/core/popen.c | 28 ++++++++++++++++++++++++----
 src/lib/core/popen.h |  6 ++++++
 2 files changed, 30 insertions(+), 4 deletions(-)

diff --git a/src/lib/core/popen.c b/src/lib/core/popen.c
index df7f797b9..a5a305013 100644
--- a/src/lib/core/popen.c
+++ b/src/lib/core/popen.c
@@ -138,6 +138,19 @@ handle_new(struct popen_opts *opts)
 
 	assert(opts->argv != NULL && opts->nr_argv > 0);
 
+	/*
+	 * Killing group of signals allowed for a new
+	 * session only where it makes sense, otherwise
+	 * child gonna inherit group and we will be killing
+	 * ourself.
+	 */
+	if (opts->flags & POPEN_FLAG_GROUP_SIGNAL &&
+	    (opts->flags & POPEN_FLAG_SETSID) == 0) {
+		diag_set(IllegalParams,
+			 "popen: group signal without setting sid");
+		return NULL;
+	}
+
 	for (i = 0; i < opts->nr_argv; i++) {
 		if (opts->argv[i] == NULL)
 			continue;
@@ -526,6 +539,9 @@ popen_state_str(unsigned int state)
 int
 popen_send_signal(struct popen_handle *handle, int signo)
 {
+	static const char *killops[] = { "kill", "killpg" };
+	const char *killop = handle->flags & POPEN_FLAG_GROUP_SIGNAL ?
+		killops[1] : killops[0];
 	int ret;
 
 	assert(handle != NULL);
@@ -535,17 +551,21 @@ popen_send_signal(struct popen_handle *handle, int signo)
 	 */
 	ret = popen_may_pidop(handle);
 	if (ret == 0) {
-		say_debug("popen: kill %d signo %d", handle->pid, signo);
+		say_debug("popen: %s %d signo %d", killop,
+			  handle->pid, signo);
 		assert(handle->pid != -1);
-		ret = kill(handle->pid, signo);
+		if (handle->flags & POPEN_FLAG_GROUP_SIGNAL)
+			ret = killpg(handle->pid, signo);
+		else
+			ret = kill(handle->pid, signo);
 	}
 	if (ret < 0 && errno == ESRCH) {
 		diag_set(SystemError, "Attempt to send a signal %d to a "
 			 "process that does not exist anymore", signo);
 		return -1;
 	} else if (ret < 0) {
-		diag_set(SystemError, "Unable to kill %d signo %d",
-			 handle->pid, signo);
+		diag_set(SystemError, "Unable to %s %d signo %d",
+			 killop, handle->pid, signo);
 		return -1;
 	}
 	return 0;
diff --git a/src/lib/core/popen.h b/src/lib/core/popen.h
index 97c581c13..623d826b9 100644
--- a/src/lib/core/popen.h
+++ b/src/lib/core/popen.h
@@ -96,6 +96,12 @@ enum popen_flag_bits {
 	 */
 	POPEN_FLAG_RESTORE_SIGNALS_BIT	= 15,
 	POPEN_FLAG_RESTORE_SIGNALS	= (1 << POPEN_FLAG_RESTORE_SIGNALS_BIT),
+
+	/*
+	 * Send signal to a process group.
+	 */
+	POPEN_FLAG_GROUP_SIGNAL_BIT	= 16,
+	POPEN_FLAG_GROUP_SIGNAL		= (1 << POPEN_FLAG_GROUP_SIGNAL_BIT),
 };
 
 /**
-- 
2.25.0



More information about the Tarantool-patches mailing list