Reintroduce kmod builtin as an optional dependency #3

Closed
ryao opened this Issue Nov 18, 2012 · 1 comment

Comments

Projects
None yet
1 participant
@ryao
Member

ryao commented Nov 18, 2012

We removed the kmod builtin because it was problematic both for those of us on Gentoo stable and in the sense that it was mandatory. We can reintroduce it as an optional dependency with a configure script option. This will either use some busybox-style module loading code or make it invoke modprobe.

The plan is for this to occur before our first release.

@ghost ghost assigned ryao Nov 18, 2012

@ryao

This comment has been minimized.

Show comment
Hide comment
@ryao

ryao Nov 23, 2012

Member

This has been fixed by the following commits to HEAD:

9b99681
135d61d
c4d1ce9
3bb4e05

Member

ryao commented Nov 23, 2012

This has been fixed by the following commits to HEAD:

9b99681
135d61d
c4d1ce9
3bb4e05

@ryao ryao closed this Nov 23, 2012

blueness added a commit that referenced this issue Sep 10, 2013

libudev: enumerate: fix NULL-deref for subsystem-matches
udev_device_get_subsystem() may return NULL if no subsystem could be
figured out by libudev. This might be due to OOM or if the device
disconnected between the udev_device_new() call and
udev_device_get_subsystem(). Therefore, we need to handle subsystem==NULL
safely.

Instead of testing for it in each helper, we treat subsystem==NULL as
empty subsystem in match_subsystem().

Backtrace of udev_enumerate with an input-device disconnecting in exactly
this time-frame:
 (gdb) bt
 #0  0x00007ffff569dc24 in strnlen () from /usr/lib/libc.so.6
 #1  0x00007ffff56d9e04 in fnmatch@@GLIBC_2.2.5 () from /usr/lib/libc.so.6
 #2  0x00007ffff5beb83d in match_subsystem (udev_enumerate=0x7a05f0, subsystem=0x0) at src/libudev/libudev-enumerate.c:727
 #3  0x00007ffff5bebb30 in parent_add_child (enumerate=enumerate@entry=0x7a05f0, path=<optimized out>) at src/libudev/libudev-enumerate.c:834
 #4  0x00007ffff5bebc3f in parent_crawl_children (enumerate=enumerate@entry=0x7a05f0, path=0x7a56b0 "/sys/devices/<shortened>/input/input97", maxdepth=maxdepth@entry=254) at src/libudev/libudev-enumerate.c:866
 #5  0x00007ffff5bebc54 in parent_crawl_children (enumerate=enumerate@entry=0x7a05f0, path=0x79e8c0 "/sys/devices/<shortened>/input", maxdepth=maxdepth@entry=255) at src/libudev/libudev-enumerate.c:868
 #6  0x00007ffff5bebc54 in parent_crawl_children (enumerate=enumerate@entry=0x7a05f0, path=path@entry=0x753190 "/sys/devices/<shortened>", maxdepth=maxdepth@entry=256) at src/libudev/libudev-enumerate.c:868
 #7  0x00007ffff5bec7df in scan_devices_children (enumerate=0x7a05f0) at src/libudev/libudev-enumerate.c:882
 #8  udev_enumerate_scan_devices (udev_enumerate=udev_enumerate@entry=0x7a05f0) at src/libudev/libudev-enumerate.c:919
 #9  0x00007ffff5df8777 in <random_caller> () at some/file.c:181
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment