diff options
author | Michal Schmidt <mschmidt@redhat.com> | 2012-10-30 10:29:40 +0100 |
---|---|---|
committer | Michal Schmidt <mschmidt@redhat.com> | 2012-10-30 10:30:04 +0100 |
commit | f3910003bce32ebdc1dbb71fd9ca2d4b8352b563 (patch) | |
tree | 86d7662a2c26332b67add144ad20e63abdbd1aa1 /src/shared/conf-parser.c | |
parent | 0eb59ccfe619cbc4b42ef8ff02b52971994dfe05 (diff) |
shared, libsystemd-daemon: check for empty strings in strto*l conversions
strtol() and friends may set EINVAL if no conversion was performed, but
they are not required to do so. In practice they don't. We need to check
for it.
https://bugzilla.redhat.com/show_bug.cgi?id=870577
Diffstat (limited to 'src/shared/conf-parser.c')
-rw-r--r-- | src/shared/conf-parser.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/shared/conf-parser.c b/src/shared/conf-parser.c index 4bf3147f2d..9f5c07c761 100644 --- a/src/shared/conf-parser.c +++ b/src/shared/conf-parser.c @@ -865,7 +865,7 @@ int config_parse_mode( errno = 0; l = strtol(rvalue, &x, 8); - if (!x || *x || errno) { + if (!x || x == rvalue || *x || errno) { log_error("[%s:%u] Failed to parse mode value, ignoring: %s", filename, line, rvalue); return 0; } |