index d30ec6084179a614fcf386cf87e3a9dcad9e0079..6c3114d8e692274d8223c7601c3082d5174b5fe2 100644 (file)
static void
data_printer(sdb_strbuf_t *buf)
{
- if (sdb_strbuf_len(buf) <= sizeof(uint32_t)) {
+ size_t len = sdb_strbuf_len(buf);
+
+ if ((! len) || (len == sizeof(uint32_t))) {
+ /* empty command or empty reply */
+ return;
+ }
+ else if (len < sizeof(uint32_t)) {
printf("ERROR: Received a DATA message with invalid "
"or missing data-type\n");
return;
int status;
void (*printer)(sdb_strbuf_t *);
} response_printers[] = {
- { CONNECTION_LOG, log_printer },
- { CONNECTION_DATA, data_printer },
+ { SDB_CONNECTION_LOG, log_printer },
+ { SDB_CONNECTION_DATA, data_printer },
};
+static void
+clear_query(sdb_input_t *input)
+{
+ sdb_strbuf_skip(input->input, 0, input->query_len);
+ input->tokenizer_pos -= input->query_len;
+ input->query_len = 0;
+ input->have_input = 0;
+} /* clear_query */
+
/*
* public API
*/
if (query_len) {
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);
-
- /* The server will send back *something*, either error/log messages
- * and/or the reply to the query. Here, we don't care about what it
- * sends back. We'll wait for the first reply and then return to the
- * main loop which will handle any subsequent replies, including
- * eventually the reply to the query (if it's not the first reply). */
- if (sdb_command_print_reply(input->client) < 0) {
- if (data)
- free(data);
- return NULL;
+ if (sdb_client_eof(input->client)) {
+ if (sdb_input_reconnect()) {
+ clear_query(input);
+ return data;
}
}
-
- sdb_strbuf_skip(input->input, 0, input->query_len);
- input->tokenizer_pos -= input->query_len;
- input->query_len = 0;
+ else if (! query_len)
+ return NULL;
+
+ sdb_client_send(input->client, SDB_CONNECTION_QUERY, query_len, query);
+
+ /* The server will send back *something*, either error/log messages
+ * and/or the reply to the query. Here, we don't care about what it
+ * sends back. We'll wait for the first reply and then return to the
+ * main loop which will handle any subsequent replies, including
+ * eventually the reply to the query (if it's not the first reply). */
+ sdb_command_print_reply(input->client);
+ clear_query(input);
return data;
} /* sdb_command_exec */