diff options
| author | Mike Frysinger | 2016-02-12 23:26:51 -0500 |
|---|---|---|
| committer | Mike Frysinger | 2016-02-12 23:26:51 -0500 |
| commit | 3a5cc989025eefe03fda0552b253a4a8f015a761 (patch) | |
| tree | 30a9aff158adbf1792a23de9aa229b80ff3c5568 | |
| parent | 43e56639c6739953d5a6686823bcd0d256512ea5 (diff) | |
| download | busybox-3a5cc989025eefe03fda0552b253a4a8f015a761.zip busybox-3a5cc989025eefe03fda0552b253a4a8f015a761.tar.gz | |
modprobe: only parse files that end in .conf
This matches behavior with kmod which has been the standard for a long
time at this point.
URL: https://bugs.busybox.net/8021
Reported-by: Jö <jorrit@jorrit.de>
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
| -rw-r--r-- | modutils/modprobe.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/modutils/modprobe.c b/modutils/modprobe.c index ec490b7..997ee3c 100644 --- a/modutils/modprobe.c +++ b/modutils/modprobe.c @@ -220,8 +220,16 @@ static int FAST_FUNC config_file_action(const char *filename, parser_t *p; struct module_entry *m; int rc = TRUE; + const char *base, *ext; - if (bb_basename(filename)[0] == '.') + /* Skip files that begin with a ".". */ + base = bb_basename(filename); + if (base[0] == '.') + goto error; + + /* Skip files that do not end with a ".conf". */ + ext = strrchr(base, '.'); + if (ext == NULL || strcmp(ext + 1, "conf")) goto error; p = config_open2(filename, fopen_for_read); |