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.c?rev=50908&r1=50907&r2=50908&view=diff
==============================================================================
--- 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)
