Bug 9265 - systemd-logind: segfaults systemd == 225
Summary: systemd-logind: segfaults systemd == 225
Status: RESOLVED FIXED
Alias: None
Product: General Runtime
Classification: Runtime
Component: General Runtime (show other bugs)
Version: 2.0
Hardware: x86 Multiple
: Medium+ normal
Target Milestone: 2.0.2
Assignee: Bill Randle
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2016-03-15 18:00 UTC by David Turgeon
Modified: 2016-04-26 15:47 UTC (History)
0 users

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David Turgeon 2016-03-15 18:00:29 UTC 
On shutdown/reboot systemd-logind segfaults.

root@kraken-node:~# shutdown
systemd-logind[280]: segfault at 55ce64000000 ip 00007fe9b5a7df9a sp 00007fff931224b8 error 4 in libc-2.22.so[7fe9b59fd000+19a000]
Failed to set wall message, ignoring: Message did not receive a reply (timeout by message bus)
Failed to call ScheduleShutdown in logind, proceeding with immediate shutdown: Message did not receive a reply (timeout by message b
us)
[  OK  ] Stopped target Timers.

Investigating, this a bit more I also noticed that this issue was addressed in 1120's issue.  I've tried out the various patches, and confirmed that they do indeed fix the issue.  I can submit a patch for this shortly if Jethro is interested in such patches as these are likely already in 2.29.
Comment 1 David Turgeon 2016-03-16 18:04:58 UTC 
Basically these changes from systemd's repo:
    5744f59a3ee883ef3a78214bd5236157acdc35ba - 1141
    2cf088b56d72cb6a3243041524f1fbae7c1cb28e - 1141
    c7430c3d1a0c14aed631864b9da504ba1a9352c2 - 1157
Comment 2 Bill Randle 2016-03-21 23:23:51 UTC 
Applied suggested commits to Jethro branch version of systemd and created patch for OE mailing list.
Comment 3 Bill Randle 2016-03-22 00:02:00 UTC 
Confirmed via git log that the listed commits are included in systemd v229, which is the version used in master.
Comment 4 Bill Randle 2016-04-25 19:42:03 UTC 
Merged to Jethro.
Comment 5 David Turgeon 2016-04-26 15:47:44 UTC 
Yup, tested and seems to work as expected now.

(In reply to comment #4)
> Merged to Jethro.