dev-setup: normalize logging around lock_dev_console()

Previously this function would log loudly in some cases but not in
others. Clean this up, and dont log at all, matching our coding style
which says we should either log in all error cases or in none.

Both callers of this function do logging already, hence no need to
duplicate it here.

diff --git a/src/shared/dev-setup.c b/src/shared/dev-setup.c
index 08fdd60570470a66c0b767a30ba3203b2c686e71..3592b0a6505ade49dcde4715f61d275a91769e07 100644 (file)
--- a/src/shared/dev-setup.c
+++ b/src/shared/dev-setup.c
@@ -28,7 +28,7 @@ int lock_dev_console(void) {
 
         r = lock_generic(fd, LOCK_BSD, LOCK_EX);
         if (r < 0)
-                return log_error_errno(r, "Failed to lock /dev/console: %m");
+                return r;
 
         return TAKE_FD(fd);
 }