X-Git-Url: http://git.tuebingen.mpg.de/?a=blobdiff_plain;f=command.c;h=2171457168c294861ece3c644efe34709f74f5cf;hb=478cc53ff8efa50861d0b310f4b0b5a9a572dc91;hp=8ea725de7898ef817a62c0ba2755c73a0f744572;hpb=c75c35b176cdacf157d27617e954961c88f33975;p=paraslash.git

diff --git a/command.c b/command.c
index 8ea725de..21714571 100644
--- a/command.c
+++ b/command.c
@@ -717,8 +717,7 @@ EXPORT_SERVER_CMD_HANDLER(ff);
 
 static int com_jmp(struct command_context *cc, struct lls_parse_result *lpr)
 {
-	long unsigned int i;
-	int ret;
+	int i, ret;
 	char *errctx;
 
 	ret = lls(lls_check_arg_count(lpr, 1, 1, &errctx));
@@ -726,18 +725,16 @@ static int com_jmp(struct command_context *cc, struct lls_parse_result *lpr)
 		send_errctx(cc, errctx);
 		return ret;
 	}
-	if (sscanf(lls_input(0, lpr), "%lu", &i) <= 0)
+	if (sscanf(lls_input(0, lpr), "%d", &i) <= 0)
+		return -ERRNO_TO_PARA_ERROR(EINVAL);
+	if (i < 0 || i > 100)
 		return -ERRNO_TO_PARA_ERROR(EINVAL);
 	mutex_lock(mmd_mutex);
 	ret = -E_NO_AUDIO_FILE;
 	if (!mmd->afd.afhi.chunks_total)
 		goto out;
-	if (i > 100)
-		i = 100;
-	PARA_INFO_LOG("jumping to %lu%%\n", i);
+	PARA_INFO_LOG("jumping to %d%%\n", i);
 	mmd->repos_request = (mmd->afd.afhi.chunks_total * i + 50) / 100;
-	PARA_INFO_LOG("sent: %lu, offset before jmp: %li\n",
-		mmd->chunks_sent, mmd->offset);
 	mmd->new_vss_status_flags |= VSS_REPOS;
 	mmd->new_vss_status_flags &= ~VSS_NEXT;
 	ret = 1;
@@ -765,7 +762,7 @@ static void reset_signals(void)
 }
 
 struct connection_features {
-	int dummy; /* none at the moment */
+	bool sha256_requested; /* can be removed after 0.7.0 */
 };
 
 static int parse_auth_request(char *buf, int len, const struct user **u,
@@ -790,11 +787,22 @@ static int parse_auth_request(char *buf, int len, const struct user **u,
 		*p = '\0';
 		p++;
 		create_argv(p, ",", &features);
+		/*
+		 * Still accept sideband and AES feature requests (as a no-op)
+		 * because some 0.6.x clients request them. The two checks
+		 * below may be removed after 0.7.1.
+		 */
 		for (i = 0; features[i]; i++) {
 			if (strcmp(features[i], "sideband") == 0)
 				continue;
 			if (strcmp(features[i], "aes_ctr128") == 0)
 				continue;
+			/*
+			 * ->sha256_requested can go away after 0.7.0 but the
+			 * check has to stay until 0.9.0.
+			 */
+			if (strcmp(features[i], "sha256") == 0)
+				cf->sha256_requested = true;
 			else {
 				ret = -E_BAD_FEATURE;
 				goto out;
@@ -892,7 +900,7 @@ int handle_connect(int fd)
 {
 	int ret;
 	unsigned char rand_buf[APC_CHALLENGE_SIZE + 2 * SESSION_KEY_LEN];
-	unsigned char challenge_hash[HASH_SIZE];
+	unsigned char challenge_hash[HASH2_SIZE];
 	char *command = NULL, *buf = para_malloc(HANDSHAKE_BUFSIZE) /* must be on the heap */;
 	size_t numbytes;
 	struct command_context cc_struct = {.u = NULL}, *cc = &cc_struct;
@@ -908,7 +916,7 @@ int handle_connect(int fd)
 	/* send Welcome message */
 	ret = write_va_buffer(fd, "This is para_server, version "
 		PACKAGE_VERSION ".\n"
-		"Features: sideband,aes_ctr128\n"
+		"Features: sha256\n" /* no longer announce this after 0.8.0 */
 	);
 	if (ret < 0)
 		goto net_err;
@@ -956,11 +964,19 @@ int handle_connect(int fd)
 	 * of the random data.
 	 */
 	ret = -E_BAD_AUTH;
-	if (numbytes != HASH_SIZE)
-		goto net_err;
-	hash_function((char *)rand_buf, APC_CHALLENGE_SIZE, challenge_hash);
-	if (memcmp(challenge_hash, buf, HASH_SIZE))
-		goto net_err;
+	if (cf.sha256_requested) {
+		if (numbytes != HASH2_SIZE)
+			goto net_err;
+		hash2_function((char *)rand_buf, APC_CHALLENGE_SIZE, challenge_hash);
+		if (memcmp(challenge_hash, buf, HASH2_SIZE))
+			goto net_err;
+	} else { /* old client. This can be removed after 0.7.0 */
+		if (numbytes != HASH_SIZE)
+			goto net_err;
+		hash_function((char *)rand_buf, APC_CHALLENGE_SIZE, challenge_hash);
+		if (memcmp(challenge_hash, buf, HASH_SIZE))
+			goto net_err;
+	}
 	/* auth successful */
 	alarm(0);
 	PARA_INFO_LOG("good auth for %s\n", cc->u->name);