sysdb: When using -c, exit non-zero if any command failed.

[sysdb.git] / src / tools / sysdb / command.c

diff --git a/src/tools/sysdb/command.c b/src/tools/sysdb/command.c
index 3b64def385ebd64251bbd07dc4f79fe500353da8..77f3c8346498c430be10fcbafcf1ab9a06ee5cc2 100644 (file)
--- a/src/tools/sysdb/command.c
+++ b/src/tools/sysdb/command.c
@@ -33,58 +33,90 @@
 #include "tools/sysdb/input.h"
 
 #include "frontend/proto.h"
+#include "utils/error.h"
 #include "utils/strbuf.h"
 
+#include <errno.h>
+
 #include <assert.h>
 #include <ctype.h>
+#include <string.h>
 
 /*
  * public API
  */
 
 int
+sdb_command_print_reply(sdb_client_t *client)
+{
+       sdb_strbuf_t *recv_buf;
+       const char *result;
+       uint32_t rcode = 0;
+       int status = 0;
+
+       recv_buf = sdb_strbuf_create(1024);
+       if (! recv_buf)
+               return -1;
+
+       if (sdb_client_recv(client, &rcode, recv_buf) < 0) {
+               rcode = UINT32_MAX;
+               status = -1;
+       }
+
+       if (sdb_client_eof(client))
+               return -1;
+
+       if (rcode == CONNECTION_ERROR)
+               status = 1;
+
+       if (rcode == UINT32_MAX)
+               printf("ERROR: ");
+       result = sdb_strbuf_string(recv_buf);
+       if (result && *result)
+               printf("%s\n", result);
+       else if (rcode == UINT32_MAX) {
+               char errbuf[1024];
+               printf("%s\n", sdb_strerror(errno, errbuf, sizeof(errbuf)));
+       }
+
+       sdb_strbuf_destroy(recv_buf);
+       return status;
+} /* sdb_command_print_reply */
+
+char *
 sdb_command_exec(sdb_input_t *input)
 {
        const char *query;
        uint32_t query_len;
 
+       char *data = NULL;
+
        query = sdb_strbuf_string(input->input);
        query_len = (uint32_t)input->query_len;
 
        assert(input->query_len <= input->tokenizer_pos);
 
-       /* removing leading and trailing whitespace */
-       while (isspace((int)*query) && query_len) {
+       /* removing leading and trailing newlines */
+       while (query_len && (*query == '\n')) {
                ++query;
                --query_len;
        }
-       while (query_len && (isspace((int)query[query_len - 1])
-                               || (query[query_len - 1] == ';')))
+       while (query_len && (query[query_len - 1]) == '\n')
                --query_len;
 
        if (query_len) {
-               sdb_strbuf_t *recv_buf;
-               uint32_t rcode = 0;
-
-               recv_buf = sdb_strbuf_create(1024);
-               if (! recv_buf)
-                       return -1;
+               data = strndup(query, query_len);
+               /* ignore errors; we'll only hide the command from the caller */
 
                sdb_client_send(input->client, CONNECTION_QUERY, query_len, query);
-               if (sdb_client_recv(input->client, &rcode, recv_buf) < 0)
-                       rcode = UINT32_MAX;
-
-               if (rcode == UINT32_MAX)
-                       printf("ERROR: ");
-               printf("%s\n", sdb_strbuf_string(recv_buf));
-
-               sdb_strbuf_destroy(recv_buf);
+               if (sdb_command_print_reply(input->client) < 0)
+                       return NULL;
        }
 
        sdb_strbuf_skip(input->input, 0, input->query_len);
        input->tokenizer_pos -= input->query_len;
        input->query_len = 0;
-       return 0;
+       return data;
 } /* sdb_command_exec */
 
 /* vim: set tw=78 sw=4 ts=4 noexpandtab : */