Only collect remote command output if the caller requires it

This addresses GitHub #216 and #167.
This commit is contained in:
Gianni Ciolli
2016-08-04 20:59:47 +02:00
committed by Ian Barwick
parent c88ea62643
commit 59b1924d5b

View File

@@ -3453,7 +3453,7 @@ do_standby_switchover(void)
remote_host,
runtime_options.remote_user,
command,
&command_output);
NULL);
termPQExpBuffer(&command_output);
@@ -5691,15 +5691,18 @@ remote_command(const char *host, const char *user, const char *command, PQExpBuf
return false;
}
/* TODO: better error handling */
while (fgets(output, MAXLEN, fp) != NULL)
if (outputbuf != NULL)
{
appendPQExpBuffer(outputbuf, "%s", output);
/* TODO: better error handling */
while (fgets(output, MAXLEN, fp) != NULL)
{
appendPQExpBuffer(outputbuf, "%s", output);
}
}
pclose(fp);
log_verbose(LOG_DEBUG, "remote_command(): output returned was:\n%s", outputbuf->data);
if (outputbuf != NULL)
log_verbose(LOG_DEBUG, "remote_command(): output returned was:\n%s", outputbuf->data);
return true;
}