View Issue Details

IDProjectCategoryView StatusLast Update
0001571GNUnetutil librarypublic2011-10-05 22:22
ReporterLRN Assigned ToNDurner  
PrioritynormalSeveritytweakReproducibilityalways
Status closedResolutionfixed 
Summary0001571: A signal() could use a better error check
DescriptionPatch attached (just a general idea, feel free to adjust/expand).
TagsNo tags attached.
Attached Files
win32_signal_tweak.diff (599 bytes)   
Index: src/util/signal.c
===================================================================
--- src/util/signal.c	(revision 11817)
+++ src/util/signal.c	(working copy)
@@ -67,7 +67,16 @@
   if (signum == GNUNET_SIGCHLD)
     w32_sigchld_handler = handler;
   else
-    signal (signum, handler);
+  {
+    __p_sig_fn_t sigret = signal (signum, handler);
+    if (sigret == SIG_ERR)
+    {
+      GNUNET_log (GNUNET_ERROR_TYPE_WARNING,
+                  _
+                  ("signal (%d, %p) returned %d.\n"),
+                  signum, handler, sigret);
+    }
+  }
 #endif
   return ret;
 }
win32_signal_tweak.diff (599 bytes)   

Activities

NDurner

2010-06-21 22:05

developer   ~0004042

Applied, thanks

Issue History

Date Modified Username Field Change
2010-06-20 18:35 LRN New Issue
2010-06-20 18:35 LRN File Added: win32_signal_tweak.diff
2010-06-20 18:44 Christian Grothoff Project gnunet-qt => GNUnet
2010-06-21 22:05 NDurner Note Added: 0004042
2010-06-21 22:05 NDurner Status new => resolved
2010-06-21 22:05 NDurner Resolution open => fixed
2010-06-21 22:05 NDurner Assigned To => NDurner
2010-07-03 20:55 Christian Grothoff Status resolved => closed
2011-10-05 22:22 Christian Grothoff Category => util library