URL:
https://github.com/SSSD/sssd/pull/579
Title: #579: krb5 locator: add support for multiple addresses and sub-domains
jhrozek commented:
"""
I think the general intention of the patch is good and I like destructors being used in
general, but there seems to be a bug. When I send SIGUSR1 to sssd to force it to go
offline, sssd_be crashes:
```
Program received signal SIGUSR1, User defined signal 1.
0x00007f8562e72aa0 in epoll_pwait () from /lib64/libc.so.6
(gdb) c
Continuing.
Program received signal SIGABRT, Aborted.
0x00007f8562d9266b in raise () from /lib64/libc.so.6
(gdb) bt
#0 0x00007f8562d9266b in raise () from /lib64/libc.so.6
#1 0x00007f8562d94381 in abort () from /lib64/libc.so.6
#2 0x00007f85635d030c in talloc_abort () from /lib64/libtalloc.so.2
#3 0x00007f85635d06d3 in talloc_check_name () from /lib64/libtalloc.so.2
#4 0x00007f855cacdade in remove_krb5_info_files_callback (pvt=0xf4a130)
at /sssd/src/providers/krb5/krb5_common.c:921
#5 0x000000000040ea69 in be_run_cb_step (ev=0xe70960, te=0xedfac0, current_time=...,
pvt=0xf6b4c0)
at /sssd/src/providers/data_provider_callbacks.c:96
#6 0x00007f85637ea491 in tevent_common_loop_timer_delay () from /lib64/libtevent.so.0
#7 0x00007f85637eb751 in epoll_event_loop_once () from /lib64/libtevent.so.0
#8 0x00007f85637e9af7 in std_event_loop_once () from /lib64/libtevent.so.0
#9 0x00007f85637e5f5d in _tevent_loop_once () from /lib64/libtevent.so.0
#10 0x00007f85637e617b in tevent_common_loop_wait () from /lib64/libtevent.so.0
#11 0x00007f85637e9a97 in std_event_loop_wait () from /lib64/libtevent.so.0
#12 0x00007f8567346420 in server_loop (main_ctx=0xe71df0) at /sssd/src/util/server.c:725
#13 0x0000000000409e96 in main (argc=8, argv=0x7ffe71409c18) at
/sssd/src/providers/data_provider_be.c:639
(gdb)
```
"""
See the full comment at
https://github.com/SSSD/sssd/pull/579#issuecomment-395111558