X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=client_common.c;h=cee76f8b9449d8b9bcd1b9af1db83ec7bdf03354;hp=53f7b5a96e524214798e549d755141a5bb48c3ed;hb=277ed4a605f68118aff9e671f16c0ac6edb1d55a;hpb=9c2a265397821d91ec628f549516b25757f5c801

diff --git a/client_common.c b/client_common.c
index 53f7b5a9..cee76f8b 100644
--- a/client_common.c
+++ b/client_common.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 1997-2012 Andre Noll <maan@systemlinux.org>
+ * Copyright (C) 1997-2013 Andre Noll <maan@systemlinux.org>
  *
  * Licensed under the GPL v2. For licencing details see COPYING.
  */
@@ -44,12 +44,12 @@ void client_disconnect(struct client_task *ct)
 	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_free_node(ct->btrn);
-	ct->btrn = NULL;
+	btr_remove_node(&ct->btrn);
 }
 
 /**
@@ -240,11 +240,79 @@ static char **parse_features(char *buf)
 	return features;
 }
 
+static int dispatch_sbb(struct client_task *ct, struct sb_buffer *sbb)
+{
+	int ret;
+	const char *designator[] = {SB_DESIGNATORS_ARRAY};
+
+	if (!sbb)
+		return 0;
+	if (sbb->band < NUM_SB_DESIGNATORS)
+		PARA_DEBUG_LOG("band: %s\n", designator[sbb->band]);
+
+	switch (sbb->band) {
+	case SBD_OUTPUT:
+		if (iov_valid(&sbb->iov))
+			btr_add_output(sbb->iov.iov_base, sbb->iov.iov_len,
+				ct->btrn);
+		ret = 1;
+		goto out;
+	case SBD_DEBUG_LOG:
+	case SBD_INFO_LOG:
+	case SBD_NOTICE_LOG:
+	case SBD_WARNING_LOG:
+	case SBD_ERROR_LOG:
+	case SBD_CRIT_LOG:
+	case SBD_EMERG_LOG:
+		if (iov_valid(&sbb->iov)) {
+			int ll = sbb->band - SBD_DEBUG_LOG;
+			para_log(ll, "remote: %s", (char *)sbb->iov.iov_base);
+		}
+		ret = 1;
+		goto deallocate;
+	case SBD_EXIT__SUCCESS:
+		ret = -E_SERVER_CMD_SUCCESS;
+		goto deallocate;
+	case SBD_EXIT__FAILURE:
+		ret = -E_SERVER_CMD_FAILURE;
+		goto deallocate;
+	default:
+		PARA_ERROR_LOG("invalid band %d\n", sbb->band);
+		ret = -E_BAD_BAND;
+		goto deallocate;
+	}
+deallocate:
+	free(sbb->iov.iov_base);
+out:
+	sbb->iov.iov_base = NULL;
+	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;
+
+	if (ct->sbc)
+		return send_sb(ct, NULL, 0, 0, false);
+
+	for (i = 0; i < ct->conf.inputs_num; i++)
+		len += strlen(ct->conf.inputs[i]) + 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;
+	}
+	PARA_DEBUG_LOG("--> %s\n", command);
+	return send_sb(ct, command, len, SBD_COMMAND, false);
+}
+
 /**
  * The post select hook for client commands.
  *
@@ -266,7 +334,9 @@ static void client_post_select(struct sched *s, struct task *t)
 	size_t n;
 	char buf[CLIENT_BUFSIZE];
 
-	t->error = 0;
+	ret = task_get_notification(t);
+	if (ret < 0)
+		goto out;
 	if (ct->scc.fd < 0)
 		return;
 	switch (ct->status) {
@@ -275,16 +345,17 @@ static void client_post_select(struct sched *s, struct task *t)
 		if (ret < 0 || n == 0)
 			goto out;
 		ct->features = parse_features(buf);
+		if (!has_feature("sideband", ct)) {
+			PARA_ERROR_LOG("server has no sideband support\n");
+			ret = -E_INCOMPAT_FEAT;
+			goto out;
+		}
 		ct->status = CL_RECEIVED_WELCOME;
 		return;
 	case CL_RECEIVED_WELCOME: /* send auth command */
 		if (!FD_ISSET(ct->scc.fd, &s->wfds))
 			return;
-		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", ct->user);
 		PARA_INFO_LOG("--> %s\n", buf);
 		ret = write_buffer(ct->scc.fd, buf);
 		if (ret < 0)
@@ -299,35 +370,23 @@ static void 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);
@@ -339,83 +398,50 @@ static void client_post_select(struct sched *s, struct task *t)
 		return;
 		}
 	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, 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;
+		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;
-		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;
 		return;
 		}
 	case CL_SENT_COMMAND:
 		{
-		char *buf2;
-		/* 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;
-			}
-			ct->status = CL_RECEIVING;
-			btr_add_output(buf2, n, btrn);
-		} else
-			free(buf2);
+		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;
+		}
+		ct->status = CL_RECEIVING;
+		ret = dispatch_sbb(ct, &sbb);
 		goto out;
 		}
 	case CL_SENDING:
@@ -438,7 +464,7 @@ static void client_post_select(struct sched *s, struct task *t)
 		}
 	case CL_RECEIVING:
 		{
-		char *buf2;
+		struct sb_buffer sbb;
 		ret = btr_node_status(btrn, 0, BTR_NT_ROOT);
 		if (ret < 0)
 			goto out;
@@ -450,23 +476,16 @@ static void client_post_select(struct sched *s, struct task *t)
 		 */
 		if (!FD_ISSET(ct->scc.fd, &s->rfds))
 			return;
-		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 = recv_sb(ct, &s->rfds, &sbb);
+		if (ret > 0)
+			ret = dispatch_sbb(ct, &sbb);
 		goto out;
 		}
 	}
 out:
 	t->error = ret;
-	if (ret < 0) {
-		if (ret != -E_SERVER_EOF && ret != -E_BTR_EOF && ret != -E_EOF)
-			PARA_ERROR_LOG("%s\n", para_strerror(-t->error));
-		btr_remove_node(btrn);
-	}
+	if (ret < 0)
+		btr_remove_node(&ct->btrn);
 }
 
 /**