From 93379cc552745c579a5177b6cc94e3bbb20f33bc Mon Sep 17 00:00:00 2001
From: Kolja Waschk <urjtag@ixo.de>
Date: Sun, 25 Nov 2007 13:08:27 +0000
Subject: [PATCH] Allow arbitrary number of arguments for "shell" command (fix
 for [1837985] "shell" command causes segfault)

git-svn-id: https://urjtag.svn.sourceforge.net/svnroot/urjtag/trunk@785 b68d4a1b-bc3d-0410-92ed-d4ac073336b7
---
 jtag/src/cmd/shell.c | 41 +++++++++++++++++++++++++++++------------
 1 file changed, 29 insertions(+), 12 deletions(-)

diff --git a/jtag/src/cmd/shell.c b/jtag/src/cmd/shell.c
index a1ba2bd7..ee1db899 100644
--- a/jtag/src/cmd/shell.c
+++ b/jtag/src/cmd/shell.c
@@ -24,7 +24,11 @@
 
 #include "sysdep.h"
 
-//#include <stdio.h>
+#include <sys/types.h>
+#include <sys/wait.h>
+#include <unistd.h>
+
+#include <stdio.h>
 #include <string.h>
 #include <stdlib.h>
 //#include "part.h"
@@ -36,8 +40,8 @@
 static int
 cmd_shell_run( char *params[] )
 {
-int n,l1,l2;
-char *t;
+	int i, len, n = cmd_params(params);
+	char *shell_cmd;
 
 	if((n=cmd_params( params )) == 1)
 		return -1;
@@ -46,17 +50,30 @@ char *t;
 	* the following. If you can pass a shell argument past strtok the
 	* please fix this.
 	*/
+	/* The problem is the parser which splits commands into params[]
+	* and doesn't allow quoting. So we concatenate the params[] here
+	* with single spaces, although the original might have different
+	* whitespace (more than one space, tabs, ...) - kawk */
+
+	for(i=1,len=0; i<n; i++) len += (1 + strlen(params[i]));
+
+	shell_cmd = malloc(len);
+	if(shell_cmd == NULL)
+	{
+		printf( _("Out of memory\n") );
+		exit -1;
+	};
 
-	l1 = strlen(params[1]);
-	l2 = strlen(params[2]);
-	t = malloc(l1+l2+2); 	/* space + term */
-	strcpy(t,params[1]);	/* main command */
+	strcpy(shell_cmd, params[1]);
+	for(i=2; i<n; i++)
+	{
+		strcat(shell_cmd, " ");
+		strcat(shell_cmd, params[i]);
+	};
+	printf("Executing '%s'\n", shell_cmd);
 
-	if(n == 3) {
-		*(t+l1)= ' ';		/* add space */
-		strcpy((t+l1+1),params[2]);
-	}
-	system(t);
+	system(shell_cmd);
+	free(shell_cmd);
 
 	return 1;
 }