From 00e948049acf0ca1b61ed9c2b8579b06b4bcb46a Mon Sep 17 00:00:00 2001
From: Rob Crittenden <rcritten@redhat.com>
Date: Sep 08 2020 17:56:49 +0000
Subject: Don't send SIGKILL to children, give them a chance to die


This was causing issues in IPA which uses a lock file to
serialize some operations. The kill was leaving the lock in
place causing things to time out.

---

diff --git a/src/subproc.c b/src/subproc.c
index 8df836a..70d4ed9 100644
--- a/src/subproc.c
+++ b/src/subproc.c
@@ -240,7 +240,6 @@ cm_subproc_done(struct cm_subproc_state *state)
 
 	if (state != NULL) {
 		if (state->pid != -1) {
-			kill(state->pid, SIGKILL);
 			do {
 				pid = waitpid(state->pid, &state->status, 0);
 				cm_log(4, "Waited for %ld, got %ld.\n",