chiark / gitweb /
log: properly return -EINVAL from log_set_max_level_from_string()
authorLennart Poettering <lennart@poettering.net>
Wed, 30 Sep 2015 12:16:40 +0000 (14:16 +0200)
committerSven Eden <yamakuzure@gmx.net>
Wed, 29 Mar 2017 08:45:10 +0000 (10:45 +0200)
If we just return the value we got from log_level_from_string() on
failure we'll return -1, which is not a proper error code.

log_set_target_from_string() did get this right already, hence let's fix
this here too.

src/basic/log.c

index 921c952..69977f1 100644 (file)
@@ -942,7 +942,7 @@ int log_set_max_level_from_string(const char *e) {
 
         t = log_level_from_string(e);
         if (t < 0)
-                return t;
+                return -EINVAL;
 
         log_set_max_level(t);
         return 0;