Bug 875653

Summary:

[abrt] systemd-195-6.fc18: server_rotate: Process /usr/lib/systemd/systemd-journald was killed by signal 11 (SIGSEGV)

Product:

[Fedora] Fedora

Reporter:

Elad Alfassa <elad>

Component:

systemd

Assignee:

systemd-maint

Status:

CLOSED ERRATA

QA Contact:

Fedora Extras Quality Assurance <extras-qa>

Severity:

unspecified

Docs Contact:

Priority:

unspecified

Version:

CC:

johannbg, lnykryn, metherid, mschmidt, msekleta, notting, plautrba, sheepdestroyer, systemd-maint, vpavlin

Target Milestone:

---

Target Release:

---

Hardware:

x86_64

OS:

Unspecified

Whiteboard:

abrt_hash:86d3d7da5a1d7ba3facb397ab953f09986be225c

Fixed In Version:

Doc Type:

Bug Fix

Doc Text:

Story Points:

---

Clone Of:

Environment:

Last Closed:

2012-12-08 04:35:08 UTC

Type:

---

Regression:

---

Mount Type:

---

Documentation:

---

CRM:

Verified Versions:

Category:

---

oVirt Team:

---

RHEL 7.3 requirements from Atomic Host:

Cloudforms Team:

---

Target Upstream Version:

Embargoed:

Attachments:

Description	Flags
File: core_backtrace	none
File: environ	none
File: backtrace	none
File: limits	none
File: cgroup	none
File: smolt_data	none
File: executable	none
File: maps	none
File: dso_list	none
File: proc_pid_status	none
File: open_fds	none

Description Elad Alfassa 2012-11-12 09:56:41 UTC

Version-Release number of selected component:
systemd-195-6.fc18

Additional info:
libreport version: 2.0.18
abrt_version:   2.0.18
backtrace_rating: 4
cmdline:        /usr/lib/systemd/systemd-journald
crash_function: server_rotate
kernel:         3.6.6-3.fc18.x86_64

truncated backtrace:
:Thread no. 1 (6 frames)
: #0 server_rotate at src/journal/journald.c:345
: #1 write_to_journal at src/journal/journald.c:507
: #2 dispatch_message_real at src/journal/journald.c:684
: #3 server_dispatch_message at src/journal/journald.c:801
: #4 server_process_syslog_message at src/journal/journald-syslog.c:403
: #5 process_event at src/journal/journald.c:1157

Comment 1 Elad Alfassa 2012-11-12 09:56:48 UTC

Created attachment 643350 [details]
File: core_backtrace

Comment 2 Elad Alfassa 2012-11-12 09:56:53 UTC

Created attachment 643351 [details]
File: environ

Comment 3 Elad Alfassa 2012-11-12 09:56:59 UTC

Created attachment 643352 [details]
File: backtrace

Comment 4 Elad Alfassa 2012-11-12 09:57:03 UTC

Created attachment 643353 [details]
File: limits

Comment 5 Elad Alfassa 2012-11-12 09:57:06 UTC

Created attachment 643355 [details]
File: cgroup

Comment 6 Elad Alfassa 2012-11-12 09:57:10 UTC

Created attachment 643357 [details]
File: smolt_data

Comment 7 Elad Alfassa 2012-11-12 09:57:14 UTC

Created attachment 643359 [details]
File: executable

Comment 8 Elad Alfassa 2012-11-12 09:57:17 UTC

Created attachment 643361 [details]
File: maps

Comment 9 Elad Alfassa 2012-11-12 09:57:20 UTC

Created attachment 643362 [details]
File: dso_list

Comment 10 Elad Alfassa 2012-11-12 09:57:23 UTC

Created attachment 643363 [details]
File: proc_pid_status

Comment 11 Elad Alfassa 2012-11-12 09:57:27 UTC

Created attachment 643364 [details]
File: open_fds

Comment 12 Michal Schmidt 2012-11-13 09:29:46 UTC

So journald crashed here in server_rotate():

        HASHMAP_FOREACH_KEY(f, k, s->user_journals, i) { 
                r = journal_file_rotate(&f, s->compress, s->seal); 
                if (r < 0) 
                        if (f->path)  // <=== f was NULL here!
                            ...

f was not NULL on entering journal_file_rotate() (otherwise an assertion would have failed). f gets assigned to NULL when journal_file_open() fails.

Looks like we should simply check f in server_rotate(), not f->path.

Comment 13 Michal Schmidt 2012-11-13 09:31:46 UTC

*** Bug 875996 has been marked as a duplicate of this bug. ***

Comment 14 Lennart Poettering 2012-11-19 22:38:32 UTC

(In reply to comment #12)
> So journald crashed here in server_rotate():
> 
>         HASHMAP_FOREACH_KEY(f, k, s->user_journals, i) { 
>                 r = journal_file_rotate(&f, s->compress, s->seal); 
>                 if (r < 0) 
>                         if (f->path)  // <=== f was NULL here!
>                             ...
> 
> f was not NULL on entering journal_file_rotate() (otherwise an assertion
> would have failed). f gets assigned to NULL when journal_file_open() fails.
> 
> Looks like we should simply check f in server_rotate(), not f->path.

Indeed. Will fix.

Comment 15 Lennart Poettering 2012-11-20 20:26:28 UTC

Fixed in git.

Comment 16 Fedora Update System 2012-12-05 17:19:33 UTC

systemd-195-10.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/systemd-195-10.fc18

Comment 17 Fedora Update System 2012-12-05 23:15:13 UTC

Package systemd-195-10.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing systemd-195-10.fc18'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-19811/systemd-195-10.fc18
then log in and leave karma (feedback).

Comment 18 Fedora Update System 2012-12-08 04:35:10 UTC

systemd-195-10.fc18 has been pushed to the Fedora 18 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 19 Fedora Update System 2013-02-15 10:15:43 UTC

systemd-44-24.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/systemd-44-24.fc17

Comment 20 Fedora Update System 2013-03-04 22:33:33 UTC

systemd-44-24.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.