confile: cleanup set_config_console_rotate()

Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>

diff --git a/src/lxc/confile.c b/src/lxc/confile.c
index 2f04cedf0d18eb01d11c1d4729ef0500e1db6482..f65ac70836d4b53e8c9f711091e568e17da7fbd0 100644 (file)
--- a/src/lxc/confile.c
+++ b/src/lxc/confile.c
@@ -2308,19 +2308,19 @@ static int set_config_console_path(const char *key, const char *value,
 static int set_config_console_rotate(const char *key, const char *value,
                                     struct lxc_conf *lxc_conf, void *data)
 {
+       int ret;
+
        if (lxc_config_value_empty(value)) {
                lxc_conf->console.log_rotate = 0;
                return 0;
        }
 
-       if (lxc_safe_uint(value, &lxc_conf->console.log_rotate) < 0)
-               return -1;
+       ret = lxc_safe_uint(value, &lxc_conf->console.log_rotate);
+       if (ret)
+               return ret_errno(EINVAL);
 
-       if (lxc_conf->console.log_rotate > 1) {
-               ERROR("The \"lxc.console.rotate\" config key can only be set "
-                     "to 0 or 1");
-               return -1;
-       }
+       if (lxc_conf->console.log_rotate > 1)
+               return log_error_errno(-EINVAL, EINVAL, "The \"lxc.console.rotate\" config key can only be set to 0 or 1");
 
        return 0;
 }