X-Git-Url: http://git.tuebingen.mpg.de/?a=blobdiff_plain;f=audioc.c;h=f68aee542e1ed2ef48694eae309112cccd3314f2;hb=cf39e478f5379506a003ab8a0a2f14ee8fcf98e4;hp=abd763596e8b0039084d502e428d1ddec00ea232;hpb=f652ce6afe7f6fd0e42814505234cbf4321a736e;p=paraslash.git

diff --git a/audioc.c b/audioc.c
index abd76359..f68aee54 100644
--- a/audioc.c
+++ b/audioc.c
@@ -42,6 +42,25 @@ static char *concat_args(unsigned argc, char * const *argv)
 	return buf;
 }
 
+static int connect_audiod(const char *sname, char *args)
+{
+	int fd = -1, ret;
+
+	ret = connect_local_socket(sname);
+	if (ret < 0)
+		goto fail;
+	fd = ret;
+	ret = send_cred_buffer(fd, args);
+	if (ret < 0)
+		goto fail;
+	return fd;
+fail:
+	PARA_ERROR_LOG("could not connect %s\n", sname);
+	if (fd >= 0)
+		close(fd);
+	return ret;
+}
+
 #ifdef HAVE_READLINE
 #include "list.h"
 #include "sched.h"
@@ -106,7 +125,7 @@ static void audioc_pre_select(struct sched *s, struct task *t)
 	para_fd_set(at->fd, &s->rfds, &s->max_fileno);
 }
 
-static void audioc_post_select(struct sched *s, struct task *t)
+static int audioc_post_select(struct sched *s, struct task *t)
 {
 	char *buf = NULL;
 	struct audioc_task *at = container_of(t, struct audioc_task, task);
@@ -115,7 +134,7 @@ static void audioc_post_select(struct sched *s, struct task *t)
 	if (ret < 0)
 		goto out;
 	if (!FD_ISSET(at->fd, &s->rfds))
-		return;
+		return 0;
 	buf = para_malloc(conf.bufsize_arg);
 	ret = recv_bin_buffer(at->fd, buf, conf.bufsize_arg);
 	PARA_DEBUG_LOG("recv: %d\n", ret);
@@ -124,14 +143,14 @@ static void audioc_post_select(struct sched *s, struct task *t)
 	if (ret < 0)
 		goto out;
 	btr_add_output(buf, ret, at->btrn);
-	return;
+	return 0;
 out:
 	if (ret < 0) {
 		free(buf);
 		btr_remove_node(&at->btrn);
 		close(at->fd);
 	}
-	t->error = ret;
+	return ret;
 }
 
 static struct audioc_task audioc_task = {
@@ -157,14 +176,11 @@ static int audioc_i9e_line_handler(char *line)
 	args = concat_args(conf.inputs_num, conf.inputs);
 	free_argv(conf.inputs);
 	conf.inputs_num = 0; /* required for audioc_cmdline_parser_free() */
-	ret = connect_local_socket(socket_name);
+	ret = connect_audiod(socket_name, args);
 	if (ret < 0)
 		goto out;
 	at->fd = ret;
 	ret = mark_fd_nonblocking(at->fd);
-	if (ret < 0)
-		goto close;
-	ret = send_cred_buffer(at->fd, args);
 	if (ret < 0)
 		goto close;
 	free(args);
@@ -320,21 +336,16 @@ int main(int argc, char *argv[])
 		interactive_session();
 	args = concat_args(conf.inputs_num, conf.inputs);
 
-	ret = connect_local_socket(socket_name);
+	ret = connect_audiod(socket_name, args);
 	free(socket_name);
-	if (ret < 0) {
-		PARA_EMERG_LOG("failed to connect to local socket\n");
+	if (ret < 0)
 		goto out;
-	}
 	fd = ret;
-	ret = send_cred_buffer(fd, args);
+	ret = mark_fd_blocking(STDOUT_FILENO);
 	if (ret < 0)
 		goto out;
 	bufsize = conf.bufsize_arg;
 	buf = para_malloc(bufsize);
-	ret = mark_fd_blocking(STDOUT_FILENO);
-	if (ret < 0)
-		goto out;
 	do {
 		size_t n = ret = recv_bin_buffer(fd, buf, bufsize);
 		if (ret <= 0)