diff options
-rw-r--r-- | libkmod/libkmod-config.c | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/libkmod/libkmod-config.c b/libkmod/libkmod-config.c index 57fbe37..19f56a7 100644 --- a/libkmod/libkmod-config.c +++ b/libkmod/libkmod-config.c @@ -844,15 +844,20 @@ int kmod_config_new(struct kmod_ctx *ctx, struct kmod_config **p_config, config->ctx = ctx; for (; list != NULL; list = kmod_list_remove(list)) { - char fn[PATH_MAX]; + char buf[PATH_MAX]; + const char *fn = buf; struct conf_file *cf = list->data; int fd; - if (cf->is_single) - strcpy(fn, cf->path); - else - snprintf(fn, sizeof(fn),"%s/%s", cf->path, - cf->name); + if (cf->is_single) { + fn = cf->path; + } else if (snprintf(buf, sizeof(buf), "%s/%s", + cf->path, cf->name) >= (int)sizeof(buf)) { + ERR(ctx, "Error parsing %s/%s: path too long\n", + cf->path, cf->name); + free(cf); + continue; + } fd = open(fn, O_RDONLY|O_CLOEXEC); DBG(ctx, "parsing file '%s' fd=%d\n", fn, fd); |