-
-
Notifications
You must be signed in to change notification settings - Fork 226
kernel32!VerifyVersionInfo and ntdll!RtlVerifyVersionInfo #438
Conversation
AArnott
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this valuable contribution. You've clearly taken great care to follow many of our guidelines. Just a few touch-ups please and I'll be happy to merge this.
|
Thanks for the quick feedback - I hope I've addressed all of them. |
|
Something odd I noticed while working on this PR: In the OS, kernel32.dll depends on ntdll.dll, and not the other way around. In the There are functions in I haven't analyzed to check whether any of these have public documentation for each of the Even without necessarily invoking the (possibly dubious) optimization suggested above, the architectural inversion caught my eye. |
kernel32!VerifyVersionInfo&ntdll!RtlVerifyVersionInfokernel32!VerifyVersionInfo, so usingkernel32directly inDllImportkernel32!VerSetConditionMaskOSVERSIONINFOEX,OS_TYPE,PRODUCT_SUITE,VER_MASK,VER_CONDITION