commit cda8e4410ace2616fdabde7cebaad3bdeacbbf1f
author Jörg Höhle <hoehle@users.sourceforge.net> Tue Dec 16 10:29:35 2008 +0100
committer Alexandre Julliard <julliard@winehq.org> Thu Dec 18 14:53:55 2008 +0100
tree cd519c16b7b72155a04f9c1f910f45f0e94f2283
parent 061302483f5aa8fe7ea219a6cdf93b9ffce0c8fb

winecfg: Prevent crash when clearing volume serial number input.

programs/winecfg/drive.c

diff --git a/programs/winecfg/drive.c b/programs/winecfg/drive.c
index 1a3671c..9e6077c 100644
--- a/programs/winecfg/drive.c
+++ b/programs/winecfg/drive.c
@@ -187,7 +187,7 @@
     HANDLE hFile;
 
     filename[0] = letter;
-    WINE_TRACE("Putting serial number of %08x into file '%s'\n", serial, filename);
+    WINE_TRACE("Putting serial number of %08X into file '%s'\n", serial, filename);
     hFile = CreateFile(filename, GENERIC_WRITE, FILE_SHARE_READ, NULL,
                        CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL);
     if (hFile != INVALID_HANDLE_VALUE)

programs/winecfg/driveui.c

diff --git a/programs/winecfg/driveui.c b/programs/winecfg/driveui.c
index 375eccb..b9bcd2d 100644
--- a/programs/winecfg/driveui.c
+++ b/programs/winecfg/driveui.c
@@ -520,10 +520,11 @@
             char *serial;
 
             serial = get_text(dialog, id);
-            current_drive->serial = strtoul( serial, NULL, 16 );
+            current_drive->serial = serial ? strtoul( serial, NULL, 16 ) : 0;
+            HeapFree(GetProcessHeap(), 0, serial);
             current_drive->modified = TRUE;
 
-            WINE_TRACE("set serial to %08x\n", current_drive->serial);
+            WINE_TRACE("set serial to %08X\n", current_drive->serial);
 
             /* enable the apply button  */
             SendMessage(GetParent(dialog), PSM_CHANGED, (WPARAM) dialog, 0);
@@ -775,7 +776,8 @@
                     current_drive->label = str;
 
                     str = get_textW(dialog, IDC_EDIT_SERIAL);
-                    current_drive->serial = strtoulW( str, NULL, 16 );
+                    current_drive->serial = str ? strtoulW( str, NULL, 16 ) : 0;
+                    HeapFree(GetProcessHeap(), 0, str);
                     current_drive->modified = TRUE;
 
                     /* TODO: we don't have a device at this point */