commit 6b34fedc8c8bd3d866c2953fd3aff840686c202f
author Alexandre Julliard <julliard@winehq.org> Thu Nov 25 22:15:11 1999 +0000
committer Alexandre Julliard <julliard@winehq.org> Thu Nov 25 22:15:11 1999 +0000
tree 4e7f7e5debb48e11fc6990623aa1a872b476e115
parent 2c655f5e6e95df9b9dc97797cdf79c31e4d48f2d

Several bug fixes in save_key().

misc/registry.c

diff --git a/misc/registry.c b/misc/registry.c
index cf0b15d..a04246a 100644
--- a/misc/registry.c
+++ b/misc/registry.c
@@ -181,10 +181,12 @@
     int count = 0;
     DWORD ret;
     HANDLE handle;
+    char *p;
+    char *name = HeapAlloc( GetProcessHeap(), 0, strlen(filename) + 20 );
 
-    char *name = xmalloc( strlen(filename) + 10 );
-    char *p = strrchr( name, '/' );
-    if (p) p++;
+    if (!name) return;
+    strcpy( name, filename );
+    if ((p = strrchr( name, '/' ))) p++;
     else p = name;
 
     for (;;)
@@ -193,20 +195,24 @@
         handle = FILE_CreateFile( name, GENERIC_WRITE, 0, NULL,
                                   CREATE_NEW, FILE_ATTRIBUTE_NORMAL, -1 );
         if (handle != INVALID_HANDLE_VALUE) break;
-        if ((ret = GetLastError()) != ERROR_FILE_EXISTS) return;
+        if ((ret = GetLastError()) != ERROR_FILE_EXISTS) break;
     }
 
-    req->hkey = hkey;
-    req->file = handle;
-    ret = server_call_noerr( REQ_SAVE_REGISTRY );
-    CloseHandle( handle );
-    if (ret) unlink( name );
-    else if (rename( name, filename ) == -1)
+    if (handle != INVALID_HANDLE_VALUE)
     {
-        ERR( "Failed to move %s to %s: ", name, filename );
-        perror( "rename" );
-        unlink( name );
+        req->hkey = hkey;
+        req->file = handle;
+        ret = server_call_noerr( REQ_SAVE_REGISTRY );
+        CloseHandle( handle );
+        if (ret) unlink( name );
+        else if (rename( name, filename ) == -1)
+        {
+            ERR( "Failed to move %s to %s: ", name, filename );
+            perror( "rename" );
+            unlink( name );
+        }
     }
+    HeapFree( GetProcessHeap(), 0, name );
 }