[89242] trunk/base/src/programs/daemondo/main.c

[raimue at macports.org]

Revision: 89242
          http://trac.macports.org/changeset/89242
Author:   raimue at macports.org
Date:     2012-01-21 15:18:56 -0800 (Sat, 21 Jan 2012)
Log Message:
-----------
daemondo:
Found with clang analyzer: "Value stored to 'status' is never read"
Write a log message if mach_msg_send fails.

Modified Paths:
--------------
    trunk/base/src/programs/daemondo/main.c

Modified: trunk/base/src/programs/daemondo/main.c
===================================================================
--- trunk/base/src/programs/daemondo/main.c	2012-01-21 23:18:36 UTC (rev 89241)
+++ trunk/base/src/programs/daemondo/main.c	2012-01-21 23:18:56 UTC (rev 89242)
@@ -921,7 +921,9 @@
     header.msgh_id          = sig;
     
     mach_msg_return_t status = mach_msg_send(&header);
-    status = 0;
+    if (status != 0) {
+        LogMessage("mach_msg_send failed in handle_child_signal!\n");
+    }
 }
 
 
@@ -939,7 +941,9 @@
     header.msgh_id          = sig;
     
     mach_msg_return_t status = mach_msg_send(&header);
-    status = 0;
+    if (status != 0) {
+        LogMessage("mach_msg_send failed in handle_generic_signal!\n");
+    }
 }
 
 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macports-changes/attachments/20120121/5c923ef7/attachment.html>