diff options
| author | Daniel J Walsh <dwalsh@redhat.com> | 2019-08-07 08:01:33 -0400 |
|---|---|---|
| committer | Daniel J Walsh <dwalsh@redhat.com> | 2019-08-09 15:33:16 -0400 |
| commit | 44126969f1b0fee257b79f0f3ba9ba74e1da1459 (patch) | |
| tree | 05a57d5ee475c741aebcf57634fc46c1747058ab | |
| parent | d05798e5e8efcb1acdd8794edfa0f82bf5f01045 (diff) | |
| download | podman-44126969f1b0fee257b79f0f3ba9ba74e1da1459.tar.gz podman-44126969f1b0fee257b79f0f3ba9ba74e1da1459.tar.bz2 podman-44126969f1b0fee257b79f0f3ba9ba74e1da1459.zip | |
Fix a couple of errors descovered by coverity
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
| -rw-r--r-- | pkg/rootless/rootless_linux.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/pkg/rootless/rootless_linux.c b/pkg/rootless/rootless_linux.c index 19b76f387..f5d7e4711 100644 --- a/pkg/rootless/rootless_linux.c +++ b/pkg/rootless/rootless_linux.c @@ -457,6 +457,11 @@ create_pause_process (const char *pause_pid_file_path, char **argv) } r = TEMP_FAILURE_RETRY (write (p[1], "0", 1)); + if (r < 0) + { + fprintf (stderr, "cannot write to pipe: %s\n", strerror (errno)); + _exit (EXIT_FAILURE); + } close (p[1]); _exit (EXIT_SUCCESS); @@ -811,6 +816,11 @@ reexec_in_user_namespace (int ready, char *pause_pid_file_path, char *file_to_re } ret = TEMP_FAILURE_RETRY (write (ready, "0", 1)); + if (ret < 0) + { + fprintf (stderr, "cannot write to ready pipe: %s\n", strerror (errno)); + _exit (EXIT_FAILURE); + } close (ready); if (sigprocmask (SIG_SETMASK, &oldsigset, NULL) < 0) |