tmpfiles: make sure not to concatenate non-absolute path
If the path is absolute was only checked later. Also do not check if path if absolute if we just specified it starting with a slash.
This commit is contained in:
@@ -1321,7 +1321,7 @@ static int parse_line(const char *fname, unsigned line, const char *buffer) {
|
||||
|
||||
case CREATE_SYMLINK:
|
||||
if (!i->argument) {
|
||||
i->argument = strappend("/usr/share/factory", i->path);
|
||||
i->argument = strappend("/usr/share/factory/", i->path);
|
||||
if (!i->argument)
|
||||
return log_oom();
|
||||
}
|
||||
@@ -1336,12 +1336,10 @@ static int parse_line(const char *fname, unsigned line, const char *buffer) {
|
||||
|
||||
case COPY_FILES:
|
||||
if (!i->argument) {
|
||||
i->argument = strappend("/usr/share/factory", i->path);
|
||||
i->argument = strappend("/usr/share/factory/", i->path);
|
||||
if (!i->argument)
|
||||
return log_oom();
|
||||
}
|
||||
|
||||
if (!path_is_absolute(i->argument)) {
|
||||
} else if (!path_is_absolute(i->argument)) {
|
||||
log_error("[%s:%u] Source path is not absolute.", fname, line);
|
||||
return -EBADMSG;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user