- 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.

svn path=/trunk/; revision=50908
This commit is contained in:
Aleksey Bragin 2011-02-26 13:31:58 +00:00
parent c1dc1d5ef0
commit b49f53a6a5

View file

@ -427,7 +427,7 @@ static void test_enum_vols(void)
hFind = pFindFirstVolumeA( Volume_2, MAX_PATH ); hFind = pFindFirstVolumeA( Volume_2, MAX_PATH );
ok(hFind != INVALID_HANDLE_VALUE, "FindFirstVolume failed, err=%u\n", ok(hFind != INVALID_HANDLE_VALUE, "FindFirstVolume failed, err=%u\n",
GetLastError()); GetLastError());
if (hFind != INVALID_HANDLE_VALUE) {
do do
{ {
/* validate correct length of unique volume name */ /* validate correct length of unique volume name */
@ -440,6 +440,7 @@ static void test_enum_vols(void)
} while (pFindNextVolumeA( hFind, Volume_2, MAX_PATH )); } while (pFindNextVolumeA( hFind, Volume_2, MAX_PATH ));
ok(found, "volume name %s not found by Find[First/Next]Volume\n", Volume_1); ok(found, "volume name %s not found by Find[First/Next]Volume\n", Volume_1);
pFindVolumeClose( hFind ); pFindVolumeClose( hFind );
}
} }
START_TEST(volume) START_TEST(volume)