log: properly return -EINVAL from log_set_max_level_from_string()

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.

diff --git a/src/basic/log.c b/src/basic/log.c
index 921c952f833dff4c76e7f8ff0cea554021171065..69977f15d69ba657ea9faa01f7eea45f62e7f0ae 100644 (file)
--- a/src/basic/log.c
+++ b/src/basic/log.c
@@ -942,7 +942,7 @@ int log_set_max_level_from_string(const char *e) {
 
         t = log_level_from_string(e);
         if (t < 0)
 
         t = log_level_from_string(e);
         if (t < 0)

-                return t;
+                return -EINVAL;

 
         log_set_max_level(t);
         return 0;
 
         log_set_max_level(t);
         return 0;