X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=client_common.c;h=97d29f8fd09948b1e97972431df1f62ec30a4989;hp=1ecba73068f04c4dbf43480f5ccf32a932e01279;hb=089fb2fb2d9e2a3473aa6fac91681ca8ddfffff3;hpb=e2eebf654cae2fc004cee516a0e165127c0214ca

diff --git a/client_common.c b/client_common.c
index 1ecba730..97d29f8f 100644
--- a/client_common.c
+++ b/client_common.c
@@ -1,14 +1,17 @@
-/*
- * Copyright (C) 1997-2013 Andre Noll <maan@systemlinux.org>
- *
- * Licensed under the GPL v2. For licencing details see COPYING.
- */
+/* Copyright (C) 1997 Andre Noll <maan@tuebingen.mpg.de>, see file COPYING. */
 
 /** \file client_common.c Common functions of para_client and para_audiod. */
 
+#include <netinet/in.h>
+#include <sys/socket.h>
 #include <regex.h>
 #include <sys/types.h>
+#include <arpa/inet.h>
+#include <sys/un.h>
+#include <netdb.h>
+#include <lopsub.h>
 
+#include "client.lsg.h"
 #include "para.h"
 #include "error.h"
 #include "list.h"
@@ -18,80 +21,47 @@
 #include "fd.h"
 #include "sideband.h"
 #include "string.h"
-#include "client.cmdline.h"
 #include "client.h"
 #include "buffer_tree.h"
 #include "version.h"
-#include "ggo.h"
 
 /** The size of the receiving buffer. */
 #define CLIENT_BUFSIZE 4000
 
-/**
- * Close the connection to para_server and deallocate per-command resources.
- *
- * \param ct The client task.
- *
- * This frees all resources of the current command but keeps the configuration
- * in \p ct->conf.
- *
- * \sa \ref client_close().
- */
-void client_disconnect(struct client_task *ct)
-{
-	if (!ct)
-		return;
-	if (ct->scc.fd >= 0)
-		close(ct->scc.fd);
-	free_argv(ct->features);
-	ct->features = NULL;
-	sc_free(ct->scc.recv);
-	ct->scc.recv = NULL;
-	sc_free(ct->scc.send);
-	ct->scc.send = NULL;
-	btr_remove_node(&ct->btrn);
-}
-
 /**
  * Close the connection to para_server and free all resources.
  *
  * \param ct Pointer to the client data.
  *
- * \sa \ref client_open(), \ref client_disconnect().
+ * \sa \ref client_open().
  */
 void client_close(struct client_task *ct)
 {
 	if (!ct)
 		return;
-	client_disconnect(ct);
 	free(ct->user);
 	free(ct->config_file);
 	free(ct->key_file);
-	client_cmdline_parser_free(&ct->conf);
+	lls_free_parse_result(ct->lpr, CLIENT_CMD_PTR);
 	free(ct->challenge_hash);
-	sb_free(ct->sbc);
+	sb_free(ct->sbc[0]);
+	sb_free(ct->sbc[1]);
 	free(ct);
 }
 
-/**
+/*
  * The preselect hook for server commands.
  *
- * \param s Pointer to the scheduler.
- * \param t Pointer to the task struct for this command.
- *
  * The task pointer must contain a pointer to the initialized client data
  * structure as it is returned by client_open().
  *
  * This function checks the state of the connection and adds the file descriptor
- * of the connection to the read or write fd set of \a s accordingly.
- *
- * \sa register_task() client_open(), struct sched, struct task.
+ * of the connection to the read or write fd set of s accordingly.
  */
-static void client_pre_select(struct sched *s, struct task *t)
+static void client_pre_select(struct sched *s, void *context)
 {
 	int ret;
-	struct client_task *ct = container_of(t, struct client_task, task);
-	struct btr_node *btrn = ct->btrn;
+	struct client_task *ct = context;
 
 	if (ct->scc.fd < 0)
 		return;
@@ -99,7 +69,6 @@ static void client_pre_select(struct sched *s, struct task *t)
 	case CL_CONNECTED:
 	case CL_SENT_AUTH:
 	case CL_SENT_CH_RESPONSE:
-	case CL_SENT_COMMAND:
 		para_fd_set(ct->scc.fd, &s->rfds, &s->max_fileno);
 		return;
 
@@ -109,76 +78,49 @@ static void client_pre_select(struct sched *s, struct task *t)
 		para_fd_set(ct->scc.fd, &s->wfds, &s->max_fileno);
 		return;
 
-	case CL_RECEIVING:
-		ret = btr_node_status(btrn, 0, BTR_NT_ROOT);
-		if (ret != 0) {
+	case CL_SENDING:
+		if (ct->btrn[1]) {
+			ret = btr_node_status(ct->btrn[1], 0, BTR_NT_LEAF);
 			if (ret < 0)
 				sched_min_delay(s);
-			else
-				para_fd_set(ct->scc.fd, &s->rfds,
-					&s->max_fileno);
+			else if (ret > 0)
+				para_fd_set(ct->scc.fd, &s->wfds, &s->max_fileno);
 		}
-		return;
-	case CL_SENDING:
-		ret = btr_node_status(btrn, 0, BTR_NT_LEAF);
-		if (ret != 0) {
+		/* fallthrough */
+	case CL_EXECUTING:
+		if (ct->btrn[0]) {
+			ret = btr_node_status(ct->btrn[0], 0, BTR_NT_ROOT);
 			if (ret < 0)
 				sched_min_delay(s);
-			else
-				para_fd_set(ct->scc.fd, &s->wfds,
-					&s->max_fileno);
+			else if (ret > 0)
+				para_fd_set(ct->scc.fd, &s->rfds, &s->max_fileno);
 		}
 		return;
 	}
 }
 
-static int client_recv_buffer(struct client_task *ct, fd_set *rfds,
-		char *buf, size_t sz, size_t *n)
-{
-	int ret;
-
-	if (ct->status < CL_SENT_CH_RESPONSE)
-		return read_nonblock(ct->scc.fd, buf, sz, rfds, n);
-
-	*n = 0;
-	ret = sc_recv_buffer(&ct->scc, buf, sz);
-	/*
-	 * sc_recv_buffer is used with blocking fds elsewhere, so it
-	 * does not use the nonblock-API. Therefore we need to
-	 * check for EOF and EAGAIN.
-	 */
-	if (ret == 0)
-		return -E_SERVER_EOF;
-	if (ret == -ERRNO_TO_PARA_ERROR(EAGAIN))
-		return 0;
-	if (ret < 0)
-		return ret;
-	*n = ret;
-	return 0;
-}
-
-static int send_sb(struct client_task *ct, void *buf, size_t numbytes,
+static int send_sb(struct client_task *ct, int channel, void *buf, size_t numbytes,
 		enum sb_designator band, bool dont_free)
 {
 	int ret, fd = ct->scc.fd;
 	struct iovec iov[2];
 
-	if (!ct->sbc) {
+	if (!ct->sbc[channel]) {
 		struct sb_buffer sbb;
 		sb_transformation trafo = ct->status < CL_RECEIVED_PROCEED?
 			NULL : sc_trafo;
 		sbb = (typeof(sbb))SBB_INIT(band, buf, numbytes);
-		ct->sbc = sb_new_send(&sbb, dont_free, trafo, ct->scc.send);
+		ct->sbc[channel] = sb_new_send(&sbb, dont_free, trafo, ct->scc.send);
 	}
-	ret = sb_get_send_buffers(ct->sbc, iov);
+	ret = sb_get_send_buffers(ct->sbc[channel], iov);
 	ret = xwritev(fd, iov, ret);
 	if (ret < 0) {
-		sb_free(ct->sbc);
-		ct->sbc = NULL;
+		sb_free(ct->sbc[channel]);
+		ct->sbc[channel] = NULL;
 		return ret;
 	}
-	if (sb_sent(ct->sbc, ret)) {
-		ct->sbc = NULL;
+	if (sb_sent(ct->sbc[channel], ret)) {
+		ct->sbc[channel] = NULL;
 		return 1;
 	}
 	return 0;
@@ -201,21 +143,24 @@ static int recv_sb(struct client_task *ct, fd_set *rfds,
 		trafo = sc_trafo;
 		trafo_context = ct->scc.recv;
 	}
-	if (!ct->sbc)
-		ct->sbc = sb_new_recv(0, trafo, trafo_context);
+	if (!ct->sbc[0])
+		ct->sbc[0] = sb_new_recv(0, trafo, trafo_context);
 again:
-	sb_get_recv_buffer(ct->sbc, &iov);
+	sb_get_recv_buffer(ct->sbc[0], &iov);
 	ret = read_nonblock(ct->scc.fd, iov.iov_base, iov.iov_len, rfds, &n);
 	if (ret < 0) {
-		sb_free(ct->sbc);
-		ct->sbc = NULL;
+		sb_free(ct->sbc[0]);
+		ct->sbc[0] = NULL;
 		return ret;
 	}
 	if (n == 0)
 		return 0;
-	if (!sb_received(ct->sbc, n, result))
+	ret = sb_received(ct->sbc[0], n, result);
+	if (ret < 0)
+		return ret;
+	if (ret == 0)
 		goto again;
-	ct->sbc = NULL;
+	ct->sbc[0] = NULL;
 	return 1;
 }
 
@@ -251,10 +196,14 @@ static int dispatch_sbb(struct client_task *ct, struct sb_buffer *sbb)
 		PARA_DEBUG_LOG("band: %s\n", designator[sbb->band]);
 
 	switch (sbb->band) {
+	case SBD_AWAITING_DATA:
+		ct->status = CL_SENDING;
+		ret = 1;
+		goto out;
 	case SBD_OUTPUT:
 		if (iov_valid(&sbb->iov))
 			btr_add_output(sbb->iov.iov_base, sbb->iov.iov_len,
-				ct->btrn);
+				ct->btrn[0]);
 		ret = 1;
 		goto out;
 	case SBD_DEBUG_LOG:
@@ -288,60 +237,51 @@ out:
 	return ret;
 }
 
-static bool has_feature(const char *feature, struct client_task *ct)
-{
-	return find_arg(feature, ct->features) >= 0? true : false;
-}
-
 static int send_sb_command(struct client_task *ct)
 {
 	int i;
 	char *command, *p;
 	size_t len = 0;
+	unsigned num_inputs = lls_num_inputs(ct->lpr);
 
-	if (ct->sbc)
-		return send_sb(ct, NULL, 0, 0, false);
+	if (ct->sbc[1])
+		return send_sb(ct, 0, NULL, 0, 0, false);
 
-	for (i = 0; i < ct->conf.inputs_num; i++)
-		len += strlen(ct->conf.inputs[i]) + 1;
+	for (i = 0; i < num_inputs; i++)
+		len += strlen(lls_input(i, ct->lpr)) + 1;
 	p = command = para_malloc(len);
-	for (i = 0; i < ct->conf.inputs_num; i++) {
-		strcpy(p, ct->conf.inputs[i]);
-		p += strlen(ct->conf.inputs[i]) + 1;
+	for (i = 0; i < num_inputs; i++) {
+		const char *str = lls_input(i, ct->lpr);
+		strcpy(p, str);
+		p += strlen(str) + 1;
 	}
 	PARA_DEBUG_LOG("--> %s\n", command);
-	return send_sb(ct, command, len, SBD_COMMAND, false);
+	return send_sb(ct, 0, command, len, SBD_COMMAND, false);
 }
 
-/**
+/*
  * The post select hook for client commands.
  *
- * \param s Pointer to the scheduler.
- * \param t Pointer to the task struct for this command.
- *
  * Depending on the current state of the connection and the status of the read
- * and write fd sets of \a s, this function performs the necessary steps to
- * authenticate the connection, to send the command given by \a t->private_data
+ * and write fd sets of s, this function performs the necessary steps to
+ * authenticate the connection, to send the command given by t->private_data
  * and to receive para_server's output, if any.
- *
- * \sa struct sched, struct task.
  */
-static int client_post_select(struct sched *s, struct task *t)
+static int client_post_select(struct sched *s, void *context)
 {
-	struct client_task *ct = container_of(t, struct client_task, task);
-	struct btr_node *btrn = ct->btrn;
+	struct client_task *ct = context;
 	int ret = 0;
 	size_t n;
 	char buf[CLIENT_BUFSIZE];
 
-	ret = task_get_notification(t);
+	ret = task_get_notification(ct->task);
 	if (ret < 0)
 		goto out;
 	if (ct->scc.fd < 0)
 		return 0;
 	switch (ct->status) {
 	case CL_CONNECTED: /* receive welcome message */
-		ret = client_recv_buffer(ct, &s->rfds, buf, sizeof(buf), &n);
+		ret = read_nonblock(ct->scc.fd, buf, sizeof(buf), &s->rfds, &n);
 		if (ret < 0 || n == 0)
 			goto out;
 		ct->features = parse_features(buf);
@@ -350,11 +290,8 @@ static int client_post_select(struct sched *s, struct task *t)
 	case CL_RECEIVED_WELCOME: /* send auth command */
 		if (!FD_ISSET(ct->scc.fd, &s->wfds))
 			return 0;
-		if (has_feature("sideband", ct)) {
-			ct->use_sideband = true;
-			sprintf(buf, AUTH_REQUEST_MSG "%s sideband", ct->user);
-		} else
-			sprintf(buf, AUTH_REQUEST_MSG "%s", ct->user);
+		sprintf(buf, AUTH_REQUEST_MSG "%s sideband,aes_ctr128",
+			ct->user);
 		PARA_INFO_LOG("--> %s\n", buf);
 		ret = write_buffer(ct->scc.fd, buf);
 		if (ret < 0)
@@ -369,35 +306,23 @@ static int client_post_select(struct sched *s, struct task *t)
 		{
 		/* decrypted challenge/session key buffer */
 		unsigned char crypt_buf[1024];
-		/* the SHA1 of the decrypted challenge */
+		struct sb_buffer sbb;
 
-		if (ct->use_sideband) {
-			struct sb_buffer sbb;
-			ret = recv_sb(ct, &s->rfds, &sbb);
-			if (ret <= 0)
-				goto out;
-			if (sbb.band != SBD_CHALLENGE) {
-				ret = -E_BAD_BAND;
-				free(sbb.iov.iov_base);
-					goto out;
-			}
-			n = sbb.iov.iov_len;
-			PARA_INFO_LOG("<-- [challenge] (%zu bytes)\n", n);
-			ret = priv_decrypt(ct->key_file, crypt_buf,
-				sbb.iov.iov_base, n);
+		ret = recv_sb(ct, &s->rfds, &sbb);
+		if (ret <= 0)
+			goto out;
+		if (sbb.band != SBD_CHALLENGE) {
+			ret = -E_BAD_BAND;
 			free(sbb.iov.iov_base);
-			if (ret < 0)
-				goto out;
-		} else {
-			ret = client_recv_buffer(ct, &s->rfds, buf, sizeof(buf), &n);
-			if (ret < 0 || n == 0)
-				goto out;
-			PARA_INFO_LOG("<-- [challenge] (%zu bytes)\n", n);
-			ret = priv_decrypt(ct->key_file, crypt_buf,
-				(unsigned char *)buf, n);
-			if (ret < 0)
 				goto out;
 		}
+		n = sbb.iov.iov_len;
+		PARA_INFO_LOG("<-- [challenge] (%zu bytes)\n", n);
+		ret = priv_decrypt(ct->key_file, crypt_buf,
+			sbb.iov.iov_base, n);
+		free(sbb.iov.iov_base);
+		if (ret < 0)
+			goto out;
 		ct->challenge_hash = para_malloc(HASH_SIZE);
 		hash_function((char *)crypt_buf, CHALLENGE_SIZE, ct->challenge_hash);
 		ct->scc.send = sc_new(crypt_buf + CHALLENGE_SIZE, SESSION_KEY_LEN);
@@ -409,162 +334,110 @@ static int client_post_select(struct sched *s, struct task *t)
 		return 0;
 		}
 	case CL_RECEIVED_CHALLENGE:
-		if (ct->use_sideband) {
-			ret = send_sb(ct, ct->challenge_hash, HASH_SIZE,
-				SBD_CHALLENGE_RESPONSE, false);
-			if (ret != 0)
-				ct->challenge_hash = NULL;
-			if (ret <= 0)
-				goto out;
-		} else {
-			ret = write_all(ct->scc.fd, (char *)ct->challenge_hash, HASH_SIZE);
-			if (ret < 0)
-				goto out;
-		}
+		ret = send_sb(ct, 0, ct->challenge_hash, HASH_SIZE,
+			SBD_CHALLENGE_RESPONSE, false);
+		if (ret != 0)
+			ct->challenge_hash = NULL;
+		if (ret <= 0)
+			goto out;
 		ct->status = CL_SENT_CH_RESPONSE;
 		goto out;
 	case CL_SENT_CH_RESPONSE: /* read server response */
 		{
-		if (ct->use_sideband) {
-			struct sb_buffer sbb;
-			ret = recv_sb(ct, &s->rfds, &sbb);
-			if (ret <= 0)
-				goto out;
-			free(sbb.iov.iov_base);
-			if (sbb.band != SBD_PROCEED)
-				ret = -E_BAD_BAND;
-			else
-				ct->status = CL_RECEIVED_PROCEED;
-			goto out;
-		}
-		ret = client_recv_buffer(ct, &s->rfds, buf, sizeof(buf), &n);
-		if (ret < 0 || n == 0)
-			goto out;
-		/* check if server has sent "Proceed" message */
-		ret = -E_CLIENT_AUTH;
-		if (n < PROCEED_MSG_LEN)
-			goto out;
-		if (!strstr(buf, PROCEED_MSG))
+		struct sb_buffer sbb;
+		ret = recv_sb(ct, &s->rfds, &sbb);
+		if (ret <= 0)
 			goto out;
-		ct->status = CL_RECEIVED_PROCEED;
-		return 0;
+		free(sbb.iov.iov_base);
+		if (sbb.band != SBD_PROCEED)
+			ret = -E_BAD_BAND;
+		else
+			ct->status = CL_RECEIVED_PROCEED;
+		goto out;
 		}
 	case CL_RECEIVED_PROCEED: /* concat args and send command */
 		{
-		int i;
-		char *command = NULL;
 		if (!FD_ISSET(ct->scc.fd, &s->wfds))
 			return 0;
-		if (ct->use_sideband) {
-			ret = send_sb_command(ct);
-			if (ret <= 0)
-				goto out;
-			ct->status = CL_SENT_COMMAND;
-			return 0;
-		}
-		for (i = 0; i < ct->conf.inputs_num; i++) {
-			char *tmp = command;
-			command = make_message("%s\n%s", command?
-				command : "", ct->conf.inputs[i]);
-			free(tmp);
-		}
-		command = para_strcat(command, EOC_MSG "\n");
-		PARA_DEBUG_LOG("--> %s\n", command);
-		ret = sc_send_buffer(&ct->scc, command);
-		free(command);
-		if (ret < 0)
+		ret = send_sb_command(ct);
+		if (ret <= 0)
 			goto out;
-		ct->status = CL_SENT_COMMAND;
+		ct->status = CL_EXECUTING;
 		return 0;
 		}
-	case CL_SENT_COMMAND:
-		{
-		char *buf2;
-		if (ct->use_sideband) {
-			struct sb_buffer sbb;
-			ret = recv_sb(ct, &s->rfds, &sbb);
-			if (ret <= 0)
-				goto out;
-			if (sbb.band == SBD_AWAITING_DATA) {
-				ct->status = CL_SENDING;
-				free(sbb.iov.iov_base);
-				goto out;
+	case CL_SENDING:
+		if (ct->btrn[1]) {
+			char *buf2;
+			size_t sz;
+			ret = btr_node_status(ct->btrn[1], 0, BTR_NT_LEAF);
+			if (ret == -E_BTR_EOF) {
+				/* empty blob data packet indicates EOF */
+				PARA_INFO_LOG("blob sent\n");
+				ret = send_sb(ct, 1, NULL, 0, SBD_BLOB_DATA, true);
+				if (ret >= 0)
+					ret = -E_BTR_EOF;
 			}
-			ct->status = CL_RECEIVING;
-			ret = dispatch_sbb(ct, &sbb);
-			goto out;
-		}
-		/* can not use "buf" here because we need a malloced buffer */
-		buf2 = para_malloc(CLIENT_BUFSIZE);
-		ret = client_recv_buffer(ct, &s->rfds, buf2, CLIENT_BUFSIZE, &n);
-		if (n > 0) {
-			if (strstr(buf2, AWAITING_DATA_MSG)) {
-				free(buf2);
-				ct->status = CL_SENDING;
-				return 0;
+			if (ret < 0)
+				goto close1;
+			if (ret > 0 && FD_ISSET(ct->scc.fd, &s->wfds)) {
+				sz = btr_next_buffer(ct->btrn[1], &buf2);
+				assert(sz);
+				ret = send_sb(ct, 1, buf2, sz, SBD_BLOB_DATA, true);
+				if (ret < 0)
+					goto close1;
+				if (ret > 0)
+					btr_consume(ct->btrn[1], sz);
 			}
-			ct->status = CL_RECEIVING;
-			btr_add_output(buf2, n, btrn);
-		} else
-			free(buf2);
-		goto out;
 		}
-	case CL_SENDING:
-		{
-		char *buf2;
-		size_t sz;
-		ret = btr_node_status(btrn, 0, BTR_NT_LEAF);
-		if (ret < 0)
-			goto out;
-		if (ret == 0)
-			return 0;
-		if (!FD_ISSET(ct->scc.fd, &s->wfds))
-			return 0;
-		sz = btr_next_buffer(btrn, &buf2);
-		ret = sc_send_bin_buffer(&ct->scc, buf2, sz);
-		if (ret < 0)
-			goto out;
-		btr_consume(btrn, sz);
-		return 0;
-		}
-	case CL_RECEIVING:
-		{
-		char *buf2;
-		ret = btr_node_status(btrn, 0, BTR_NT_ROOT);
-		if (ret < 0)
-			goto out;
-		if (ret == 0)
-			return 0;
-		/*
-		 * The FD_ISSET() is not strictly necessary, but is allows us
-		 * to skip the malloc below if there is nothing to read anyway.
-		 */
-		if (!FD_ISSET(ct->scc.fd, &s->rfds))
-			return 0;
-		if (ct->use_sideband) {
-			struct sb_buffer sbb;
-			ret = recv_sb(ct, &s->rfds, &sbb);
-			if (ret > 0)
-				ret = dispatch_sbb(ct, &sbb);
-			goto out;
+		/* fall through */
+	case CL_EXECUTING:
+		if (ct->btrn[0]) {
+			ret = btr_node_status(ct->btrn[0], 0, BTR_NT_ROOT);
+			if (ret < 0)
+				goto close0;
+			if (ret > 0 && FD_ISSET(ct->scc.fd, &s->rfds)) {
+				struct sb_buffer sbb;
+				ret = recv_sb(ct, &s->rfds, &sbb);
+				if (ret < 0)
+					goto close0;
+				if (ret > 0) {
+					ret = dispatch_sbb(ct, &sbb);
+					if (ret < 0)
+						goto close0;
+				}
+			}
 		}
-		buf2 = para_malloc(CLIENT_BUFSIZE);
-		ret = client_recv_buffer(ct, &s->rfds, buf2, CLIENT_BUFSIZE, &n);
-		if (n > 0) {
-			buf2 = para_realloc(buf2, n);
-			btr_add_output(buf2, n, btrn);
-		} else
-			free(buf2);
+		ret = 0;
 		goto out;
-		}
 	}
+close1:
+	PARA_INFO_LOG("channel 1: %s\n", para_strerror(-ret));
+	btr_remove_node(&ct->btrn[1]);
+	if (ct->btrn[0])
+		return 0;
+	goto out;
+close0:
+	PARA_INFO_LOG("channel 0: %s\n", para_strerror(-ret));
+	btr_remove_node(&ct->btrn[0]);
+	if (ct->btrn[1] && ct->status == CL_SENDING)
+		return 0;
 out:
-	if (ret < 0) {
-		if (!ct->use_sideband && ret != -E_SERVER_EOF &&
-				ret != -E_BTR_EOF && ret != -E_EOF)
-			PARA_ERROR_LOG("%s\n", para_strerror(-ret));
-		btr_remove_node(&ct->btrn);
+	if (ret >= 0)
+		return 0;
+	btr_remove_node(&ct->btrn[0]);
+	btr_remove_node(&ct->btrn[1]);
+	if (ret != -E_SERVER_CMD_SUCCESS && ret != -E_SERVER_CMD_FAILURE)
+		PARA_ERROR_LOG("%s\n", para_strerror(-ret));
+	if (ct->scc.fd >= 0) {
+		close(ct->scc.fd);
+		ct->scc.fd = -1;
 	}
+	free_argv(ct->features);
+	ct->features = NULL;
+	sc_free(ct->scc.recv);
+	ct->scc.recv = NULL;
+	sc_free(ct->scc.send);
+	ct->scc.send = NULL;
 	return ret;
 }
 
@@ -585,12 +458,12 @@ int client_connect(struct client_task *ct, struct sched *s,
 		struct btr_node *parent, struct btr_node *child)
 {
 	int ret;
+	const char *host = CLIENT_OPT_STRING_VAL(HOSTNAME, ct->lpr);
+	uint32_t port = CLIENT_OPT_UINT32_VAL(SERVER_PORT, ct->lpr);
 
-	PARA_NOTICE_LOG("connecting %s:%d\n", ct->conf.hostname_arg,
-		ct->conf.server_port_arg);
+	PARA_NOTICE_LOG("connecting %s:%u\n", host, port);
 	ct->scc.fd = -1;
-	ret = para_connect_simple(IPPROTO_TCP, ct->conf.hostname_arg,
-					       ct->conf.server_port_arg);
+	ret = para_connect_simple(IPPROTO_TCP, host, port);
 	if (ret < 0)
 		return ret;
 	ct->scc.fd = ret;
@@ -598,13 +471,17 @@ int client_connect(struct client_task *ct, struct sched *s,
 	if (ret < 0)
 		goto err_out;
 	ct->status = CL_CONNECTED;
-	ct->btrn = btr_new_node(&(struct btr_node_description)
-		EMBRACE(.name = "client", .parent = parent, .child = child));
-	ct->task.pre_select = client_pre_select;
-	ct->task.post_select = client_post_select;
-	ct->task.error = 0;
-	sprintf(ct->task.status, "client");
-	register_task(s, &ct->task);
+	ct->btrn[0] = btr_new_node(&(struct btr_node_description)
+		EMBRACE(.name = "client recv", .parent = NULL, .child = child));
+	ct->btrn[1] = btr_new_node(&(struct btr_node_description)
+		EMBRACE(.name = "client send", .parent = parent, .child = NULL));
+
+	ct->task = task_register(&(struct task_info) {
+		.name = "client",
+		.pre_select = client_pre_select,
+		.post_select = client_post_select,
+		.context = ct,
+	}, s);
 	return 1;
 err_out:
 	close(ct->scc.fd);
@@ -612,13 +489,19 @@ err_out:
 	return ret;
 }
 
-__noreturn static void print_help_and_die(struct client_task *ct)
+static void handle_help_flag(struct lls_parse_result *lpr)
 {
-	struct ggo_help h = DEFINE_GGO_HELP(client);
-	bool d = ct->conf.detailed_help_given;
+	char *help;
 
-	ggo_print_help(&h, d? GPH_STANDARD_FLAGS_DETAILED : GPH_STANDARD_FLAGS);
-	exit(0);
+	if (CLIENT_OPT_GIVEN(DETAILED_HELP, lpr))
+		help = lls_long_help(CLIENT_CMD_PTR);
+	else if (CLIENT_OPT_GIVEN(HELP, lpr))
+		help = lls_short_help(CLIENT_CMD_PTR);
+	else
+		return;
+	printf("%s\n", help);
+	free(help);
+	exit(EXIT_SUCCESS);
 }
 
 /**
@@ -642,65 +525,90 @@ __noreturn static void print_help_and_die(struct client_task *ct)
 int client_parse_config(int argc, char *argv[], struct client_task **ct_ptr,
 		int *loglevel)
 {
-	char *home = para_homedir();
-	int ret;
-	struct client_task *ct = para_calloc(sizeof(struct client_task));
-
-	*ct_ptr = ct;
-	ct->scc.fd = -1;
-	ret = -E_CLIENT_SYNTAX;
-	if (client_cmdline_parser(argc, argv, &ct->conf))
-		goto out;
-	version_handle_flag("client", ct->conf.version_given);
-	if (ct->conf.help_given || ct->conf.detailed_help_given)
-		print_help_and_die(ct);
-
-	ct->config_file = ct->conf.config_file_given?
-		para_strdup(ct->conf.config_file_arg) :
-		make_message("%s/.paraslash/client.conf", home);
-	ret = file_exists(ct->config_file);
-	if (!ret && ct->conf.config_file_given) {
-		ret = -E_NO_CONFIG;
+	const struct lls_command *cmd = CLIENT_CMD_PTR;
+	void *map;
+	size_t sz;
+	struct lls_parse_result *lpr;
+	int ret, ll;
+	struct client_task *ct;
+	char *cf = NULL, *kf = NULL, *user, *errctx, *home = para_homedir();
+
+	ret = lls(lls_parse(argc, argv, cmd, &lpr, &errctx));
+	if (ret < 0)
 		goto out;
-	}
-	if (ret) {
-		struct client_cmdline_parser_params params = {
-			.override = 0,
-			.initialize = 0,
-			.check_required = 0,
-			.check_ambiguity = 0,
-			.print_errors = 0
-		};
-		ret = -E_BAD_CONFIG;
-		if (client_cmdline_parser_config_file(ct->config_file,
-			&ct->conf, &params))
+	ll = CLIENT_OPT_UINT32_VAL(LOGLEVEL, lpr);
+	version_handle_flag("client", CLIENT_OPT_GIVEN(VERSION, lpr));
+	handle_help_flag(lpr);
+
+	if (CLIENT_OPT_GIVEN(CONFIG_FILE, lpr))
+		cf = para_strdup(CLIENT_OPT_STRING_VAL(CONFIG_FILE, lpr));
+	else
+		cf = make_message("%s/.paraslash/client.conf", home);
+	ret = mmap_full_file(cf, O_RDONLY, &map, &sz, NULL);
+	if (ret < 0) {
+		if (ret != -E_EMPTY && ret != -ERRNO_TO_PARA_ERROR(ENOENT))
 			goto out;
+		if (ret == -ERRNO_TO_PARA_ERROR(ENOENT) &&
+				CLIENT_OPT_GIVEN(CONFIG_FILE, lpr))
+			goto out;
+	} else {
+		int cf_argc;
+		char **cf_argv;
+		struct lls_parse_result *cf_lpr, *merged_lpr;
+		ret = lls(lls_convert_config(map, sz, NULL, &cf_argv, &errctx));
+		para_munmap(map, sz);
+		if (ret < 0)
+			goto out;
+		cf_argc = ret;
+		ret = lls(lls_parse(cf_argc, cf_argv, cmd, &cf_lpr, &errctx));
+		lls_free_argv(cf_argv);
+		if (ret < 0)
+			goto out;
+		ret = lls(lls_merge(lpr, cf_lpr, cmd, &merged_lpr,
+			&errctx));
+		lls_free_parse_result(cf_lpr, cmd);
+		if (ret < 0)
+			goto out;
+		lls_free_parse_result(lpr, cmd);
+		lpr = merged_lpr;
 	}
-	ct->user = ct->conf.user_given?
-		para_strdup(ct->conf.user_arg) : para_logname();
+	/* success */
+	user = CLIENT_OPT_GIVEN(USER, lpr)?
+		para_strdup(CLIENT_OPT_STRING_VAL(USER, lpr)) : para_logname();
 
-	if (ct->conf.key_file_given)
-		ct->key_file = para_strdup(ct->conf.key_file_arg);
+	if (CLIENT_OPT_GIVEN(KEY_FILE, lpr))
+		kf = para_strdup(CLIENT_OPT_STRING_VAL(KEY_FILE, lpr));
 	else {
-		ct->key_file = make_message("%s/.paraslash/key.%s",
-			home, ct->user);
-		if (!file_exists(ct->key_file)) {
-			free(ct->key_file);
-			ct->key_file = make_message("%s/.ssh/id_rsa", home);
+		kf = make_message("%s/.paraslash/key.%s", home, user);
+		if (!file_exists(kf)) {
+			free(kf);
+			kf = make_message("%s/.ssh/id_rsa", home);
 		}
 	}
-
+	PARA_INFO_LOG("user: %s\n", user);
+	PARA_INFO_LOG("config file: %s\n", cf);
+	PARA_INFO_LOG("key file: %s\n", kf);
+	PARA_INFO_LOG("loglevel: %d\n", ll);
+	ct = para_calloc(sizeof(*ct));
+	ct->scc.fd = -1;
+	ct->lpr = lpr;
+	ct->key_file = kf;
+	ct->config_file = cf;
+	ct->user = user;
+	*ct_ptr = ct;
 	if (loglevel)
-		*loglevel = get_loglevel_by_name(ct->conf.loglevel_arg);
-	PARA_INFO_LOG("loglevel: %s\n", ct->conf.loglevel_arg);
-	PARA_INFO_LOG("config_file: %s\n", ct->config_file);
-	PARA_INFO_LOG("key_file: %s\n", ct->key_file);
-	ret = ct->conf.inputs_num;
+		*loglevel = ll;
+	ret = lls_num_inputs(lpr);
 out:
 	free(home);
 	if (ret < 0) {
+		if (errctx)
+			PARA_ERROR_LOG("%s\n", errctx);
+		free(errctx);
 		PARA_ERROR_LOG("%s\n", para_strerror(-ret));
-		client_close(ct);
+		lls_free_parse_result(lpr, cmd);
+		free(cf);
+		free(kf);
 		*ct_ptr = NULL;
 	}
 	return ret;