shaba/systemd
1
0
Fork 0
mirror of https://github.com/systemd/systemd.git synced 2025-02-06 01:57:47 +03:00
Code

sd-device: filter regular files when enumerating

Browse Source 
Currently if enumerating with debug logging you'll likely see something
like this:

sd-device: the syspath "/sys/class/devcoredump/disabled" is not a directory.
sd-device: the syspath "/sys/class/firmware/timeout" is not a directory.
sd-device: the syspath "/sys/class/zram-control/hot_remove" is not a directory.
sd-device: the syspath "/sys/class/zram-control/hot_add" is not a directory.
sd-device: the syspath "/sys/class/drm/version" is not a directory.

This is because these sysfs classes place regular files in these
directories, which we so far didn't expect.

Let's filter them early, and only bother with enumerated inodes if they
actually are dirs or symlinks, i.e. can be kobject dirs. Regular file
definitely never are kobject dirs...
This commit is contained in:
Lennart Poettering 2022-04-12 11:25:00 +02:00
parent e7c7d79d52
commit 29a5428f1c
1 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/libsystemd/sd-device/device-enumerator.c
View File

@ -577,6 +577,17 @@ static int test_matches(
return true;
}
static bool relevant_sysfs_subdir(const struct dirent *de) {
assert(de);
if (de->d_name[0] == '.')
return false;
/* Also filter out regular files and such, i.e. stuff that definitely isn't a kobject path. (Note
* that we rely on the fact that sysfs fills in d_type here, i.e. doesn't do DT_UNKNOWN) */
return IN_SET(de->d_type, DT_DIR, DT_LNK);
}
static int enumerator_scan_dir_and_add_devices(
sd_device_enumerator *enumerator,
const char *basedir,
@ -607,7 +618,7 @@ static int enumerator_scan_dir_and_add_devices(
_cleanup_(sd_device_unrefp) sd_device *device = NULL;
char syspath[strlen(path) + 1 + strlen(de->d_name) + 1];
if (de->d_name[0] == '.')
if (!relevant_sysfs_subdir(de))
continue;
if (!match_sysname(enumerator, de->d_name))
@ -682,7 +693,7 @@ static int enumerator_scan_dir(
FOREACH_DIRENT_ALL(de, dir, return -errno) {
int k;
if (de->d_name[0] == '.')
if (!relevant_sysfs_subdir(de))
continue;
if (!match_subsystem(enumerator, subsystem ? : de->d_name))