Bug 622008 - Assertion 's' failed at src/util.c:1614, function bus_path_escape()
Summary: Assertion 's' failed at src/util.c:1614, function bus_path_escape()
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: systemd
Version: 14
Hardware: x86_64
OS: Linux
low
medium
Target Milestone: ---
Assignee: Lennart Poettering
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard: abrt_hash:5f83760a3648f7cb160ef69aa1d...
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-08-06 18:31 UTC by Nicolas Mailhot
Modified: 2010-08-09 15:07 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-08-09 15:07:05 UTC
Type: ---

Attachments (Terms of Use)
File: backtrace (7.27 KB, text/plain)
2010-08-06 18:31 UTC, Nicolas Mailhot 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Description Nicolas Mailhot 2010-08-06 18:31:21 UTC 
abrt version: 1.1.10
architecture: x86_64
Attached file: backtrace
cmdline: /sbin/init
component: systemd
crash_function: crash
executable: /bin/systemd
kernel: 2.6.35-2.fc14.x86_64
package: systemd-6-1.fc14
rating: 4
reason: Process /bin/systemd was killed by signal 6 (SIGABRT)
release: Fedora release 14 (Branched)
time: 1281115738
uid: 0
Comment 1 Nicolas Mailhot 2010-08-06 18:31:23 UTC 
Created attachment 437229 [details]
File: backtrace
Comment 2 Michal Schmidt 2010-08-08 13:33:24 UTC 
This is the assertion failure:
  Assertion 's' failed at src/util.c:1614, function bus_path_escape().

unit_free() was called for a unit with u->meta.id == NULL and bus_unit_send_removed_signal() was therefore unable to construct a dbus path to announce in the UnitRemoved signal.

The timeline could have been like this:
unit_merge() ->
  merge_names();  /* sets other->meta.id = NULL */
  unit_add_to_cleanup_queue(other);
Later, manager_dispatch_cleanup_queue() ->
  unit_free()
    bus_unit_send_removed_signal()

To reproduce it a DBus subscriber has to be active:
1. Run systemadm
2. systemctl daemon-reload
Comment 3 Lennart Poettering 2010-08-09 15:07:05 UTC 
Michal, thanks for tracking this down. I am pretty sure I have now fixed this in current git now. Will upload updated package soon. Please test and reopen should it turn out not to be fixed completely.
Note You need to log in before you can comment on or make changes to this bug.