Title: #107: WATCHDOG: Avoid non async-signal-safe from the signal_handler
On (05/01/17 04:52), Jakub Hrozek wrote:
btw now I'm wondering if the setpgrp should be a separate patch
also for stable branches because I guess the bug was present in sssd for quite a long
I kind of misunderstood the question.
You meant 1.13 branch.
We call `kill(-getpgrp(), SIGTERM);` only for final cleanup in
But in case of setpgrp() we would be able to
call do following diff and cleanup would be automaticaly done per service
- kret = kill(svc->pid, SIGTERM);
+ kret = kill(-svc->pid, SIGTERM);
But patch for 1.13 and for 1.14 might be different.
See the full comment at https://github.com/SSSD/sssd/pull/107#issuecomment-270641029