[fireball] 50908: - Don't free an invalid handle. Suggested by Johannes. Test failure remains in the log, so it's easy to spot the function misbehaves. - P.S. Not much of an issue if it's lost duri...

Author: fireball
Date: Sat Feb 26 13:31:58 2011
New Revision: 50908

URL: http://svn.reactos.org/svn/reactos?rev=50908&view=rev
Log:
- Don't free an invalid handle. Suggested by Johannes. Test failure remains in the log, so it's easy to spot the function misbehaves.
- P.S. Not much of an issue if it's lost during the next tests sync, as it would be a clearly visible change.
See issue #5771 for more details.

Modified:
    trunk/rostests/winetests/kernel32/volume.c

Modified: trunk/rostests/winetests/kernel32/volume.c
URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/kernel32/volume....
==============================================================================
--- trunk/rostests/winetests/kernel32/volume.c [iso-8859-1] (original)
+++ trunk/rostests/winetests/kernel32/volume.c [iso-8859-1] Sat Feb 26 13:31:58 2011
@@ -427,7 +427,7 @@
     hFind = pFindFirstVolumeA( Volume_2, MAX_PATH );
     ok(hFind != INVALID_HANDLE_VALUE, "FindFirstVolume failed, err=%u\n",
                 GetLastError());
-
+    if (hFind != INVALID_HANDLE_VALUE) {
     do
     {
         /* validate correct length of unique volume name  */
@@ -440,6 +440,7 @@
     } while (pFindNextVolumeA( hFind, Volume_2, MAX_PATH ));
     ok(found, "volume name %s not found by Find[First/Next]Volume\n", Volume_1);
     pFindVolumeClose( hFind );
+    }
 }

START_TEST(volume)