Mailinglist Archive: opensuse-bugs (4243 mails)

< Previous Next >
[Bug 909418] when ejecting DVD using hardware button, system thinks DVD is still mounted
  • From: bugzilla_noreply@xxxxxxxxxx
  • Date: Thu, 05 Jan 2017 16:06:27 +0000
  • Message-id: <bug-909418-21960-eZUhNcObcT@http.bugzilla.suse.com/>
http://bugzilla.suse.com/show_bug.cgi?id=909418
http://bugzilla.suse.com/show_bug.cgi?id=909418#c45

Peter Wullinger <wullinger@xxxxxxxxxxxxxx> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |REOPENED
CC| |wullinger@xxxxxxxxxxxxxx
Resolution|FIXED |---

--- Comment #45 from Peter Wullinger <wullinger@xxxxxxxxxxxxxx> ---
The implemented patch seems to cause an assertion fault, in particular I can
observe:

systemd[11505]: Assertion 'dev' failed at src/core/device.c:301, function
device_is_bound_by_mounts(). Aborting.
systemd[1]: Assertion 'dev' failed at src/core/device.c:301, function
device_is_bound_by_mounts(). Aborting.
systemd[1]: Caught <ABRT>, dumped core as pid 30128.
systemd[1]: Freezing execution.

It looks like it is valid for dev to be NULL in device_setup_unit(Manager *m,
struct udev_device *dev, const char *path, bool main). At least there are
various guards with if (dev) in the source.

This is not checked before the (new) call to device_is_bound_by_mounts(), which
itself does assert(dev). As a result, PID1 aborts and the system cannot be
managed any more. I'm not familiar with the new logic, but the fix may be a
simple guard condition inside device_is_bound_by_mounts().

Note that this would halt a production machine hard, being unable to even
reboot normally (since the system dbus is down

Upstream Issue: https://github.com/systemd/systemd/issues/5025

--
You are receiving this mail because:
You are on the CC list for the bug.
< Previous Next >