[systemd-commits] src/fsckd

[Lennart Poettering]

 src/fsckd/fsckd.c |   11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

New commits:
commit 54bcf1557c30d97d5ac0afaba003f69e58f82376
Author: Didier Roche <didrocks at ubuntu.com>
Date:   Tue Mar 10 10:18:00 2015 +0100

    fsckd: clean up log messages
    
    Avoid double logs printing. Not that we don't return
    manager_update_global_progress() to the handler callback as if the console or
    plymouth isn't available momentarily, we still desire to handle future
    fd progress events if those are available again (like cancellation, reports…)

diff --git a/src/fsckd/fsckd.c b/src/fsckd/fsckd.c
index 91b8df0..3c587a3 100644
--- a/src/fsckd/fsckd.c
+++ b/src/fsckd/fsckd.c
@@ -348,8 +348,7 @@ static int manager_update_global_progress(Manager *m) {
                 /* try to connect to plymouth and send message */
                 r = manager_send_plymouth_message(m, fsck_message);
                 if (r < 0)
-                        log_debug("Couldn't send message to plymouth");
-
+                        return r;
         }
         return 0;
 }
@@ -378,9 +377,7 @@ static int client_progress_handler(sd_event_source *s, int fd, uint32_t revents,
                 else {
                         log_warning("Closing bad behaving fsck client connection at fd %d", client->fd);
                         client_free(client);
-                        r = manager_update_global_progress(m);
-                        if (r < 0)
-                                log_warning_errno(r, "Couldn't update global progress: %m");
+                        manager_update_global_progress(m);
                 }
                 return 0;
         }
@@ -404,9 +401,7 @@ static int client_progress_handler(sd_event_source *s, int fd, uint32_t revents,
         } else
                 log_error_errno(r, "Unknown error while trying to read fsck data: %m");
 
-        r = manager_update_global_progress(m);
-        if (r < 0)
-                log_warning_errno(r, "Couldn't update global progress: %m");
+        manager_update_global_progress(m);
 
         return 0;
 }