[libvirt] [PATCH] virFork: placate static analyzers: ignore pthread_sigmask return value

Jim Meyering jim at meyering.net
Fri Feb 19 17:41:51 UTC 2010


Since we check all other uses of pthread_sigmask, be consistent
and clear: mark that we're deliberately ignoring this one.

>From 0cb4b2ffa51bb8c73243876221bff6d7edecd9cf Mon Sep 17 00:00:00 2001
From: Jim Meyering <meyering at redhat.com>
Date: Fri, 19 Feb 2010 18:40:14 +0100
Subject: [PATCH] virFork: placate static analyzers: ignore pthread_sigmask return value

* src/util/util.c: Include "ignore-value.h".
(virFork): We really do want to ignore pthread_sigmask failure.
---
 src/util/util.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/src/util/util.c b/src/util/util.c
index 624e570..cf7bba5 100644
--- a/src/util/util.c
+++ b/src/util/util.c
@@ -69,6 +69,7 @@
 #include "virterror_internal.h"
 #include "logging.h"
 #include "event.h"
+#include "ignore-value.h"
 #include "buf.h"
 #include "util.h"
 #include "memory.h"
@@ -344,7 +345,7 @@ int virFork(pid_t *pid) {
     if (*pid < 0) {
         /* attempt to restore signal mask, but ignore failure, to
            avoid obscuring the fork failure */
-        pthread_sigmask(SIG_SETMASK, &oldmask, NULL);
+        ignore_value (pthread_sigmask(SIG_SETMASK, &oldmask, NULL));
         virReportSystemError(saved_errno,
                              "%s", _("cannot fork child process"));
         goto cleanup;
--
1.7.0.233.g05e1a




More information about the libvir-list mailing list