[janderwald] 28426: more.c should not close INVALID_HANDLE_VALUE patch by Apal remove tab and quit when failing to open file (tested with Windows XP SP2)

Author: janderwald
Date: Mon Aug 20 00:16:10 2007
New Revision: 28426

URL: http://svn.reactos.org/svn/reactos?rev=28426&view=rev
Log:
more.c should not close INVALID_HANDLE_VALUE
patch by Apal
remove tab and quit when failing to open file (tested with Windows XP SP2)

Modified:
    trunk/reactos/base/applications/cmdutils/more/more.c

Modified: trunk/reactos/base/applications/cmdutils/more/more.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/cmdutils…
==============================================================================
--- trunk/reactos/base/applications/cmdutils/more/more.c (original)
+++ trunk/reactos/base/applications/cmdutils/more/more.c Mon Aug 20 00:16:10 2007
@@ -116,25 +116,28 @@
 
 	buff=malloc(4096);
 
-	FlushConsoleInputBuffer (hKeyboard);	
+	FlushConsoleInputBuffer (hKeyboard);
 
 	if(argc > 1)
 	{
 		GetFullPathName(argv[1], MAX_PATH, szFullPath, NULL);
 		hFile = CreateFile (szFullPath, GENERIC_READ,
 	                        0,NULL,OPEN_ALWAYS,0,0);
+        
+		if (hFile == INVALID_HANDLE_VALUE)
+		{
+			ConOutPuts(_T("The file could not be opened"));
+			return 0;
+		}
+	}
+	else
+	{
+		hFile = hStdIn;
 	}
 
 	do
 	{
-		if(hFile != INVALID_HANDLE_VALUE)
-		{			
-			bRet = ReadFile(hFile,buff,4096,&dwRead,NULL);
-		}
-		else
-		{
-			bRet = ReadFile(hStdIn,buff,4096,&dwRead,NULL);
-		}
+		bRet = ReadFile(hFile,buff,4096,&dwRead,NULL);
 
 		for(last=i=0;i<dwRead && bRet;i++)
 		{
@@ -161,7 +164,8 @@
 
 	free (buff);
 	CloseHandle (hKeyboard);
-	CloseHandle (hFile);
+	if (hFile != hStdIn)
+		CloseHandle (hFile);
 
 	return 0;
 }