X-Git-Url: http://git.lttng.org/?a=blobdiff_plain;f=libustcomm%2Fustcomm.c;h=50038cba157cdc964a553c718a22a6e38b6217c6;hb=4723ca096d740ff93da400df304c9902e9834e5f;hp=8b459df32975499a4c1137ee379fb9d40600b231;hpb=34b460e6b2180ef5e42cdc0455b3ac1a0afd912c;p=ust.git

diff --git a/libustcomm/ustcomm.c b/libustcomm/ustcomm.c
index 8b459df..50038cb 100644
--- a/libustcomm/ustcomm.c
+++ b/libustcomm/ustcomm.c
@@ -15,6 +15,8 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301 USA
  */
 
+/* API used by UST components to communicate with each other via sockets. */
+
 #define _GNU_SOURCE
 #include <sys/types.h>
 #include <signal.h>
@@ -23,6 +25,7 @@
 #include <sys/un.h>
 #include <unistd.h>
 #include <poll.h>
+#include <sys/epoll.h>
 #include <sys/stat.h>
 
 #include <stdio.h>
@@ -31,363 +34,505 @@
 #include <execinfo.h>
 
 #include "ustcomm.h"
-#include "localerr.h"
+#include "usterr.h"
+#include "share.h"
 
-#define UNIX_PATH_MAX 108
+static int mkdir_p(const char *path, mode_t mode)
+{
+	const char *path_p;
+	char *tmp;
 
-#define MSG_MAX 1000
+	int retval = 0;
+	int result;
+	mode_t old_umask;
 
-/* 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
- * block.
- */
+	tmp = zmalloc(strlen(path) + 1);
+	if (tmp == NULL)
+		return -1;
+
+	/* skip first / */
+	path_p = path+1;
+
+	old_umask = umask(0);
+	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;
+		}
+	}
 
-//static void bt(void)
-//{
-//	void *buffer[100];
-//	int result;
-//
-//	result = backtrace(&buffer, 100);
-//	backtrace_symbols_fd(buffer, result, STDERR_FILENO);
-//}
+	free(tmp);
+	umask(old_umask);
+	return retval;
+}
 
-char *strdup_malloc(const char *s)
+static struct sockaddr_un * create_sock_addr(const char *name,
+					     size_t *sock_addr_size)
 {
-	char *retval;
+	struct sockaddr_un * addr;
+	size_t alloc_size;
 
-	if(s == NULL)
+	alloc_size = (size_t) (((struct sockaddr_un *) 0)->sun_path) +
+		strlen(name) + 1;
+
+	addr = malloc(alloc_size);
+	if (addr < 0) {
+		ERR("allocating addr failed");
 		return NULL;
+	}
 
-	retval = (char *) malloc(strlen(s)+1);
+	addr->sun_family = AF_UNIX;
+	strcpy(addr->sun_path, name);
 
-	strcpy(retval, s);
+	*sock_addr_size = alloc_size;
 
-	return retval;
+	return addr;
 }
 
-static int signal_process(pid_t pid)
+struct ustcomm_sock * ustcomm_init_sock(int fd, int epoll_fd,
+					struct list_head *list)
 {
-	int result;
+	struct epoll_event ev;
+	struct ustcomm_sock *sock;
 
-	result = kill(pid, UST_SIGNAL);
-	if(result == -1) {
-		PERROR("kill");
-		return -1;
+	sock = malloc(sizeof(struct ustcomm_sock));
+	if (!sock) {
+		perror("malloc: couldn't allocate ustcomm_sock");
+		return NULL;
 	}
 
-	/* FIXME: should wait in a better way */
-	//sleep(1);
+	ev.events = EPOLLIN;
+	ev.data.ptr = sock;
+	sock->fd = fd;
 
-	return 0;
+	if (epoll_ctl(epoll_fd, EPOLL_CTL_ADD, sock->fd, &ev) == -1) {
+		perror("epoll_ctl: failed to add socket\n");
+		free(sock);
+		return NULL;
+	}
+
+	sock->epoll_fd = epoll_fd;
+	if (list) {
+		list_add(&sock->list, list);
+	} else {
+		INIT_LIST_HEAD(&sock->list);
+	}
+
+	return sock;
 }
 
-int pid_is_online(pid_t pid) {
-	return kill(pid, UST_SIGNAL) != -1;
+void ustcomm_del_sock(struct ustcomm_sock *sock, int keep_in_epoll)
+{
+	list_del(&sock->list);
+	if (!keep_in_epoll) {
+		if (epoll_ctl(sock->epoll_fd, EPOLL_CTL_DEL, sock->fd, NULL) == -1) {
+			PERROR("epoll_ctl: failed to delete socket");
+		}
+	}
+	close(sock->fd);
+	free(sock);
 }
 
-static int send_message_fd(int fd, const char *msg)
+struct ustcomm_sock * ustcomm_init_named_socket(const char *name,
+						int epoll_fd)
 {
 	int result;
+	int fd;
+	size_t sock_addr_size;
+	struct sockaddr_un * addr;
+	struct ustcomm_sock *sock;
+
+	fd = socket(PF_UNIX, SOCK_STREAM, 0);
+	if(fd == -1) {
+		PERROR("socket");
+		return NULL;
+	}
+
+	addr = create_sock_addr(name, &sock_addr_size);
+	if (addr == NULL) {
+		ERR("allocating addr, UST thread bailing");
+		goto close_sock;
+	}
+
+	result = access(name, F_OK);
+	if(result == 0) {
+		/* file exists */
+		result = unlink(name);
+		if(result == -1) {
+			PERROR("unlink of socket file");
+			goto free_addr;
+		}
+		DBG("socket already exists; overwriting");
+	}
 
-	result = send(fd, msg, strlen(msg), 0);
+	result = bind(fd, (struct sockaddr *)addr, sock_addr_size);
 	if(result == -1) {
-		PERROR("send");
-		return -1;
+		PERROR("bind");
+		goto free_addr;
 	}
-	else if(result == 0) {
-		return 0;
+
+	result = listen(fd, 1);
+	if(result == -1) {
+		PERROR("listen");
+		goto free_addr;
 	}
 
-	return 1;
+	sock = ustcomm_init_sock(fd, epoll_fd,
+				 NULL);
+	if (!sock) {
+		ERR("failed to create ustcomm_sock");
+		goto free_addr;
+	}
+
+	free(addr);
+
+	return sock;
+
+free_addr:
+	free(addr);
+close_sock:
+	close(fd);
+
+	return NULL;
+}
+
+void ustcomm_del_named_sock(struct ustcomm_sock *sock,
+			    int keep_socket_file)
+{
+	int result, fd;
+	struct stat st;
+	struct sockaddr dummy;
+	struct sockaddr_un *sockaddr = NULL;
+	int alloc_size;
+
+	fd = sock->fd;
+
+	if(!keep_socket_file) {
+
+		/* Get the socket name */
+		alloc_size = sizeof(dummy);
+		if (getsockname(fd, &dummy, (socklen_t *)&alloc_size) < 0) {
+			PERROR("getsockname failed");
+			goto del_sock;
+		}
+
+		sockaddr = zmalloc(alloc_size);
+		if (!sockaddr) {
+			ERR("failed to allocate sockaddr");
+			goto del_sock;
+		}
+
+		if (getsockname(fd, sockaddr, (socklen_t *)&alloc_size) < 0) {
+			PERROR("getsockname failed");
+			goto free_sockaddr;
+		}
+
+		/* Destroy socket */
+		result = stat(sockaddr->sun_path, &st);
+		if(result < 0) {
+			PERROR("stat (%s)", sockaddr->sun_path);
+			goto free_sockaddr;
+		}
+
+		/* Paranoid check before deleting. */
+		result = S_ISSOCK(st.st_mode);
+		if(!result) {
+			ERR("The socket we are about to delete is not a socket.");
+			goto free_sockaddr;
+		}
+
+		result = unlink(sockaddr->sun_path);
+		if(result < 0) {
+			PERROR("unlink");
+		}
+	}
+
+free_sockaddr:
+	free(sockaddr);
+
+del_sock:
+	ustcomm_del_sock(sock, keep_socket_file);
 }
 
+
 /* Called by an app to ask the consumer daemon to connect to it. */
 
 int ustcomm_request_consumer(pid_t pid, const char *channel)
 {
-	char path[UNIX_PATH_MAX];
-	int result;
-	char *msg=NULL;
+	int result, daemon_fd;
 	int retval = 0;
-	struct ustcomm_connection conn;
-	char *explicit_daemon_socket_path;
+	char *msg=NULL;
+	char *explicit_daemon_socket_path, *daemon_path;
 
 	explicit_daemon_socket_path = getenv("UST_DAEMON_SOCKET");
-	if(explicit_daemon_socket_path) {
+	if (explicit_daemon_socket_path) {
 		/* user specified explicitly a socket path */
-		result = snprintf(path, UNIX_PATH_MAX, "%s", explicit_daemon_socket_path);
-	}
-	else {
+		result = asprintf(&daemon_path, "%s", explicit_daemon_socket_path);
+	} else {
 		/* just use the default path */
-		result = snprintf(path, UNIX_PATH_MAX, "%s/ustd", SOCK_DIR);
+		result = asprintf(&daemon_path, "%s/ustd", SOCK_DIR);
 	}
-
-	if(result >= UNIX_PATH_MAX) {
+	if (result < 0) {
 		ERR("string overflow allocating socket name");
 		return -1;
 	}
 
-	asprintf(&msg, "collect %d %s", pid, channel); 
+	if (asprintf(&msg, "collect %d %s", pid, channel) < 0) {
+		ERR("ustcomm_request_consumer : asprintf failed (collect %d/%s)",
+		    pid, channel);
+		retval = -1;
+		goto free_daemon_path;
+	}
 
-	/* don't signal it because it's the daemon */
-	result = ustcomm_connect_path(path, &conn, -1);
-	if(result == -1) {
-		WARN("ustcomm_connect_path failed");
+	result = ustcomm_connect_path(daemon_path, &daemon_fd);
+	if (result < 0) {
+		WARN("ustcomm_connect_path failed, daemon_path: %s",
+		     daemon_path);
 		retval = -1;
 		goto del_string;
 	}
 
-	result = ustcomm_send_request(&conn, msg, NULL);
-	if(result == -1) {
-		WARN("ustcomm_send_request failed");
+	result = ustcomm_send_request(daemon_fd, msg, NULL);
+	if (result < 0) {
+		WARN("ustcomm_send_request failed, daemon path: %s",
+		     daemon_path);
 		retval = -1;
-		goto disconnect;
 	}
 
-	disconnect:
-	ustcomm_disconnect(&conn);
-	del_string:
+	close(daemon_fd);
+del_string:
 	free(msg);
+free_daemon_path:
+	free(daemon_path);
 
 	return retval;
 }
 
 /* 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)
+int ustcomm_recv_fd(int sock,
+		    struct ustcomm_header *header,
+		    char **data, int *fd)
 {
 	int result;
-
-	*msg = (char *) malloc(MSG_MAX+1);
-
-	result = recv(fd, *msg, MSG_MAX, 0);
-	if(result == -1) {
-		PERROR("recv");
-		return -1;
+	int retval;
+	struct ustcomm_header peek_header;
+	struct iovec iov[2];
+	struct msghdr msg;
+	struct cmsghdr *cmsg;
+	char buf[CMSG_SPACE(sizeof(int))];
+
+	result = recv(sock, &peek_header, sizeof(peek_header),
+		      MSG_PEEK | MSG_WAITALL);
+	if (result <= 0) {
+		if(errno == ECONNRESET) {
+			return 0;
+		} else if (errno == EINTR) {
+			return -1;
+		} else if (result < 0) {
+			PERROR("recv");
+			return -1;
+		}
+		return 0;
 	}
 
-	(*msg)[result] = '\0';
-	
-	DBG("ustcomm_app_recv_message: result is %d, message is %s", result, (*msg));
+	memset(&msg, 0, sizeof(msg));
 
-	if(src)
-		src->fd = fd;
+	iov[0].iov_base = (char *)header;
+	iov[0].iov_len = sizeof(struct ustcomm_header);
 
-	return 1;
-}
+	msg.msg_iov = iov;
+	msg.msg_iovlen = 1;
 
-int ustcomm_send_reply(struct ustcomm_server *server, char *msg, struct ustcomm_source *src)
-{
-	int result;
+	if (peek_header.size) {
+		if (peek_header.size < 0 || peek_header.size > 100) {
+			WARN("big peek header! %d", peek_header.size);
+		}
+		*data = malloc(peek_header.size);
+		if (!*data) {
+			ERR("failed to allocate space for message");
+		}
 
-	result = send_message_fd(src->fd, msg);
-	if(result < 0) {
-		ERR("error in send_message_fd");
-		return -1;
-	}
+		iov[1].iov_base = (char *)*data;
+		iov[1].iov_len = peek_header.size;
 
-	return 0;
-} 
+		msg.msg_iovlen++;
+	}
 
-/* Called after a fork. */
+	if (fd && peek_header.fd_included) {
+		msg.msg_control = buf;
+		msg.msg_controllen = sizeof(buf);
+	}
 
-int ustcomm_close_all_connections(struct ustcomm_server *server)
-{
-	struct ustcomm_connection *conn;
-	struct ustcomm_connection *deletable_conn = NULL;
+	result = recvmsg(sock, &msg,
+			 MSG_WAITALL);
 
-	list_for_each_entry(conn, &server->connections, list) {
-		free(deletable_conn);
-		deletable_conn = conn;
-		close(conn->fd);
-		list_del(&conn->list);
+	if (result <= 0) {
+		if(errno == ECONNRESET) {
+			retval = 0;
+		} else if (errno == EINTR) {
+			retval = -1;
+		} else if (result < 0) {
+			PERROR("recv");
+			retval = -1;
+		} else {
+			retval = 0;
+		}
+		free(*data);
+		return retval;
+	}
+
+	if (fd && peek_header.fd_included) {
+		cmsg = CMSG_FIRSTHDR(&msg);
+		result = 0;
+		while (cmsg != NULL) {
+			if (cmsg->cmsg_level == SOL_SOCKET
+			    && cmsg->cmsg_type  == SCM_RIGHTS) {
+				*fd = *(int *) CMSG_DATA(cmsg);
+				result = 1;
+				break;
+			}
+			cmsg = CMSG_NXTHDR(&msg, cmsg);
+		}
+		if (!result) {
+			ERR("Failed to receive file descriptor\n");
+		}
 	}
 
-	return 0;
+	return 1;
 }
 
-/* @timeout: max blocking time in milliseconds, -1 means infinity
- *
- * returns 1 to indicate a message was received
- * returns 0 to indicate no message was received
- * returns -1 to indicate an error
- */
-
-int ustcomm_recv_message(struct ustcomm_server *server, char **msg, struct ustcomm_source *src, int timeout)
+int ustcomm_recv(int sock,
+		 struct ustcomm_header *header,
+		 char **data)
 {
-	struct pollfd *fds;
-	struct ustcomm_connection *conn;
-	int result;
-	int retval;
-
-	for(;;) {
-		int idx = 0;
-		int n_fds = 1;
-
-		list_for_each_entry(conn, &server->connections, list) {
-			n_fds++;
-		}
-
-		fds = (struct pollfd *) malloc(n_fds * sizeof(struct pollfd));
-		if(fds == NULL) {
-			ERR("malloc returned NULL");
-			return -1;
-		}
+	return ustcomm_recv_fd(sock, header, data, NULL);
+}
 
-		/* special idx 0 is for listening socket */
-		fds[idx].fd = server->listen_fd;
-		fds[idx].events = POLLIN;
-		idx++;
 
-		list_for_each_entry(conn, &server->connections, list) {
-			fds[idx].fd = conn->fd;
-			fds[idx].events = POLLIN;
-			idx++;
-		}
+int recv_message_conn(int sock, char **msg)
+{
+	struct ustcomm_header header;
 
-		while((result = poll(fds, n_fds, timeout)) == -1 && errno == EINTR)
-			/* nothing */;
-		if(result == -1) {
-			PERROR("poll");
-			return -1;
-		}
+	return ustcomm_recv(sock, &header, msg);
+}
 
-		if(result == 0)
-			return 0;
+int ustcomm_send_fd(int sock,
+		    const struct ustcomm_header *header,
+		    const char *data,
+		    int *fd)
+{
+	struct iovec iov[2];
+	struct msghdr msg;
+	int result;
+	struct cmsghdr *cmsg;
+	char buf[CMSG_SPACE(sizeof(int))];
 
-		if(fds[0].revents) {
-			struct ustcomm_connection *newconn;
-			int newfd;
+	memset(&msg, 0, sizeof(msg));
 
-			result = newfd = accept(server->listen_fd, NULL, NULL);
-			if(result == -1) {
-				PERROR("accept");
-				return -1;
-			}
+	iov[0].iov_base = (char *)header;
+	iov[0].iov_len = sizeof(struct ustcomm_header);
 
-			newconn = (struct ustcomm_connection *) malloc(sizeof(struct ustcomm_connection));
-			if(newconn == NULL) {
-				ERR("malloc returned NULL");
-				return -1;
-			}
+	msg.msg_iov = iov;
+	msg.msg_iovlen = 1;
 
-			newconn->fd = newfd;
+	if (header->size) {
+		iov[1].iov_base = (char *)data;
+		iov[1].iov_len = header->size;
 
-			list_add(&newconn->list, &server->connections);
-		}
+		msg.msg_iovlen++;
 
-		for(idx=1; idx<n_fds; idx++) {
-			if(fds[idx].revents) {
-				retval = recv_message_fd(fds[idx].fd, msg, src);
-				if(**msg == 0) {
-					/* connection finished */
-					close(fds[idx].fd);
-
-					list_for_each_entry(conn, &server->connections, list) {
-						if(conn->fd == fds[idx].fd) {
-							list_del(&conn->list);
-							break;
-						}
-					}
-				}
-				else {
-					goto free_fds_return;
-				}
-			}
-		}
+	}
 
-		free(fds);
+	if (fd && header->fd_included) {
+		msg.msg_control = buf;
+		msg.msg_controllen = sizeof(buf);
+		cmsg = CMSG_FIRSTHDR(&msg);
+		cmsg->cmsg_level = SOL_SOCKET;
+		cmsg->cmsg_type = SCM_RIGHTS;
+		cmsg->cmsg_len = CMSG_LEN(sizeof(int));
+		*(int *) CMSG_DATA(cmsg) = *fd;
+		msg.msg_controllen = cmsg->cmsg_len;
 	}
 
-free_fds_return:
-	free(fds);
-	return retval;
+	result = sendmsg(sock, &msg, MSG_NOSIGNAL);
+	if (result < 0 && errno != EPIPE) {
+		PERROR("sendmsg failed");
+	}
+	return result;
 }
 
-int ustcomm_ustd_recv_message(struct ustcomm_ustd *ustd, char **msg, struct ustcomm_source *src, int timeout)
+int ustcomm_send(int sock,
+		 const struct ustcomm_header *header,
+		 const char *data)
 {
-	return ustcomm_recv_message(&ustd->server, msg, src, timeout);
+	return ustcomm_send_fd(sock, header, data, NULL);
 }
 
-int ustcomm_app_recv_message(struct ustcomm_app *app, char **msg, struct ustcomm_source *src, int timeout)
+int ustcomm_send_reply(char *msg, int sock)
 {
-	return ustcomm_recv_message(&app->server, msg, src, timeout);
-}
+	int result;
+	struct ustcomm_header header;
 
-/* This removes src from the list of active connections of app.
- */
+	memset(&header, 0, sizeof(header));
 
-int ustcomm_app_detach_client(struct ustcomm_app *app, struct ustcomm_source *src)
-{
-	struct ustcomm_server *server = (struct ustcomm_server *)app;
-	struct ustcomm_connection *conn;
+	header.size = strlen(msg) + 1;
 
-	list_for_each_entry(conn, &server->connections, list) {
-		if(conn->fd == src->fd) {
-			list_del(&conn->list);
-			goto found;
-		}
+	result = ustcomm_send(sock, &header, msg);
+	if(result < 0) {
+		ERR("error in ustcomm_send");
+		return result;
 	}
 
-	return -1;
-found:
-	return src->fd;
+	return 0;
 }
 
-static int init_named_socket(const char *name, char **path_out)
+int ustcomm_send_req(int sock,
+		     const struct ustcomm_header *req_header,
+		     const char *data,
+		     char **response)
 {
 	int result;
-	int fd;
-
-	struct sockaddr_un addr;
-	
-	result = fd = socket(PF_UNIX, SOCK_STREAM, 0);
-	if(result == -1) {
-		PERROR("socket");
-		return -1;
-	}
-
-	addr.sun_family = AF_UNIX;
-
-	strncpy(addr.sun_path, name, UNIX_PATH_MAX);
-	addr.sun_path[UNIX_PATH_MAX-1] = '\0';
-
-	result = access(name, F_OK);
-	if(result == 0) {
-		/* file exists */
-		result = unlink(name);
-		if(result == -1) {
-			PERROR("unlink of socket file");
-			goto close_sock;
-		}
-		WARN("socket already exists; overwriting");
-	}
-
-	result = bind(fd, (struct sockaddr *)&addr, sizeof(addr));
-	if(result == -1) {
-		PERROR("bind");
-		goto close_sock;
-	}
+	struct ustcomm_header res_header;
 
-	result = listen(fd, 1);
-	if(result == -1) {
-		PERROR("listen");
-		goto close_sock;
+	result = ustcomm_send(sock, req_header, data);
+	if ( result <= 0) {
+		return result;
 	}
 
-	if(path_out) {
-		*path_out = strdup(addr.sun_path);
+	if (!response) {
+		return 1;
 	}
 
-	return fd;
-
-	close_sock:
-	close(fd);
+	return ustcomm_recv(sock,
+			    &res_header,
+			    response);
 
-	return -1;
 }
 
 /*
@@ -400,183 +545,126 @@ static int init_named_socket(const char *name, char **path_out)
  * ECONNRESET, which is normal when the application dies.
  */
 
-int ustcomm_send_request(struct ustcomm_connection *conn, const char *req, char **reply)
+int ustcomm_send_request(int sock, const char *req, char **reply)
 {
-	int result;
+	struct ustcomm_header req_header;
 
-	result = send(conn->fd, req, strlen(req), MSG_NOSIGNAL);
-	if(result == -1) {
-		if(errno != ECONNRESET)
-			PERROR("send");
-		return -1;
-	}
-
-	if(!reply)
-		return 1;
+	req_header.size = strlen(req) + 1;
 
-	*reply = (char *) malloc(MSG_MAX+1);
-	result = recv(conn->fd, *reply, MSG_MAX, 0);
-	if(result == -1) {
-		if(errno != ECONNRESET)
-			PERROR("recv");
-		return -1;
-	}
-	else if(result == 0) {
-		return 0;
-	}
-	
-	(*reply)[result] = '\0';
+	return ustcomm_send_req(sock,
+				&req_header,
+				req,
+				reply);
 
-	return 1;
 }
 
-int ustcomm_connect_path(const char *path, struct ustcomm_connection *conn, pid_t signalpid)
+/* Return value:
+ *  0: success
+ * -1: error
+ */
+
+int ustcomm_connect_path(const char *name, int *connection_fd)
 {
-	int fd;
-	int result;
-	struct sockaddr_un addr;
+	int result, fd;
+	size_t sock_addr_size;
+	struct sockaddr_un *addr;
 
-	result = fd = socket(PF_UNIX, SOCK_STREAM, 0);
-	if(result == -1) {
+	fd = socket(PF_UNIX, SOCK_STREAM, 0);
+	if(fd == -1) {
 		PERROR("socket");
 		return -1;
 	}
 
-	addr.sun_family = AF_UNIX;
-
-	result = snprintf(addr.sun_path, UNIX_PATH_MAX, "%s", path);
-	if(result >= UNIX_PATH_MAX) {
-		ERR("string overflow allocating socket name");
-		return -1;
-	}
-
-	if(signalpid >= 0) {
-		result = signal_process(signalpid);
-		if(result == -1) {
-			ERR("could not signal process");
-			return -1;
-		}
+	addr = create_sock_addr(name, &sock_addr_size);
+	if (addr == NULL) {
+		ERR("allocating addr failed");
+		goto close_sock;
 	}
 
-	result = connect(fd, (struct sockaddr *)&addr, sizeof(addr));
+	result = connect(fd, (struct sockaddr *)addr, sock_addr_size);
 	if(result == -1) {
-		PERROR("connect");
-		return -1;
+		PERROR("connect (path=%s)", name);
+		goto free_sock_addr;
 	}
 
-	conn->fd = fd;
-
-	return 0;
-}
-
-int ustcomm_disconnect(struct ustcomm_connection *conn)
-{
-	return close(conn->fd);
-}
+	*connection_fd = fd;
 
-int ustcomm_connect_app(pid_t pid, struct ustcomm_connection *conn)
-{
-	int result;
-	char path[UNIX_PATH_MAX];
+	free(addr);
 
+	return 0;
 
-	result = snprintf(path, UNIX_PATH_MAX, "%s/%d", SOCK_DIR, pid);
-	if(result >= UNIX_PATH_MAX) {
-		ERR("string overflow allocating socket name");
-		return -1;
-	}
+free_sock_addr:
+	free(addr);
+close_sock:
+	close(fd);
 
-	return ustcomm_connect_path(path, conn, pid);
+	return -1;
 }
 
-/* Called by an application to initialize its server so daemons can
- * connect to it.
+
+/* Open a connection to a traceable app.
+ *
+ * Return value:
+ *  0: success
+ * -1: error
  */
 
-int ustcomm_init_app(pid_t pid, struct ustcomm_app *handle)
+int ustcomm_connect_app(pid_t pid, int *app_fd)
 {
 	int result;
+	int retval = 0;
 	char *name;
 
-	result = asprintf(&name, "%s/%d", SOCK_DIR, (int)pid);
-	if(result >= UNIX_PATH_MAX) {
-		ERR("string overflow allocating socket name");
+	result = asprintf(&name, "%s/%d", SOCK_DIR, pid);
+	if (result < 0) {
+		ERR("failed to allocate socket name");
 		return -1;
 	}
 
-	handle->server.listen_fd = init_named_socket(name, &(handle->server.socketpath));
-	if(handle->server.listen_fd < 0) {
-		ERR("error initializing named socket");
-		goto free_name;
-	}
-	free(name);
-
-	INIT_LIST_HEAD(&handle->server.connections);
-
-	return 0;
-
-free_name:
-	free(name);
-	return -1;
-}
-
-/* Used by the daemon to initialize its server so applications
- * can connect to it.
- */
-
-int ustcomm_init_ustd(struct ustcomm_ustd *handle, const char *sock_path)
-{
-	char *name;
-	int retval = 0;
-
-	if(sock_path) {
-		asprintf(&name, "%s", sock_path);
-	}
-	else {
-		asprintf(&name, "%s/%s", SOCK_DIR, "ustd");
-	}
-
-	handle->server.listen_fd = init_named_socket(name, &handle->server.socketpath);
-	if(handle->server.listen_fd < 0) {
-		ERR("error initializing named socket at %s", name);
+	result = ustcomm_connect_path(name, app_fd);
+	if (result < 0) {
+		ERR("failed to connect to app");
 		retval = -1;
-		goto free_name;
 	}
 
-	INIT_LIST_HEAD(&handle->server.connections);
-
-free_name:
 	free(name);
 
 	return retval;
 }
 
-void ustcomm_fini_app(struct ustcomm_app *handle)
+int ensure_dir_exists(const char *dir)
 {
-	int result;
 	struct stat st;
+	int result;
 
-	/* Destroy socket */
-	result = stat(handle->server.socketpath, &st);
-	if(result == -1) {
-		PERROR("stat (%s)", handle->server.socketpath);
-		return;
-	}
+	if(!strcmp(dir, ""))
+		return -1;
 
-	/* Paranoid check before deleting. */
-	result = S_ISSOCK(st.st_mode);
-	if(!result) {
-		ERR("The socket we are about to delete is not a socket.");
-		return;
+	result = stat(dir, &st);
+	if(result == -1 && errno != ENOENT) {
+		return -1;
 	}
+	else if(result == -1) {
+		/* ENOENT */
+		int result;
 
-	result = unlink(handle->server.socketpath);
-	if(result == -1) {
-		PERROR("unlink");
+		/* mkdir mode to 0777 */
+		result = mkdir_p(dir, S_IRWXU | S_IRWXG | S_IRWXO);
+		if(result != 0) {
+			ERR("executing in recursive creation of directory %s", dir);
+			return -1;
+		}
 	}
+
+	return 0;
 }
 
-static char *find_tok(char *str)
+/* Used by the daemon to initialize its server so applications
+ * can connect to it.
+ */
+
+
+static const char *find_tok(const char *str)
 {
 	while(*str == ' ') {
 		str++;
@@ -588,7 +676,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 +688,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 +717,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);
@@ -655,8 +743,11 @@ char *nth_token(char *str, int tok_no)
 		retval = NULL;
 	}
 
-	asprintf(&retval, "%.*s", (int)(end-start), start);
+	if (asprintf(&retval, "%.*s", (int)(end-start), start) < 0) {
+		ERR("nth_token : asprintf failed (%.*s)",
+		    (int)(end-start), start);
+		return NULL;
+	}
 
 	return retval;
 }
-