kernel32: Implement compatibility mode for VerifyVersionInfo.

Signed-off-by: Gabriel Ivăncescu <gabrielopcode@gmail.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Gabriel Ivăncescu 2020-03-18 18:31:55 +02:00 committed by Alexandre Julliard
parent 563a8c0f98
commit 543209eb23
2 changed files with 136 additions and 12 deletions

View File

@ -193,8 +193,6 @@ static void test_VerifyVersionInfo(void)
DWORD condition3;
DWORD typemask4;
DWORD condition4;
BOOL todo;
} verify_version_tests[] =
{
{
@ -665,13 +663,10 @@ static void test_VerifyVersionInfo(void)
SetLastError(0xdeadbeef);
ret = VerifyVersionInfoA(&info, test->verifymask, mask);
todo_wine_if(test->todo)
{
ok(test->err ? !ret : ret, "%u: unexpected return value %d.\n", i, ret);
if (!ret)
ok(GetLastError() == test->err, "%u: unexpected error code %d, expected %d.\n", i, GetLastError(), test->err);
}
}
/* test handling of version numbers */
/* v3.10 is always less than v4.x even

View File

@ -42,6 +42,61 @@
WINE_DEFAULT_DEBUG_CHANNEL(ver);
static inline UCHAR version_update_condition(UCHAR *last_condition, UCHAR condition)
{
switch (*last_condition)
{
case 0:
*last_condition = condition;
break;
case VER_EQUAL:
if (condition >= VER_EQUAL && condition <= VER_LESS_EQUAL)
{
*last_condition = condition;
return condition;
}
break;
case VER_GREATER:
case VER_GREATER_EQUAL:
if (condition >= VER_EQUAL && condition <= VER_GREATER_EQUAL)
return condition;
break;
case VER_LESS:
case VER_LESS_EQUAL:
if (condition == VER_EQUAL || (condition >= VER_LESS && condition <= VER_LESS_EQUAL))
return condition;
break;
}
if (!condition) *last_condition |= 0x10;
return *last_condition & 0xf;
}
static inline BOOL version_compare_values(ULONG left, ULONG right, UCHAR condition)
{
switch (condition)
{
case VER_EQUAL:
if (left != right) return FALSE;
break;
case VER_GREATER:
if (left <= right) return FALSE;
break;
case VER_GREATER_EQUAL:
if (left < right) return FALSE;
break;
case VER_LESS:
if (left >= right) return FALSE;
break;
case VER_LESS_EQUAL:
if (left > right) return FALSE;
break;
default:
return FALSE;
}
return TRUE;
}
/******************************************************************************
* VerifyVersionInfoA (KERNEL32.@)
*/
@ -68,19 +123,93 @@ BOOL WINAPI VerifyVersionInfoA( LPOSVERSIONINFOEXA lpVersionInfo, DWORD dwTypeMa
/******************************************************************************
* VerifyVersionInfoW (KERNEL32.@)
*/
BOOL WINAPI VerifyVersionInfoW( LPOSVERSIONINFOEXW lpVersionInfo, DWORD dwTypeMask,
BOOL WINAPI VerifyVersionInfoW( LPOSVERSIONINFOEXW info, DWORD dwTypeMask,
DWORDLONG dwlConditionMask)
{
switch(RtlVerifyVersionInfo( lpVersionInfo, dwTypeMask, dwlConditionMask ))
OSVERSIONINFOEXW ver;
TRACE("(%p 0x%x 0x%s)\n", info, dwTypeMask, wine_dbgstr_longlong(dwlConditionMask));
ver.dwOSVersionInfoSize = sizeof(ver);
if (!GetVersionExW((OSVERSIONINFOW*)&ver)) return FALSE;
if (!dwTypeMask || !dwlConditionMask)
{
case STATUS_INVALID_PARAMETER:
SetLastError( ERROR_BAD_ARGUMENTS );
return FALSE;
case STATUS_REVISION_MISMATCH:
SetLastError( ERROR_OLD_WIN_VERSION );
SetLastError(ERROR_BAD_ARGUMENTS);
return FALSE;
}
if (dwTypeMask & VER_PRODUCT_TYPE)
{
if (!version_compare_values(ver.wProductType, info->wProductType, dwlConditionMask >> 7*3 & 0x07))
goto mismatch;
}
if (dwTypeMask & VER_SUITENAME)
switch (dwlConditionMask >> 6*3 & 0x07)
{
case VER_AND:
if ((info->wSuiteMask & ver.wSuiteMask) != info->wSuiteMask)
goto mismatch;
break;
case VER_OR:
if (!(info->wSuiteMask & ver.wSuiteMask) && info->wSuiteMask)
goto mismatch;
break;
default:
SetLastError(ERROR_BAD_ARGUMENTS);
return FALSE;
}
if (dwTypeMask & VER_PLATFORMID)
{
if (!version_compare_values(ver.dwPlatformId, info->dwPlatformId, dwlConditionMask >> 3*3 & 0x07))
goto mismatch;
}
if (dwTypeMask & VER_BUILDNUMBER)
{
if (!version_compare_values(ver.dwBuildNumber, info->dwBuildNumber, dwlConditionMask >> 2*3 & 0x07))
goto mismatch;
}
if (dwTypeMask & (VER_MAJORVERSION | VER_MINORVERSION | VER_SERVICEPACKMAJOR | VER_SERVICEPACKMINOR))
{
unsigned char condition, last_condition = 0;
BOOL succeeded = TRUE, do_next_check = TRUE;
if (dwTypeMask & VER_MAJORVERSION)
{
condition = version_update_condition(&last_condition, dwlConditionMask >> 1*3 & 0x07);
succeeded = version_compare_values(ver.dwMajorVersion, info->dwMajorVersion, condition);
do_next_check = (ver.dwMajorVersion == info->dwMajorVersion) &&
((condition >= VER_EQUAL) && (condition <= VER_LESS_EQUAL));
}
if ((dwTypeMask & VER_MINORVERSION) && do_next_check)
{
condition = version_update_condition(&last_condition, dwlConditionMask >> 0*3 & 0x07);
succeeded = version_compare_values(ver.dwMinorVersion, info->dwMinorVersion, condition);
do_next_check = (ver.dwMinorVersion == info->dwMinorVersion) &&
((condition >= VER_EQUAL) && (condition <= VER_LESS_EQUAL));
}
if ((dwTypeMask & VER_SERVICEPACKMAJOR) && do_next_check)
{
condition = version_update_condition(&last_condition, dwlConditionMask >> 5*3 & 0x07);
succeeded = version_compare_values(ver.wServicePackMajor, info->wServicePackMajor, condition);
do_next_check = (ver.wServicePackMajor == info->wServicePackMajor) &&
((condition >= VER_EQUAL) && (condition <= VER_LESS_EQUAL));
}
if ((dwTypeMask & VER_SERVICEPACKMINOR) && do_next_check)
{
condition = version_update_condition(&last_condition, dwlConditionMask >> 4*3 & 0x07);
succeeded = version_compare_values(ver.wServicePackMinor, info->wServicePackMinor, condition);
}
if (!succeeded) goto mismatch;
}
return TRUE;
mismatch:
SetLastError(ERROR_OLD_WIN_VERSION);
return FALSE;
}
/***********************************************************************