[Pkg-shadow-commits] r418 - branches/experimental/debian/patches

[Nicolas FRANCOIS]

Author: nekral-guest
Date: 2005-07-27 20:51:30 +0000 (Wed, 27 Jul 2005)
New Revision: 418

Modified:
   branches/experimental/debian/patches/423_su_arguments_are_concatenated
Log:
Uninitialized variable.


Modified: branches/experimental/debian/patches/423_su_arguments_are_concatenated
===================================================================
--- branches/experimental/debian/patches/423_su_arguments_are_concatenated	2005-07-27 18:53:49 UTC (rev 417)
+++ branches/experimental/debian/patches/423_su_arguments_are_concatenated	2005-07-27 20:51:30 UTC (rev 418)
@@ -9,20 +9,20 @@
 Note: the fix of the man page is still missing.
       (to be taken from the trunk)
 
-Index: shadow-4.0.10/src/su.c
+Index: shadow-4.0.11.1/src/su.c
 ===================================================================
---- shadow-4.0.10.orig/src/su.c	2005-07-24 10:50:14.000000000 +0200
-+++ shadow-4.0.10/src/su.c	2005-07-24 10:51:11.000000000 +0200
-@@ -218,6 +218,8 @@
+--- shadow-4.0.11.1.orig/src/su.c	2005-07-27 22:01:02.000000000 +0200
++++ shadow-4.0.11.1/src/su.c	2005-07-27 22:04:09.000000000 +0200
+@@ -214,6 +214,8 @@
  	int status;
  	int ret;
  	int argno = 0;
-+	int cmd_len = 0;
++	unsigned int cmd_len = 0;
 +	char *cmd = NULL;
  
  	/* There is at most 4 arguments (arg0, '-c', <command>, NULL) + the
  	 * number of arguments in the additional_args array. */
-@@ -243,6 +245,16 @@
+@@ -239,6 +241,16 @@
  	if (command) {
  		args[argno++] = "-c";
  		args[argno++] = command;
@@ -39,7 +39,7 @@
  	}
  
  	if (additional_args) {
-@@ -253,6 +265,29 @@
+@@ -249,6 +261,30 @@
  
  	args[argno] = NULL;
  
@@ -57,6 +57,7 @@
 +			cmd_len += strlen(*parg) + 1;
 +		}
 +		cmd = (char *) xmalloc(sizeof(char) * cmd_len);
++		cmd[0] = '\0';
 +		for (parg = &args[2]; *parg; parg++) {
 +			strcat (cmd, " ");
 +			strcat (cmd, *parg);