wineboot cleanup

[Shachar Shemesh]

Changelog
Shachar Shemesh <winecode at sun.consumer.org.il>

programs/wineboot/wineboot.c

    * Cleaned up the code (saving on chaining strings) by CDing to the
      system root directory before doing anything else.
    * Will now rename wininit.ini to wininit.bak instead of erasing it
      as before.


-------------- next part --------------
Index: programs/wineboot/wineboot.c
===================================================================
RCS file: /home/sun/sources/cvs/wine/programs/wineboot/wineboot.c,v
retrieving revision 1.1
diff -u -r1.1 wineboot.c
--- programs/wineboot/wineboot.c	4 Jan 2003 02:52:05 -0000	1.1
+++ programs/wineboot/wineboot.c	6 Jan 2003 17:24:23 -0000
@@ -23,8 +23,6 @@
 
 WINE_DEFAULT_DEBUG_CHANNEL(wineboot);
 
-const char * const RENAME_FILE="wininit.ini";
-const char * const RENAME_FILE_SECTION="[rename]";
 #define MAX_LINE_LENGTH (2*MAX_PATH+2)
 
 static BOOL GetLine( HANDLE hFile, char *buf, size_t buflen )
@@ -67,31 +65,14 @@
  */
 static BOOL wininit()
 {
+    const char * const RENAME_FILE="wininit.ini";
+    const char * const RENAME_FILE_TO="wininit.bak";
+    const char * const RENAME_FILE_SECTION="[rename]";
     char buffer[MAX_LINE_LENGTH];
-    char ini_path[MAX_PATH];
     HANDLE hFile;
-    DWORD res;
-
-    res=GetWindowsDirectoryA( ini_path, sizeof(ini_path) );
-
-    if( res==0 )
-    {
-	WINE_ERR("Couldn't get the windows directory - error %ld\n",
-		GetLastError() );
-
-	return FALSE;
-    }
 
-    if( res>=sizeof(ini_path) )
-    {
-	WINE_ERR("Windows path too long (%ld)\n", res );
 
-	return FALSE;
-    }
-
-    sprintf( ini_path+res, "\\%s", RENAME_FILE );
-
-    hFile=CreateFileA(ini_path, GENERIC_READ,
+    hFile=CreateFileA(RENAME_FILE, GENERIC_READ,
 		    FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL,
 		    NULL );
     
@@ -162,9 +143,9 @@
 
     CloseHandle( hFile );
 
-    if( !DeleteFileA( ini_path ) )
+    if( !MoveFileExA( RENAME_FILE, RENAME_FILE_TO, MOVEFILE_REPLACE_EXISTING) )
     {
-        WINE_ERR("Couldn't erase %s, error %ld\n", ini_path, GetLastError() );
+        WINE_ERR("Couldn't rename wininit.ini, error %ld\n", GetLastError() );
 
         return FALSE;
     }
@@ -174,7 +155,35 @@
 
 int main( int argc, char *argv[] )
 {
-    wininit();
+    /* First, set the current directory to SystemRoot */
+    TCHAR gen_path[MAX_PATH];
+    DWORD res;
+
+    res=GetWindowsDirectory( gen_path, sizeof(gen_path) );
+    
+    if( res==0 )
+    {
+	WINE_ERR("Couldn't get the windows directory - error %ld\n",
+		GetLastError() );
+
+	return 100;
+    }
+
+    if( res>=sizeof(gen_path) )
+    {
+	WINE_ERR("Windows path too long (%ld)\n", res );
+
+	return 100;
+    }
+
+    if( !SetCurrentDirectory( gen_path ) )
+    {
+        WINE_ERR("Cannot set the dir to %s (%ld)\n", gen_path, GetLastError() );
+
+        return 100;
+    }
+
+    res=wininit();
 
-    return 0;
+    return res?0:101;
 }