[sssd PR#581][opened] LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/581
Author: jhrozek
 Title: #581: LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler
Action: opened

PR body:
"""
The DEBUG macros internally use several signal-unsafe calls so it's better
to not use any DEBUG macros at all.

man 7 signal-safety lists functions that can be used in a signal handler.
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/581/head:pr581
git checkout pr581

[sssd PR#581][comment] LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

[fidencio]

URL: https://github.com/SSSD/sssd/pull/581
Title: #581: LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

fidencio commented:
"""
Nice catch!

This is quite similar problem to the one we faced long-time ago in the watchdog code.

Ack! I'll fire internal CI and add the "Accepted" label when I get the results.
"""

See the full comment at https://github.com/SSSD/sssd/pull/581#issuecomment-392707600

[sssd PR#581][comment] LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

[fidencio]

URL: https://github.com/SSSD/sssd/pull/581
Title: #581: LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

fidencio commented:
"""
Ci: http://vm-031.$%7Babc%7D/logs/job/89/57/summary.html

There was one failure on rawhide, which doesn't seem related to this patch.
"""

See the full comment at https://github.com/SSSD/sssd/pull/581#issuecomment-393044031

[sssd PR#581][+Accepted] LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

[fidencio]

URL: https://github.com/SSSD/sssd/pull/581
Title: #581: LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

Label: +Accepted

[sssd PR#581][comment] LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

[fidencio]

URL: https://github.com/SSSD/sssd/pull/581
Title: #581: LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

fidencio commented:
"""
master:
 50a90eb
"""

See the full comment at https://github.com/SSSD/sssd/pull/581#issuecomment-393681969

[sssd PR#581][closed] LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

[fidencio]

URL: https://github.com/SSSD/sssd/pull/581
Author: jhrozek
 Title: #581: LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler
Action: closed

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/581/head:pr581
git checkout pr581

[sssd PR#581][+Pushed] LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

[fidencio]

URL: https://github.com/SSSD/sssd/pull/581
Title: #581: LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

Label: +Pushed

[sssd PR#581][-Accepted] LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

[fidencio]

URL: https://github.com/SSSD/sssd/pull/581
Title: #581: LDAP: Do not use signal-unsafe calls in ldap_child SIGTERM handler

Label: -Accepted