[ros-diffs] [greatlrd] 17808: Make some more dymatic buffer and correct some free()

Make some more dymatic buffer and correct some free()
Modified: trunk/reactos/subsys/system/cmd/cmd.c
  _____

Modified: trunk/reactos/subsys/system/cmd/cmd.c
--- trunk/reactos/subsys/system/cmd/cmd.c	2005-09-11 20:26:36 UTC
(rev 17807)
+++ trunk/reactos/subsys/system/cmd/cmd.c	2005-09-11 20:41:39 UTC
(rev 17808)
@@ -295,7 +295,7 @@

static VOID
 Execute (LPTSTR Full, LPTSTR First, LPTSTR Rest)
 {
-	TCHAR szFullName[CMDLINE_LENGTH];
+	TCHAR *szFullName=NULL;
    TCHAR *first = NULL;
    TCHAR *rest = NULL; 
    TCHAR *full = NULL; 
@@ -321,7 +321,7 @@
    rest =	malloc ( (_tcslen(Rest) + 512) * sizeof(TCHAR)); 
    if (rest == NULL)
    {
-	 free (full);
+	 free (first);
     error_out_of_memory();
     return ;
    }
@@ -329,13 +329,23 @@
    full =	malloc ( (_tcslen(Full) + 512) * sizeof(TCHAR));
    if (full == NULL)
    {
-	 free (full);
+	 free (first);
     free (rest);
     error_out_of_memory();
     return ;
    }

+	szFullName = malloc ( (_tcslen(Full) + 512) * sizeof(TCHAR));
+	if (full == NULL)
+	{
+	 free (first);
+	 free (rest);
+	 free (full);
+	 error_out_of_memory();
+	 return ;
+	}

+
    /* Though it was already parsed once, we have a different set of
rules
       for parsing before we pass to CreateProccess */
    if(!_tcschr(Full,_T('"')))
@@ -398,6 +408,11 @@

if (!working) ConErrResPuts (STRING_FREE_ERROR1);

+		free (first);
+	    free (rest);
+	    free (full);
+		free (szFullName);
+
    	return;
    }

@@ -406,6 +421,11 @@
    if (!SearchForExecutable (first, szFullName))
    {
    	error_bad_command ();
+
+		free (first);
+	    free (rest);
+	    free (full);
+		free (szFullName);
    	return;
    }

@@ -495,7 +515,8 @@

free(first);
  free(rest);
- free(full);
+ free(full); 
+ free (szFullName);
 }