Several bug fixes in save_key().

misc/registry.c

@@ -181,10 +181,12 @@ static void save_key( HKEY hkey, const char *filename )
     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,9 +195,11 @@ static void save_key( HKEY hkey, const char *filename )
         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;
     }
 
+    if (handle != INVALID_HANDLE_VALUE)
+    {
         req->hkey = hkey;
         req->file = handle;
         ret = server_call_noerr( REQ_SAVE_REGISTRY );
@@ -207,6 +211,8 @@ static void save_key( HKEY hkey, const char *filename )
             perror( "rename" );
             unlink( name );
         }
+    }
+    HeapFree( GetProcessHeap(), 0, name );
 }