diff options
author | Denys Vlasenko | 2018-11-27 11:26:48 +0100 |
---|---|---|
committer | Denys Vlasenko | 2018-11-27 11:26:48 +0100 |
commit | f4fc303e3679e4ab0d45f60c31f9b687f27f7452 (patch) | |
tree | c270be417b97696e3a9897eab5602ce30faf313d /modutils | |
parent | 2d217799e8f23514ad3be6a951aa928c265bc6a4 (diff) | |
download | busybox-f4fc303e3679e4ab0d45f60c31f9b687f27f7452.zip busybox-f4fc303e3679e4ab0d45f60c31f9b687f27f7452.tar.gz |
tar: fix too eager autodetection, closes 11531
function old new delta
is_suffixed_with - 54 +54
tar_main 1006 1026 +20
open_transformer 92 79 -13
config_file_action 478 458 -20
------------------------------------------------------------------------------
(add/remove: 1/0 grow/shrink: 1/2 up/down: 74/-33) Total: 41 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'modutils')
-rw-r--r-- | modutils/modprobe.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/modutils/modprobe.c b/modutils/modprobe.c index 59f6d54..291e4cb 100644 --- a/modutils/modprobe.c +++ b/modutils/modprobe.c @@ -245,7 +245,7 @@ static int FAST_FUNC config_file_action(const char *filename, parser_t *p; struct module_entry *m; int rc = TRUE; - const char *base, *ext; + const char *base; /* Skip files that begin with a "." */ base = bb_basename(filename); @@ -266,8 +266,7 @@ static int FAST_FUNC config_file_action(const char *filename, * "include FILE_NOT_ENDING_IN_CONF" must work too. */ if (depth != 0) { - ext = strrchr(base, '.'); - if (ext == NULL || strcmp(ext + 1, "conf")) + if (!is_suffixed_with(base, ".conf")) goto error; } |