X-Git-Url: https://git.lttng.org/?a=blobdiff_plain;f=libustcomm%2Fustcomm.c;h=ba98098b2d3a76387d57a40d3f811f16a4b1e0c8;hb=c555b1339182be95b625dc4c1d6ac7217d3f471b;hp=2e490b56b7f3b9bde99a3db8ade8b66f99f090da;hpb=68ab7a5d576faa16e0da19d0f824785eac215ae9;p=ust.git diff --git a/libustcomm/ustcomm.c b/libustcomm/ustcomm.c index 2e490b5..ba98098 100644 --- a/libustcomm/ustcomm.c +++ b/libustcomm/ustcomm.c @@ -31,11 +31,12 @@ #include #include "ustcomm.h" -#include "localerr.h" +#include "usterr.h" +#include "share.h" #define UNIX_PATH_MAX 108 -#define MSG_MAX 1000 +#define MSG_MAX 10000 /* FIXME: ustcomm blocks on message sending, which might be problematic in * some cases. Fix the poll() usage so sends are buffered until they don't @@ -51,6 +52,55 @@ // backtrace_symbols_fd(buffer, result, STDERR_FILENO); //} +static int mkdir_p(const char *path, mode_t mode) +{ + const char *path_p; + char *tmp; + + int retval = 0; + int result; + + tmp = malloc(strlen(path) + 1); + if (tmp == NULL) + return -1; + + /* skip first / */ + path_p = path+1; + + for(;;) { + while (*path_p != '/') { + if(*path_p == 0) + break; + ++path_p; + } + if (*path_p == '/') { + strncpy(tmp, path, path_p - path); + tmp[path_p-path] = '\0'; + if (tmp[path_p - path - 1] != '/') { + result = mkdir(tmp, mode); + if(result == -1) { + if (!(errno == EEXIST || errno == EACCES || errno == EROFS)) { + /* Then this is a real error */ + retval = -1; + break; + } + } + } + /* pass / */ + path_p++; + } else { + /* last component */ + result = mkdir(path, mode); + if (result == -1) + retval = -1; + break; + } + } + + free(tmp); + return retval; +} + char *strdup_malloc(const char *s) { char *retval; @@ -67,37 +117,40 @@ char *strdup_malloc(const char *s) static int signal_process(pid_t pid) { - int result; - - result = kill(pid, UST_SIGNAL); - if(result == -1) { - PERROR("kill"); - return -1; - } - - /* FIXME: should wait in a better way */ - //sleep(1); - return 0; } int pid_is_online(pid_t pid) { - return kill(pid, UST_SIGNAL) != -1; + return 1; } +/* Send a message + * + * @fd: file descriptor to send to + * @msg: a null-terminated string containing the message to send + * + * Return value: + * -1: error + * 0: connection closed + * 1: success + */ + static int send_message_fd(int fd, const char *msg) { int result; - result = send(fd, msg, strlen(msg), 0); + /* Send including the final \0 */ + result = patient_send(fd, msg, strlen(msg)+1, MSG_NOSIGNAL); if(result == -1) { - PERROR("send"); + if(errno != EPIPE) + PERROR("send"); return -1; } else if(result == 0) { return 0; } + DBG("sent message \"%s\"", msg); return 1; } @@ -153,30 +206,67 @@ int ustcomm_request_consumer(pid_t pid, const char *channel) } /* returns 1 to indicate a message was received - * returns 0 to indicate no message was received (cannot happen) + * returns 0 to indicate no message was received (end of stream) * returns -1 to indicate an error */ -static int recv_message_fd(int fd, char **msg, struct ustcomm_source *src) +#define RECV_INCREMENT 1 +#define RECV_INITIAL_BUF_SIZE 10 + +static int recv_message_fd(int fd, char **msg) { int result; + int buf_alloc_size = 0; + char *buf = NULL; + int buf_used_size = 0; - *msg = (char *) malloc(MSG_MAX+1); + buf = malloc(RECV_INITIAL_BUF_SIZE); + buf_alloc_size = RECV_INITIAL_BUF_SIZE; - result = recv(fd, *msg, MSG_MAX, 0); - if(result == -1) { - PERROR("recv"); - return -1; - } + for(;;) { + if(buf_used_size + RECV_INCREMENT > buf_alloc_size) { + char *new_buf; + buf_alloc_size *= 2; + new_buf = (char *) realloc(buf, buf_alloc_size); + if(new_buf == NULL) { + ERR("realloc returned NULL"); + free(buf); + return -1; + } + buf = new_buf; + } - (*msg)[result] = '\0'; - - DBG("ustcomm_app_recv_message: result is %d, message is %s", result, (*msg)); + /* FIXME: this is really inefficient; but with count>1 we would + * need a buffering mechanism */ + result = recv(fd, buf+buf_used_size, RECV_INCREMENT, 0); + if(result == -1) { + free(buf); + if(errno != ECONNRESET) + PERROR("recv"); + return -1; + } + if(result == 0) { + if(buf_used_size) + goto ret; + else { + free(buf); + return 0; + } + } - if(src) - src->fd = fd; + buf_used_size += result; + + if(buf[buf_used_size-1] == 0) { + goto ret; + } + } + +ret: + *msg = buf; + DBG("received message \"%s\"", buf); return 1; + } int ustcomm_send_reply(struct ustcomm_server *server, char *msg, struct ustcomm_source *src) @@ -281,8 +371,11 @@ int ustcomm_recv_message(struct ustcomm_server *server, char **msg, struct ustco for(idx=1; idxfd = fds[idx].fd; + + if(retval == 0) { /* connection finished */ close(fds[idx].fd); @@ -404,29 +497,22 @@ int ustcomm_send_request(struct ustcomm_connection *conn, const char *req, char { int result; - result = send(conn->fd, req, strlen(req), MSG_NOSIGNAL); - if(result == -1) { - if(errno != EPIPE) - PERROR("send"); - return -1; - } + /* Send including the final \0 */ + result = send_message_fd(conn->fd, req); + if(result != 1) + return result; if(!reply) return 1; - *reply = (char *) malloc(MSG_MAX+1); - result = recv(conn->fd, *reply, MSG_MAX, 0); + result = recv_message_fd(conn->fd, reply); if(result == -1) { - if(errno != ECONNRESET) - PERROR("recv"); return -1; } else if(result == 0) { return 0; } - (*reply)[result] = '\0'; - return 1; } @@ -460,7 +546,7 @@ int ustcomm_connect_path(const char *path, struct ustcomm_connection *conn, pid_ result = connect(fd, (struct sockaddr *)&addr, sizeof(addr)); if(result == -1) { - PERROR("connect"); + PERROR("connect (path=%s)", path); return -1; } @@ -489,6 +575,32 @@ int ustcomm_connect_app(pid_t pid, struct ustcomm_connection *conn) return ustcomm_connect_path(path, conn, pid); } +static int ensure_dir_exists(const char *dir) +{ + struct stat st; + int result; + + if(!strcmp(dir, "")) + return -1; + + result = stat(dir, &st); + if(result == -1 && errno != ENOENT) { + return -1; + } + else if(result == -1) { + /* ENOENT */ + int result; + + result = mkdir_p(dir, 0777); + if(result != 0) { + ERR("executing in recursive creation of directory %s", dir); + return -1; + } + } + + return 0; +} + /* Called by an application to initialize its server so daemons can * connect to it. */ @@ -504,6 +616,12 @@ int ustcomm_init_app(pid_t pid, struct ustcomm_app *handle) return -1; } + result = ensure_dir_exists(SOCK_DIR); + if(result == -1) { + ERR("Unable to create socket directory %s", SOCK_DIR); + return -1; + } + handle->server.listen_fd = init_named_socket(name, &(handle->server.socketpath)); if(handle->server.listen_fd < 0) { ERR("Error initializing named socket (%s). Check that directory exists and that it is writable.", name); @@ -533,6 +651,15 @@ int ustcomm_init_ustd(struct ustcomm_ustd *handle, const char *sock_path) asprintf(&name, "%s", sock_path); } else { + int result; + + /* Only check if socket dir exists if we are using the default directory */ + result = ensure_dir_exists(SOCK_DIR); + if(result == -1) { + ERR("Unable to create socket directory %s", SOCK_DIR); + return -1; + } + asprintf(&name, "%s/%s", SOCK_DIR, "ustd"); } @@ -576,7 +703,7 @@ void ustcomm_fini_app(struct ustcomm_app *handle) } } -static char *find_tok(char *str) +static const char *find_tok(const char *str) { while(*str == ' ') { str++; @@ -588,7 +715,7 @@ static char *find_tok(char *str) return str; } -static char *find_sep(char *str) +static const char *find_sep(const char *str) { while(*str != ' ') { str++; @@ -600,11 +727,11 @@ static char *find_sep(char *str) return str; } -int nth_token_is(char *str, char *token, int tok_no) +int nth_token_is(const char *str, const char *token, int tok_no) { int i; - char *start; - char *end; + const char *start; + const char *end; for(i=0; i<=tok_no; i++) { str = find_tok(str); @@ -629,12 +756,12 @@ int nth_token_is(char *str, char *token, int tok_no) return 1; } -char *nth_token(char *str, int tok_no) +char *nth_token(const char *str, int tok_no) { static char *retval = NULL; int i; - char *start; - char *end; + const char *start; + const char *end; for(i=0; i<=tok_no; i++) { str = find_tok(str);