X-Git-Url: http://git.tuebingen.mpg.de/?a=blobdiff_plain;f=command.c;h=84b7d9438fa21a3fd491ab4a0ee0918f6ff9c086;hb=17c140cc24838c2b7049e2b4c17621f4284d53dc;hp=9d5723173e043e3d2d882c3c0948da1064011144;hpb=1c8218665d39f212eb7cddaea70c5ab66d7e26c7;p=paraslash.git

diff --git a/command.c b/command.c
index 9d572317..84b7d943 100644
--- a/command.c
+++ b/command.c
@@ -48,7 +48,6 @@ static const char * const server_command_perms_txt[] = {LSG_SERVER_CMD_AUX_INFOS
 
 extern int mmd_mutex;
 extern struct misc_meta_data *mmd;
-extern struct sender senders[];
 int send_afs_status(struct command_context *cc, int parser_friendly);
 
 static void dummy(__a_unused int s)
@@ -235,10 +234,10 @@ static int check_sender_args(struct command_context *cc,
 		return ret;
 	}
 	arg = lls_input(0, lpr);
-	for (i = 0; senders[i].name; i++)
-		if (!strcmp(senders[i].name, arg))
+	FOR_EACH_SENDER(i)
+		if (strcmp(senders[i]->name, arg) == 0)
 			break;
-	if (!senders[i].name)
+	if (!senders[i])
 		return -E_COMMAND_SYNTAX;
 	scd->sender_num = i;
 	arg = lls_input(1, lpr);
@@ -248,7 +247,7 @@ static int check_sender_args(struct command_context *cc,
 	if (i == NUM_SENDER_CMDS)
 		return -E_COMMAND_SYNTAX;
 	scd->cmd_num = i;
-	if (!senders[scd->sender_num].client_cmds[scd->cmd_num])
+	if (!senders[scd->sender_num]->client_cmds[scd->cmd_num])
 		return -E_SENDER_CMD;
 	switch (scd->cmd_num) {
 	case SENDER_on:
@@ -330,10 +329,10 @@ static int com_sender(struct command_context *cc, struct lls_parse_result *lpr)
 	struct sender_command_data scd;
 
 	if (lls_num_inputs(lpr) == 0) {
-		for (i = 0; senders[i].name; i++) {
+		FOR_EACH_SENDER(i) {
 			char *tmp;
 			ret = xasprintf(&tmp, "%s%s\n", msg? msg : "",
-				senders[i].name);
+				senders[i]->name);
 			free(msg);
 			msg = tmp;
 		}
@@ -344,17 +343,17 @@ static int com_sender(struct command_context *cc, struct lls_parse_result *lpr)
 		if (scd.sender_num < 0)
 			return ret;
 		if (strcmp(lls_input(1, lpr), "status") == 0)
-			msg = senders[scd.sender_num].status();
+			msg = senders[scd.sender_num]->status();
 		else
-			msg = senders[scd.sender_num].help();
+			msg = senders[scd.sender_num]->help();
 		return send_sb(&cc->scc, msg, strlen(msg), SBD_OUTPUT, false);
 	}
 
 	switch (scd.cmd_num) {
 	case SENDER_add:
 	case SENDER_delete:
-		assert(senders[scd.sender_num].resolve_target);
-		ret = senders[scd.sender_num].resolve_target(lls_input(2, lpr),
+		assert(senders[scd.sender_num]->resolve_target);
+		ret = senders[scd.sender_num]->resolve_target(lls_input(2, lpr),
 			&scd);
 		if (ret < 0)
 			return ret;
@@ -394,7 +393,7 @@ static int com_si(struct command_context *cc,
 		"supported audio formats: %s\n",
 		ut, mmd->num_played,
 		(int)getppid(),
-		(int)mmd->afs_pid,
+		(int)afs_pid,
 		mmd->active_connections,
 		mmd->num_commands,
 		mmd->num_connects,
@@ -740,12 +739,11 @@ out:
 }
 EXPORT_SERVER_CMD_HANDLER(jmp);
 
-static int com_tasks(struct command_context *cc,
+/* deprecated, does nothing */
+static int com_tasks(__a_unused struct command_context *cc,
 		__a_unused struct lls_parse_result *lpr)
 {
-	char *tl = server_get_tasks();
-	assert(tl);
-	return send_sb(&cc->scc, tl, strlen(tl), SBD_OUTPUT, false);
+	return 1;
 }
 EXPORT_SERVER_CMD_HANDLER(tasks);
 
@@ -873,13 +871,13 @@ static int run_command(struct command_context *cc, struct iovec *iov)
  * permissions to execute that command, the function calls the corresponding
  * command handler which does argument checking and further processing.
  *
- * In order to cope with a DOS attacks, a timeout is set up which terminates
+ * In order to cope with DOS attacks, a timeout is set up which terminates
  * the function if the connection was not authenticated when the timeout
  * expires.
  *
  * \sa alarm(2), \ref crypt.c, \ref crypt.h.
  */
-__noreturn void handle_connect(int fd)
+int handle_connect(int fd)
 {
 	int ret;
 	unsigned char rand_buf[CHALLENGE_SIZE + 2 * SESSION_KEY_LEN];
@@ -989,5 +987,5 @@ out:
 	}
 	sc_free(cc->scc.recv);
 	sc_free(cc->scc.send);
-	exit(ret < 0? EXIT_FAILURE : EXIT_SUCCESS);
+	return ret;
 }