Skip to content

Conversation

@rhc54
Copy link
Contributor

@rhc54 rhc54 commented Jun 27, 2017

Thanks to Ted Sussman for the report and patience in tracking it down

Signed-off-by: Ralph Castain [email protected]

Thanks to Ted Sussman for the report and patience in tracking it down

Signed-off-by: Ralph Castain <[email protected]>
@jjhursey
Copy link
Member

Interesting. We found a similar issue and fixed it differently in PR #3337 - Note that PR was not accepted in whole. I like the technique used this this PR though.

-            kill_local(cd->child->pid, SIGKILL);
+
+            /* Send signal to the negative of the PID to send the signal to all
+             * of the children of that PID - the process group under it.
+             * Otherwise it is delivered to only that PID.
+             */
+            kill_local(cd->child->pid * -1, SIGKILL);
+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants