URL: https://github.com/SSSD/sssd/pull/107 Title: #107: WATCHDOG: Avoid non async-signal-safe from the signal_handler
lslebodn commented: """ 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 >time? > I kind of misunderstood the question. You meant 1.13 branch. We call `kill(-getpgrp(), SIGTERM);` only for final cleanup in monitor_quit. 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. LS """ See the full comment at https://github.com/SSSD/sssd/pull/107#issuecomment-270641029
_______________________________________________ sssd-devel mailing list -- [email protected] To unsubscribe send an email to [email protected]
