[PATCH cluster 03/12] pmxcfs: server: simplify non-negative ipc message result responses

[Daniel Kral <d.kral@proxmox.com>]

The result variable is used as the error code in the QB response header,
where libqb and PVE::IPCC expects it to be 0 for successful responses.

The request handler for CFS_IPC_GET_CONFIG is the last remaining handler
to set result to a non-negative value: it does set result to the
returned config file size in case of a successful memdb_read() request.

Therefore, set result to 0 in case of a successful read operation and
remove the now unnecessary catch-all zero assignment when handling
successful responses.

No functional changes intended.

Signed-off-by: Daniel Kral <d.kral@proxmox.com>
---
 src/pmxcfs/server.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/pmxcfs/server.c b/src/pmxcfs/server.c
index 278e2c9..22e812c 100644
--- a/src/pmxcfs/server.c
+++ b/src/pmxcfs/server.c
@@ -251,6 +251,7 @@ static int32_t s1_msg_process_fn(qb_ipcs_connection_t *c, void *data, size_t siz
                 if (result > 0) {
                     g_string_append_len(outbuf, tmp, result);
                     g_free(tmp);
+                    result = 0;
                 }
             }
         }
@@ -452,7 +453,6 @@ static int32_t s1_msg_process_fn(qb_ipcs_connection_t *c, void *data, size_t siz
 
     if (result >= 0) {
         resp = outbuf->str;
-        result = 0;
     }
 
     int iov_len = 2;
-- 
2.47.3