[greatlrd] 16463: set right errorlevel for cd / chdir, left to do we do not have /d param that are new in win2k maby in older windows also

set right errorlevel for cd / chdir, left to do we do not have /d param
that are new in win2k maby in older windows also
Modified: trunk/reactos/subsys/system/cmd/internal.c
  _____  

Modified: trunk/reactos/subsys/system/cmd/internal.c
--- trunk/reactos/subsys/system/cmd/internal.c	2005-07-06 19:42:41 UTC
(rev 16462)
+++ trunk/reactos/subsys/system/cmd/internal.c	2005-07-06 20:15:36 UTC
(rev 16463)
@@ -172,6 +172,8 @@

 		return 0;
 	}
 
+   nErrorLevel = 0;
+
 	/* The whole param string is our parameter these days. The only
thing we do is eliminating every quotation mark */
 	/* Is it safe to change the characters param is pointing to? I
presume it is, as there doesn't seem to be any
 	post-processing of it after the function call (what would that
accomplish?) */
@@ -258,13 +260,14 @@
 	{
 
 	    hSearch = FindFirstFile(dir, &FileData); 
-        if (hSearch == INVALID_HANDLE_VALUE) 
-        { 
+      if (hSearch == INVALID_HANDLE_VALUE) 
+      { 
 	        ConOutFormatMessage(GetLastError());
-			free (lpOldPath);
-		    lpOldPath = NULL;
-            return 1;
-		}
+          free (lpOldPath);
+          lpOldPath = NULL;
+          nErrorLevel = 1;
+          return 1;
+      }
 
 		
         while (!fFinished) 
@@ -282,6 +285,7 @@
 				 ConOutFormatMessage(GetLastError());
 			     free (lpOldPath);
 		         lpOldPath = NULL;
+         nErrorLevel = 1;
 				 return 1;
 			 }
 				
@@ -293,6 +297,7 @@
             {             
 		     FindClose(hSearch);
 			 ConOutFormatMessage(GetLastError());
+       nErrorLevel = 1;
 			 free (lpOldPath);
 		     lpOldPath = NULL;
 			 return 1;
@@ -301,6 +306,7 @@
 
 		//ErrorMessage (GetLastError(), _T("CD"));
 		ConOutFormatMessage(GetLastError());
+    nErrorLevel = 1;
 
 		/* throw away current directory */
 		free (lpOldPath);