teknoraver / rpms / systemd

Forked from rpms/systemd 2 months ago
Clone

Blame 0384-delta-warn-if-diff-failed.patch

Zbigniew Jędrzejewski-Szmek 62fe94
From 3d1b90bd7fb562fdb3d15e0d7750ae0c36bc15b6 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 62fe94
From: Tom Gundersen <teg@jklm.no>
Zbigniew Jędrzejewski-Szmek 62fe94
Date: Thu, 25 Sep 2014 18:16:04 +0200
Zbigniew Jędrzejewski-Szmek 62fe94
Subject: [PATCH] delta: warn if diff failed
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
Found by Coverity. Fixes CID #1237541.
Zbigniew Jędrzejewski-Szmek 62fe94
---
Zbigniew Jędrzejewski-Szmek 62fe94
 src/delta/delta.c | 6 ++++--
Zbigniew Jędrzejewski-Szmek 62fe94
 1 file changed, 4 insertions(+), 2 deletions(-)
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
diff --git a/src/delta/delta.c b/src/delta/delta.c
Zbigniew Jędrzejewski-Szmek 62fe94
index 91f8592b40..2fdbeeae81 100644
Zbigniew Jędrzejewski-Szmek 62fe94
--- a/src/delta/delta.c
Zbigniew Jędrzejewski-Szmek 62fe94
+++ b/src/delta/delta.c
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -159,7 +159,7 @@ static int notify_override_unchanged(const char *f) {
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
 static int found_override(const char *top, const char *bottom) {
Zbigniew Jędrzejewski-Szmek 62fe94
         _cleanup_free_ char *dest = NULL;
Zbigniew Jędrzejewski-Szmek 62fe94
-        int k;
Zbigniew Jędrzejewski-Szmek 62fe94
+        int k, r;
Zbigniew Jędrzejewski-Szmek 62fe94
         pid_t pid;
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         assert(top);
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -194,7 +194,9 @@ static int found_override(const char *top, const char *bottom) {
Zbigniew Jędrzejewski-Szmek 62fe94
                 _exit(1);
Zbigniew Jędrzejewski-Szmek 62fe94
         }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        wait_for_terminate(pid, NULL);
Zbigniew Jędrzejewski-Szmek 62fe94
+        r = wait_for_terminate(pid, NULL);
Zbigniew Jędrzejewski-Szmek 62fe94
+        if (r < 0)
Zbigniew Jędrzejewski-Szmek 62fe94
+                log_warning("Failed to wait for diff: %s", strerror(-r));
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         putchar('\n');
Zbigniew Jędrzejewski-Szmek 62fe94