lib-program-client: remote: Don't change exit_code in program_client_remote_disconnec...

When the program_input is NULL, the stream is finished, meaning that the
exit_code is set based on the return code. There can be a program_input for
remote streams, even when the program produces no output.

diff --git a/src/lib-program-client/program-client-remote.c b/src/lib-program-client/program-client-remote.c
index 56707e4040d50f98674db1e820e1081dc8fbefb7..250538a665ab9a1313b8c235df66c60818a1adff 100644 (file)
--- a/src/lib-program-client/program-client-remote.c
+++ b/src/lib-program-client/program-client-remote.c
@@ -544,8 +544,10 @@ void program_client_remote_disconnect(struct program_client *pclient, bool force
 
        timeout_remove(&prclient->to_retry);
 
-       if (pclient->error == PROGRAM_CLIENT_ERROR_NONE && !prclient->noreply &&
-           pclient->program_input != NULL && !force) {
+       if (pclient->program_input == NULL) {
+               /* nothing */
+       } else if (pclient->error == PROGRAM_CLIENT_ERROR_NONE &&
+                  !prclient->noreply && !force) {
                const unsigned char *data;
                size_t size;