X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=command.c;h=5d6a0990597165bc5b17a7cfaf5595e73955ca09;hp=2ef9c5a87b54760ed38fc41e4f3b83391a4aa3f5;hb=16c6fc10f3344e3eec66e51580821d854d49aa62;hpb=a7967902aa937e91d35767b3eba2b77343e8b822

diff --git a/command.c b/command.c
index 2ef9c5a8..5d6a0990 100644
--- a/command.c
+++ b/command.c
@@ -40,6 +40,22 @@
 #include "signal.h"
 #include "version.h"
 
+/* Defines one command of para_server. */
+struct server_command {
+	/* The name of the command. */
+	const char *name;
+	/* Pointer to the function that handles the command. */
+	int (*handler)(struct command_context *);
+	/* The privileges a user must have to execute this command. */
+	unsigned int perms;
+	/* One-line description of the command. */
+	const char *description;
+	/* Summary of the command line options. */
+	const char *usage;
+	/* The long help text. */
+	const char *help;
+};
+
 static struct server_command afs_cmds[] = {DEFINE_AFS_CMD_ARRAY};
 static struct server_command server_cmds[] = {DEFINE_SERVER_CMD_ARRAY};
 
@@ -112,21 +128,15 @@ static char *vss_get_status_flags(unsigned int flags)
 static unsigned get_status(struct misc_meta_data *nmmd, int parser_friendly,
 		char **result)
 {
-	char mtime[30] = "";
 	char *status, *flags; /* vss status info */
 	/* nobody updates our version of "now" */
 	long offset = (nmmd->offset + 500) / 1000;
 	struct timeval current_time;
-	struct tm mtime_tm;
 	struct para_buffer b = {.flags = parser_friendly? PBF_SIZE_PREFIX : 0};
 
 	/* report real status */
 	status = vss_status_tohuman(nmmd->vss_status_flags);
 	flags = vss_get_status_flags(nmmd->vss_status_flags);
-	if (nmmd->size) { /* parent currently has an audio file open */
-		localtime_r(&nmmd->mtime, &mtime_tm);
-		strftime(mtime, 29, "%b %d %Y", &mtime_tm);
-	}
 	clock_get_realtime(&current_time);
 	/*
 	 * The calls to WRITE_STATUS_ITEM() below never fail because
@@ -134,8 +144,6 @@ static unsigned get_status(struct misc_meta_data *nmmd, int parser_friendly,
 	 * is not smart enough to prove this and complains nevertheless.
 	 * Casting the return value to void silences clang.
 	 */
-	(void)WRITE_STATUS_ITEM(&b, SI_FILE_SIZE, "%zu\n", nmmd->size / 1024);
-	(void)WRITE_STATUS_ITEM(&b, SI_MTIME, "%s\n", mtime);
 	(void)WRITE_STATUS_ITEM(&b, SI_STATUS, "%s\n", status);
 	(void)WRITE_STATUS_ITEM(&b, SI_STATUS_FLAGS, "%s\n", flags);
 	(void)WRITE_STATUS_ITEM(&b, SI_OFFSET, "%li\n", offset);
@@ -780,7 +788,7 @@ static int com_tasks(struct command_context *cc)
  * check if perms are sufficient to exec a command having perms cmd_perms.
  * Returns 0 if perms are sufficient, -E_PERM otherwise.
  */
-static int check_perms(unsigned int perms, struct server_command *cmd_ptr)
+static int check_perms(unsigned int perms, const struct server_command *cmd_ptr)
 {
 	PARA_DEBUG_LOG("checking permissions\n");
 	return (cmd_ptr->perms & perms) < cmd_ptr->perms ? -E_PERM : 0;
@@ -842,22 +850,23 @@ out:
 
 #define HANDSHAKE_BUFSIZE 4096
 
-static int parse_sb_command(struct command_context *cc, struct iovec *iov)
+static int run_command(struct command_context *cc, struct iovec *iov,
+		const char *peername)
 {
 	int ret, i;
 	char *p, *end;
+	struct server_command *cmd;
 
-	ret = -E_BAD_CMD;
 	if (iov->iov_base == NULL || iov->iov_len == 0)
-		goto out;
+		return -E_BAD_CMD;
 	p = iov->iov_base;
 	p[iov->iov_len - 1] = '\0'; /* just to be sure */
-	cc->cmd = get_cmd_ptr(p, NULL);
-	if (!cc->cmd)
-		goto out;
-	ret = check_perms(cc->u->perms, cc->cmd);
+	cmd = get_cmd_ptr(p, NULL);
+	if (!cmd)
+		return -E_BAD_CMD;
+	ret = check_perms(cc->u->perms, cmd);
 	if (ret < 0)
-		goto out;
+		return ret;
 	end = iov->iov_base + iov->iov_len;
 	for (i = 0; p < end; i++)
 		p += strlen(p) + 1;
@@ -868,9 +877,15 @@ static int parse_sb_command(struct command_context *cc, struct iovec *iov)
 		p += strlen(p) + 1;
 	}
 	cc->argv[cc->argc] = NULL;
-	ret = cc->argc;
-out:
-	free(iov->iov_base);
+	PARA_NOTICE_LOG("calling com_%s() for %s@%s\n", cmd->name,
+		cc->u->name, peername);
+	ret = cmd->handler(cc);
+	free_argv(cc->argv);
+	mutex_lock(mmd_mutex);
+	mmd->num_commands++;
+	if (ret >= 0 && (cmd->perms & AFS_WRITE))
+		mmd->events++;
+	mutex_unlock(mmd_mutex);
 	return ret;
 }
 
@@ -991,17 +1006,10 @@ __noreturn void handle_connect(int fd, const char *peername)
 	ret = recv_sb(&cc->scc, SBD_COMMAND, MAX_COMMAND_LEN, &iov);
 	if (ret < 0)
 		goto net_err;
-	ret = parse_sb_command(cc, &iov);
+	ret = run_command(cc, &iov, peername);
+	free(iov.iov_base);
 	if (ret < 0)
 		goto err_out;
-	cc->argc = ret;
-	PARA_NOTICE_LOG("calling com_%s() for %s@%s\n", cc->cmd->name,
-		cc->u->name, peername);
-	ret = cc->cmd->handler(cc);
-	free_argv(cc->argv);
-	mutex_lock(mmd_mutex);
-	mmd->num_commands++;
-	mutex_unlock(mmd_mutex);
 	if (ret >= 0)
 		goto out;
 err_out:
@@ -1013,8 +1021,6 @@ out:
 	free(buf);
 	free(command);
 	mutex_lock(mmd_mutex);
-	if (cc->cmd && (cc->cmd->perms & AFS_WRITE) && ret >= 0)
-		mmd->events++;
 	mmd->active_connections--;
 	mutex_unlock(mmd_mutex);
 	if (ret >= 0) {