aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Grothoff <christian@grothoff.org>2021-08-09 15:28:56 +0200
committerChristian Grothoff <christian@grothoff.org>2021-08-09 15:28:56 +0200
commit2c2cc95e7ad9c315374068aa0b3d3dca4ede928f (patch)
tree93e8b4b9ba13f848a93591c32784d9d7eb91929b
parent6db56797ae56332504477ef4185ea3a53cc59c80 (diff)
downloadgnunet-2c2cc95e7ad9c315374068aa0b3d3dca4ede928f.tar.gz
gnunet-2c2cc95e7ad9c315374068aa0b3d3dca4ede928f.zip
-check return values
-rw-r--r--src/util/os_priority.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/src/util/os_priority.c b/src/util/os_priority.c
index 1ed9bcbf7..dc2f0f97e 100644
--- a/src/util/os_priority.c
+++ b/src/util/os_priority.c
@@ -316,7 +316,7 @@ open_dev_null (int target_fd, int flags)
316 if (-1 == dup2 (fd, target_fd)) 316 if (-1 == dup2 (fd, target_fd))
317 { 317 {
318 GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "dup2"); 318 GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "dup2");
319 (void) close (fd); 319 GNUNET_break (0 == close (fd));
320 return; 320 return;
321 } 321 }
322 GNUNET_break (0 == close (fd)); 322 GNUNET_break (0 == close (fd));
@@ -395,7 +395,7 @@ start_process (enum GNUNET_OS_InheritStdioFlags std_inheritance,
395 if (NULL != childpipe_write) 395 if (NULL != childpipe_write)
396 GNUNET_DISK_file_close (childpipe_write); 396 GNUNET_DISK_file_close (childpipe_write);
397 if (0 <= dup_childpipe_read_fd) 397 if (0 <= dup_childpipe_read_fd)
398 close (dup_childpipe_read_fd); 398 GNUNET_break (0 == close (dup_childpipe_read_fd));
399 return NULL; 399 return NULL;
400 } 400 }
401 childpipe_read_fd = dup_childpipe_read_fd; 401 childpipe_read_fd = dup_childpipe_read_fd;
@@ -474,7 +474,7 @@ start_process (enum GNUNET_OS_InheritStdioFlags std_inheritance,
474 if (NULL != childpipe_write) 474 if (NULL != childpipe_write)
475 GNUNET_DISK_file_close (childpipe_write); 475 GNUNET_DISK_file_close (childpipe_write);
476 if (0 <= childpipe_read_fd) 476 if (0 <= childpipe_read_fd)
477 close (childpipe_read_fd); 477 GNUNET_break (0 == close (childpipe_read_fd));
478 errno = eno; 478 errno = eno;
479 return NULL; 479 return NULL;
480 } 480 }
@@ -486,7 +486,7 @@ start_process (enum GNUNET_OS_InheritStdioFlags std_inheritance,
486 gnunet_proc->control_pipe = childpipe_write; 486 gnunet_proc->control_pipe = childpipe_write;
487 if (0 != (std_inheritance & GNUNET_OS_USE_PIPE_CONTROL)) 487 if (0 != (std_inheritance & GNUNET_OS_USE_PIPE_CONTROL))
488 { 488 {
489 close (childpipe_read_fd); 489 GNUNET_break (0 == close (childpipe_read_fd));
490 } 490 }
491 GNUNET_array_grow (lscp, ls, 0); 491 GNUNET_array_grow (lscp, ls, 0);
492 return gnunet_proc; 492 return gnunet_proc;
@@ -564,7 +564,7 @@ start_process (enum GNUNET_OS_InheritStdioFlags std_inheritance,
564 { 564 {
565 /* Bury any existing FD, no matter what; they should all be closed 565 /* Bury any existing FD, no matter what; they should all be closed
566 * on exec anyway and the important ones have been dup'ed away */ 566 * on exec anyway and the important ones have been dup'ed away */
567 (void) close (tgt); 567 GNUNET_break (0 == close (tgt));
568 GNUNET_assert (-1 != dup2 (lscp[i], tgt)); 568 GNUNET_assert (-1 != dup2 (lscp[i], tgt));
569 } 569 }
570 /* unset close-on-exec flag */ 570 /* unset close-on-exec flag */