shared/conf-parser: be nice and ignore lines without "="
We generally don't treat syntax error as fatal, but in this case we would completely refuse to load the file. I think we should treat the the same as assignment outside of a section, or an unknown key name.
This commit is contained in:
@@ -239,7 +239,6 @@ static int parse_line(
|
||||
}
|
||||
|
||||
if (sections && !*section) {
|
||||
|
||||
if (!(flags & CONFIG_PARSE_RELAXED) && !*section_ignored)
|
||||
log_syntax(unit, LOG_WARNING, filename, line, 0, "Assignment outside of section. Ignoring.");
|
||||
|
||||
@@ -247,10 +246,9 @@ static int parse_line(
|
||||
}
|
||||
|
||||
e = strchr(l, '=');
|
||||
if (!e) {
|
||||
log_syntax(unit, LOG_WARNING, filename, line, 0, "Missing '='.");
|
||||
return -EINVAL;
|
||||
}
|
||||
if (!e)
|
||||
return log_syntax(unit, LOG_WARNING, filename, line, 0,
|
||||
"Missing '=', ignoring line.");
|
||||
|
||||
*e = 0;
|
||||
e++;
|
||||
|
||||
Reference in New Issue
Block a user