svn commit: samba r2360 - in trunk/source: client lib

[jra at samba.org]

Author: jra
Date: 2004-09-15 22:49:18 +0000 (Wed, 15 Sep 2004)
New Revision: 2360

WebSVN: http://websvn.samba.org/websvn/changeset.php?rep=samba&path=/trunk/source&rev=2360&nolog=1

Log:
Fix the appalling toktocliplist() fn. Bug found by Luis Benvenutto.
Jeremy.

Modified:
   trunk/source/client/clitar.c
   trunk/source/lib/util_str.c


Changeset:
Modified: trunk/source/client/clitar.c
===================================================================
--- trunk/source/client/clitar.c	2004-09-15 22:35:15 UTC (rev 2359)
+++ trunk/source/client/clitar.c	2004-09-15 22:49:18 UTC (rev 2360)
@@ -1345,8 +1345,9 @@
 int cmd_tar(void)
 {
 	fstring buf;
-	char **argl;
-	int argcl;
+	char **argl = NULL;
+	int argcl = 0;
+	int ret;
 
 	if (!next_token_nr(NULL,buf,NULL,sizeof(buf))) {
 		DEBUG(0,("tar <c|x>[IXbgan] <filename>\n"));
@@ -1357,8 +1358,9 @@
 	if (!tar_parseargs(argcl, argl, buf, 0))
 		return 1;
 
+	ret = process_tar();
 	SAFE_FREE(argl);
-	return process_tar();
+	return ret;
 }
 
 /****************************************************************************

Modified: trunk/source/lib/util_str.c
===================================================================
--- trunk/source/lib/util_str.c	2004-09-15 22:35:15 UTC (rev 2359)
+++ trunk/source/lib/util_str.c	2004-09-15 22:49:18 UTC (rev 2360)
@@ -134,17 +134,20 @@
 	*ctok=ictok;
 	s=(char *)last_ptr;
 	
-	if (!(ret=iret=malloc(ictok*sizeof(char *))))
+	if (!(ret=iret=malloc((ictok+1)*sizeof(char *))))
 		return NULL;
 	
 	while(ictok--) {    
 		*iret++=s;
-		while(*s++)
-			;
-		while(!*s)
-			s++;
+		if (ictok > 0) {
+			while(*s++)
+				;
+			while(!*s)
+				s++;
+		}
 	}
 
+	ret[*ctok] = NULL;
 	return ret;
 }