Bug 1930535 - [abrt] [faf] sssd: monitor_service_shutdown(): /usr/sbin/sssd killed by 11
Summary: [abrt] [faf] sssd: monitor_service_shutdown(): /usr/sbin/sssd killed by 11
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 8
Classification: Red Hat
Component: sssd
Version: 8.4
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: ---
Assignee: Alexey Tikhonov
QA Contact: Steeve Goveas
URL: http://faf.lab.eng.brq.redhat.com/faf...
Whiteboard: sync-to-jira
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-02-19 05:41 UTC by Steeve Goveas
Modified: 2021-11-10 09:08 UTC (History)
9 users (show)

Fixed In Version: sssd-2.5.0-1.el8
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-11-09 19:47:00 UTC
Type: Bug
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
coredump and other files (574.72 KB, application/gzip)
2021-02-19 05:41 UTC, Steeve Goveas
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Github SSSD sssd issues 5598 0 None open NULL dereference in monitor_service_shutdown() 2021-04-22 16:07:07 UTC
Red Hat Product Errata RHBA-2021:4435 0 None None None 2021-11-09 19:47:16 UTC

Description Steeve Goveas 2021-02-19 05:41:37 UTC
Created attachment 1757991 [details]
coredump and other files

This bug has been created based on an anonymous crash report requested by the package maintainer.

Report URL: http://faf.lab.eng.brq.redhat.com/faf/reports/bthash/d41e23acdb73f15bb85975144c7d7de04afa9e2d/

http://faf.lab.eng.brq.redhat.com/faf/reports/20480/

Comment 1 Alexey Tikhonov 2021-04-19 19:35:11 UTC
#0  0x00005598c44aaa7c in monitor_service_shutdown (svc=0x5598c53d20e0) at src/monitor/monitor.c:2151
#1  0x00007f9951e0c682 in n (conn=conn@entry=0x5598c5386770) at src/sbus/connection/sbus_connection.c:78
#2  0x00007f99517d4d10 in _tc_free_internal (location=0x7f99517dbed5 "../../talloc.c:440", tc=0x5598c5386710) at ../../talloc.c:1157
...
#17 0x00007f99517da390 in talloc_lib_atexit () at ../../talloc.c:440
#18 0x00007f9950ed2b0c in __run_exit_handlers () from /lib64/libc.so.6
#19 0x00007f9950ed2c40 in exit () from /lib64/libc.so.6
#20 0x00005598c44ac781 in monitor_quit (ret=0, mt_ctx=<optimized out>) at src/monitor/monitor.c:1473
#21 0x00005598c44ad078 in monitor_quit_signal (ev=<optimized out>, se=<optimized out>, signum=15, count=<optimized out>, siginfo=<optimized out>, 
    private_data=<optimized out>) at src/monitor/monitor.c:1490
#22 0x00007f99519eb666 in tevent_common_invoke_signal_handler (se=se@entry=0x5598c5374fd0, signum=signum@entry=15, count=count@entry=1, 
    siginfo=siginfo@entry=0x0, removed=removed@entry=0x0) at ../../tevent_signal.c:370


(gdb) frame
#0  0x00005598c44aaa7c in monitor_service_shutdown (svc=0x5598c53d20e0) at src/monitor/monitor.c:2151
2151	    ctx->num_services--;


(gdb) p ctx
$1 = (struct mt_ctx *) 0x0


https://github.com/SSSD/sssd/blob/95adf488f94f5968f6cfba9e3bef74c07c02ccff/src/monitor/monitor.c#L1599 :
```
    /* zero out references in SVCs so that they don't try
     * to access the monitor context on process shutdown */

    for (svc = mon->svc_list; svc; svc = svc->next) {
        svc->mt_ctx = NULL;
    }
```

Comment 4 Pavel Březina 2021-04-26 09:33:52 UTC
Pushed PR: https://github.com/SSSD/sssd/pull/5599

* `master`
    * 38905cac4b67f0e4c4b0f59af9ea7474482f088e - monitor: avoid NULL deref in monitor_service_shutdown(

Comment 12 errata-xmlrpc 2021-11-09 19:47:00 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (sssd bug fix and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2021:4435


Note You need to log in before you can comment on or make changes to this bug.