Revert "Make boot.log world readable by default"

This reverts commit fc5cd88767db61805519fef53182386ba56c6405.

It's better to let the admin of the system decide the permissions,
than forcefully change them.

diff --git a/src/libply/ply-logger.c b/src/libply/ply-logger.c
index 429e1ff1f1e057ae75d49eb8d6c2f4b05d6f0087..1b56ea82a988c16f173b821829024d185b04f54b 100644 (file)
--- a/src/libply/ply-logger.c
+++ b/src/libply/ply-logger.c
@@ -310,8 +310,7 @@ ply_logger_free (ply_logger_t *logger)
 
 bool
 ply_logger_open_file (ply_logger_t *logger,
-                      const char   *filename,
-                      bool          world_readable)
+                      const char   *filename)
 {
         int fd;
         mode_t mode;
@@ -319,21 +318,11 @@ ply_logger_open_file (ply_logger_t *logger,
         assert (logger != NULL);
         assert (filename != NULL);
 
-        if (world_readable)
-                mode = 0644;
-        else
-                mode = 0600;
-
-        fd = open (filename, PLY_LOGGER_OPEN_FLAGS, mode);
+        fd = open (filename, PLY_LOGGER_OPEN_FLAGS, 0600);
 
         if (fd < 0)
                 return false;
 
-        if (fchmod (fd, mode) < 0) {
-                close (fd);
-                return false;
-        }
-
         ply_logger_set_output_fd (logger, fd);
 
         free (logger->filename);

diff --git a/src/libply/ply-logger.h b/src/libply/ply-logger.h
index 545e64cc988cda9316d339af1276d21bce93162d..53d1ad9d29aa120716dac3fd0c9b2451f01008a6 100644 (file)
--- a/src/libply/ply-logger.h
+++ b/src/libply/ply-logger.h
@@ -46,8 +46,7 @@ typedef void (*ply_logger_filter_handler_t) (void         *user_data,
 ply_logger_t *ply_logger_new (void);
 void ply_logger_free (ply_logger_t *logger);
 bool ply_logger_open_file (ply_logger_t *logger,
-                           const char   *filename,
-                           bool          world_readable);
+                           const char   *filename);
 void ply_logger_close_file (ply_logger_t *logger);
 void ply_logger_set_output_fd (ply_logger_t *logger,
                                int           fd);
@@ -108,7 +107,7 @@ bool ply_logger_is_tracing_enabled (ply_logger_t *logger);
 /* convenience macros
  */
 #define ply_open_log_file(filename)                                            \
-        ply_logger_open_file (ply_logger_get_default (), filename, false)
+        ply_logger_open_file (ply_logger_get_default (), filename)
 #define ply_close_log_file()                                                   \
         ply_logger_close_file (ply_logger_get_default ())
 #define ply_flush_log()                                                        \

diff --git a/src/libply/ply-terminal-session.c b/src/libply/ply-terminal-session.c
index a697b8fae1ac33997790b7185c8960f42f07d1c4..7c2cf07892ba127611e6177e06cdd66db44b7cca 100644 (file)
--- a/src/libply/ply-terminal-session.c
+++ b/src/libply/ply-terminal-session.c
@@ -555,7 +555,7 @@ ply_terminal_session_open_log (ply_terminal_session_t *session,
 
         ply_save_errno ();
         unlink (filename);
-        log_is_opened = ply_logger_open_file (session->logger, filename, true);
+        log_is_opened = ply_logger_open_file (session->logger, filename);
         if (log_is_opened)
                 ply_logger_flush (session->logger);
         ply_restore_errno ();