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
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/sssd-devel@lists.fedorahosted.org/message/GCI5WM6WAFOARWEDNCQP7JHKCIIOFGMI/

Reply via email to