inject: Handle Windows API Sets.

This is the proper fix for issue #122 and #172.

Windows 8.x's gdi32.dll was getting LoadLibrary/GetProcAddress from
API-MS-WIN-CORE-LIBRARYLOADER-L1-?-?.DLL instead of kernel32.dll, hence
bypassing our hooks.

1 files changed, 2 insertions, 7 deletions

diff --git a/cli/cli_trace.cpp b/cli/cli_trace.cpp
index a01b389b..e5b0cfdc 100644
--- a/cli/cli_trace.cpp
+++ b/cli/cli_trace.cpp
@@ -137,14 +137,9 @@ traceProgram(trace::API api,
 #if defined(_WIN32)
     /*
      * Use DLL injection method on Windows, even for APIs that don't stricly
-     * need it.  Except when tracing OpenGL on Windows 8, as the injection
-     * method seems to have troubles tracing the internal
-     * gdi32.dll!SwapBuffers -> opengl32.dll!wglSwapBuffer calls, per github
-     * issue #172.
+     * need it.
      */
-    if (api != trace::API_GL || !IsWindows8OrGreater()) {
-        useInject = true;
-    }
+    useInject = true;
 
     if (useInject) {
         args.push_back("inject");