X-Git-Url: https://git.tokkee.org/?a=blobdiff_plain;f=src%2Futils%2Funixsock.c;h=59bf5631f451e4f401883532e8cd9b1dc4c8791d;hb=65ae9e277cf77952608dc05b7cf52db535a2b90d;hp=b4b9772a0e9dbe1684abed2e7af3d8b9fad8fc06;hpb=a6bb88e99588be4c62aa7a81bfae643b3c1888fa;p=sysdb.git diff --git a/src/utils/unixsock.c b/src/utils/unixsock.c index b4b9772..59bf563 100644 --- a/src/utils/unixsock.c +++ b/src/utils/unixsock.c @@ -25,9 +25,8 @@ * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include "utils/unixsock.h" #include "utils/error.h" -#include "utils/string.h" +#include "utils/unixsock.h" #include #include @@ -117,8 +116,8 @@ sdb_unixsock_parse_cell(char *string, int type, sdb_data_t *data) data->data.binary.datum = (const unsigned char *)string; break; default: - sdb_error_set(SDB_LOG_ERR, "unixsock: Unexpected type %i while " - "parsing query result.\n", type); + sdb_log(SDB_LOG_ERR, "unixsock: Unexpected type %i while " + "parsing query result.", type); return -1; } @@ -126,14 +125,14 @@ sdb_unixsock_parse_cell(char *string, int type, sdb_data_t *data) || (type == SDB_TYPE_DATETIME)) { if (errno || (string == endptr)) { char errbuf[1024]; - sdb_error_set(SDB_LOG_ERR, "unixsock: Failed to parse string " - "'%s' as numeric value (type %i): %s\n", string, type, + sdb_log(SDB_LOG_ERR, "unixsock: Failed to parse string " + "'%s' as numeric value (type %i): %s", string, type, sdb_strerror(errno, errbuf, sizeof(errbuf))); return -1; } else if (endptr && (*endptr != '\0')) - sdb_error_set(SDB_LOG_WARNING, "unixsock: Ignoring garbage after " - "number while parsing numeric value (type %i): %s.\n", + sdb_log(SDB_LOG_WARNING, "unixsock: Ignoring garbage after " + "number while parsing numeric value (type %i): %s.", type, endptr); } @@ -158,9 +157,9 @@ sdb_unixsock_client_process_one_line(sdb_unixsock_client_t *client, char *next; if (! line) { /* this must no happen */ - sdb_error_set(SDB_LOG_ERR, "unixsock: Unexpected EOL while " + sdb_log(SDB_LOG_ERR, "unixsock: Unexpected EOL while " "parsing line (expected %i columns delimited by '%s'; " - "got %i): %s\n", column_count, delim, + "got %i): %s", column_count, delim, /* last line number */ i, orig_line); return -1; } @@ -232,7 +231,7 @@ sdb_unixsock_client_connect(sdb_unixsock_client_t *client) fd = socket(AF_UNIX, SOCK_STREAM, /* protocol = */ 0); if (fd < 0) { char errbuf[1024]; - sdb_error_set(SDB_LOG_ERR, "unixsock: Failed to open socket: %s\n", + sdb_log(SDB_LOG_ERR, "unixsock: Failed to open socket: %s", sdb_strerror(errno, errbuf, sizeof(errbuf))); return -1; } @@ -243,7 +242,7 @@ sdb_unixsock_client_connect(sdb_unixsock_client_t *client) if (connect(fd, (struct sockaddr *)&sa, sizeof(sa))) { char errbuf[1024]; - sdb_error_set(SDB_LOG_ERR, "unixsock: Failed to connect to %s: %s\n", + sdb_log(SDB_LOG_ERR, "unixsock: Failed to connect to %s: %s", sa.sun_path, sdb_strerror(errno, errbuf, sizeof(errbuf))); close(fd); return -1; @@ -252,13 +251,16 @@ sdb_unixsock_client_connect(sdb_unixsock_client_t *client) client->fh = fdopen(fd, "r+"); if (! client->fh) { char errbuf[1024]; - sdb_error_set(SDB_LOG_ERR, "unixsock: Failed to open I/O " - "stream for %s: %s\n", sa.sun_path, + sdb_log(SDB_LOG_ERR, "unixsock: Failed to open I/O " + "stream for %s: %s", sa.sun_path, sdb_strerror(errno, errbuf, sizeof(errbuf))); close(fd); return -1; } + /* enable line-buffering */ + setvbuf(client->fh, NULL, _IOLBF, 0); + client->shutdown = 0; return 0; } /* sdb_unixsock_client_connect */ @@ -278,8 +280,8 @@ sdb_unixsock_client_send(sdb_unixsock_client_t *client, status = fprintf(client->fh, "%s\r\n", msg); if (status < 0) { char errbuf[1024]; - sdb_error_set(SDB_LOG_ERR, "unixsock: Failed to write to " - "socket (%s): %s\n", client->path, + sdb_log(SDB_LOG_ERR, "unixsock: Failed to write to " + "socket (%s): %s", client->path, sdb_strerror(errno, errbuf, sizeof(errbuf))); return status; } @@ -300,8 +302,8 @@ sdb_unixsock_client_recv(sdb_unixsock_client_t *client, if (! buffer) { if (! feof(client->fh)) { char errbuf[1024]; - sdb_error_set(SDB_LOG_ERR, "unixsock: Failed to read " - "from socket (%s): %s\n", client->path, + sdb_log(SDB_LOG_ERR, "unixsock: Failed to read " + "from socket (%s): %s", client->path, sdb_strerror(errno, errbuf, sizeof(errbuf))); } return buffer; @@ -341,9 +343,9 @@ sdb_unixsock_client_process_lines(sdb_unixsock_client_t *client, types[i] = va_arg(ap, int); if ((types[i] < 1) || (types[i] > SDB_TYPE_BINARY)) { - sdb_error_set(SDB_LOG_ERR, "unixsock: Unknown column " + sdb_log(SDB_LOG_ERR, "unixsock: Unknown column " "type %i while processing response from the " - "UNIX socket @ %s.\n", types[i], client->path); + "UNIX socket @ %s.", types[i], client->path); va_end(ap); free(types); return -1; @@ -374,10 +376,10 @@ sdb_unixsock_client_process_lines(sdb_unixsock_client_t *client, column_count = sdb_unixsock_get_column_count(line, delim); if ((n_cols >= 0) && (n_cols != column_count)) { - sdb_error_set(SDB_LOG_ERR, "unixsock: number of columns (%i) " + sdb_log(SDB_LOG_ERR, "unixsock: number of columns (%i) " "does not match the number of requested columns (%i) " - "while processing response from the UNIX socket @ %s: " - "%s\n", column_count, n_cols, client->path, line); + "while processing response from the UNIX socket @ %s: %s", + column_count, n_cols, client->path, line); continue; } @@ -395,8 +397,8 @@ sdb_unixsock_client_process_lines(sdb_unixsock_client_t *client, || ((max_lines < 0) && (! sdb_unixsock_client_eof(client))) || sdb_unixsock_client_error(client)) { char errbuf[1024]; - sdb_error_set(SDB_LOG_ERR, "unixsock: Unexpected end of data while " - "reading from socket (%s): %s\n", client->path, + sdb_log(SDB_LOG_ERR, "unixsock: Unexpected end of data while " + "reading from socket (%s): %s", client->path, sdb_strerror(errno, errbuf, sizeof(errbuf))); return -1; }