diff options
author | Robert Haas <rhaas@postgresql.org> | 2023-04-12 11:37:13 -0400 |
---|---|---|
committer | Robert Haas <rhaas@postgresql.org> | 2023-04-12 11:37:13 -0400 |
commit | 8e82db97b0a474008d8212a63f34e449a8c50bcd (patch) | |
tree | 37b14a23d0a4579c2a6a80b7a901460d41c1546c | |
parent | 4b1ad19a4e221897d01fbfe68b89412c8c5a9cbb (diff) | |
download | postgresql-8e82db97b0a474008d8212a63f34e449a8c50bcd.tar.gz postgresql-8e82db97b0a474008d8212a63f34e449a8c50bcd.zip |
basebackup_to_shell: Check for a NULL return from OpenPipeStream.
Per complaint from Peter Eisentraut.
Discussion: http://postgr.es/m/4f1707cc-2432-da35-64a2-5c2a8d92a388@enterprisedb.com
-rw-r--r-- | contrib/basebackup_to_shell/basebackup_to_shell.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/contrib/basebackup_to_shell/basebackup_to_shell.c b/contrib/basebackup_to_shell/basebackup_to_shell.c index 29f5069d427..57ed587d488 100644 --- a/contrib/basebackup_to_shell/basebackup_to_shell.c +++ b/contrib/basebackup_to_shell/basebackup_to_shell.c @@ -263,6 +263,11 @@ shell_run_command(bbsink_shell *sink, const char *filename) /* Run it. */ sink->pipe = OpenPipeStream(sink->current_command, PG_BINARY_W); + if (sink->pipe == NULL) + ereport(ERROR, + (errcode_for_file_access(), + errmsg("could not execute command \"%s\": %m", + sink->current_command))); } /* |