[PATCH v16 0/1] MR564: cabinet: implement DllGetVersion & GetDllVersion

[Tobias Görgens (＠tobi.goergens)]

Fix stub DllGetVersion implementation to read Dll version
and put it in correct structure.
Implement GetDllVersion by using DllGetVersion function
and return version

--
  v16: cabinet/tests: add test for DllGetVersion

https://gitlab.winehq.org/wine/wine/-/merge_requests/564

[PATCH v16 1/1] cabinet/tests: add test for DllGetVersion expect function to fail in wine as DllGetVersion isn't correctly implemented currently

[Tobias Görgens]

From: Tobias G��rgens

---
 dlls/cabinet/cabinet.h         |  9 +++++
 dlls/cabinet/cabinet.spec      |  2 +-
 dlls/cabinet/cabinet_main.c    |  2 +-
 dlls/cabinet/tests/Makefile.in |  3 +-
 dlls/cabinet/tests/version.c   | 67 ++++++++++++++++++++++++++++++++++
 5 files changed, 80 insertions(+), 3 deletions(-)
 create mode 100644 dlls/cabinet/tests/version.c

diff --git a/dlls/cabinet/cabinet.h b/dlls/cabinet/cabinet.h
index 6193622cb11..707744e16cc 100644
--- a/dlls/cabinet/cabinet.h
+++ b/dlls/cabinet/cabinet.h
@@ -160,6 +160,15 @@ struct QTMstate {
     struct QTMmodelsym m80sym[0x40 + 1], mC0sym[0x40 + 1];
 };

+/*version information used in cabinet.dll*/
+typedef struct _CABINETDLLVERSIONINFO {
+  DWORD cbStruct;
+  DWORD dwReserved1;
+  DWORD dwReserved2;
+  DWORD dwFileVersionMS;
+  DWORD dwFileVersionLS;
+} CABINETDLLVERSIONINFO, *PCABINETDLLVERSIONINFO;
+
 /* LZX stuff */

/* some constants defined by the LZX specification */
diff --git a/dlls/cabinet/cabinet.spec b/dlls/cabinet/cabinet.spec
index 96127cfc429..7cf709de60e 100644
--- a/dlls/cabinet/cabinet.spec
+++ b/dlls/cabinet/cabinet.spec
@@ -1,5 +1,5 @@
 1 stub GetDllVersion
-2 stdcall -private DllGetVersion (ptr)
+2 stdcall -private DllGetVersion (ptr) cabinet_dll_get_version
 3 stdcall Extract(ptr str)
 4 stub DeleteExtractedFiles
 10 cdecl FCICreate(ptr ptr ptr ptr ptr ptr ptr ptr ptr ptr ptr ptr ptr)
diff --git a/dlls/cabinet/cabinet_main.c b/dlls/cabinet/cabinet_main.c
index f95eca93c52..4187cf0140f 100644
--- a/dlls/cabinet/cabinet_main.c
+++ b/dlls/cabinet/cabinet_main.c
@@ -52,7 +52,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(cabinet);
  * NOTES
  *     Supposedly returns version from IE6SP1RP1
  */
-HRESULT WINAPI DllGetVersion (DLLVERSIONINFO *pdvi)
+HRESULT WINAPI cabinet_dll_get_version (DLLVERSIONINFO *pdvi)
 {
   WARN("hmmm... not right version number "5.1.1106.1"?\n");

diff --git a/dlls/cabinet/tests/Makefile.in b/dlls/cabinet/tests/Makefile.in
index f301617473d..fc9fff75c7b 100644
--- a/dlls/cabinet/tests/Makefile.in
+++ b/dlls/cabinet/tests/Makefile.in
@@ -3,4 +3,5 @@ IMPORTS   = cabinet

C_SRCS = \
    extract.c \
-	fdi.c
+	fdi.c \
+	version.c
diff --git a/dlls/cabinet/tests/version.c b/dlls/cabinet/tests/version.c
new file mode 100644
index 00000000000..733ad451d31
--- /dev/null
+++ b/dlls/cabinet/tests/version.c
@@ -0,0 +1,67 @@
+#include <stdio.h>
+#include <string.h>
+#include <windows.h>
+#include <winbase.h>
+#include <wine/test.h>
+
+
+#include "windef.h"
+#define NO_SHLWAPI_REG
+#include "shlwapi.h"
+#undef NO_SHLWAPI_REG
+#include "winbase.h"
+#include "../cabinet.h"
+
+
+typedef VOID (__stdcall *f_dllget)(PCABINETDLLVERSIONINFO);
+
+
+static void test_dllget(HMODULE libHandle)
+{
+    PCABINETDLLVERSIONINFO verInfo;
+    char                   *version;
+    int                    sizeVerInfo;
+    DWORD                  FileVersionMS;
+    DWORD                  FileVersionLS;
+    int                    majorV;
+    int                    minorV;
+    int                    buildV;
+    int                    revisV;
+
+    f_dllget DllGetVersion = (f_dllget)GetProcAddress(libHandle, "DllGetVersion");
+    ok(DllGetVersion != NULL, "Function DllGetVersion in DLL not found: Error = %ld.\n", GetLastError());
+
+
+    verInfo = malloc(sizeof(CABINETDLLVERSIONINFO));
+    ok(verInfo != NULL, "Couldn't allocate memory to run tests properly!\n");
+    if (DllGetVersion)
+        DllGetVersion(verInfo);
+
+    FileVersionMS = verInfo->dwFileVersionMS;
+    FileVersionLS = verInfo->dwFileVersionLS;
+
+    /*length of 4 DWORDs + buffer*/
+    sizeVerInfo = 32;
+
+    version = malloc(sizeVerInfo);
+    ok(version != NULL, "Couldn't allocate memory to run tests properly!\n");
+
+    majorV = (int)( FileVersionMS >> 16 ) & 0xffff;
+    minorV = (int)( FileVersionMS >>  0 ) & 0xffff;
+    buildV = (int)( FileVersionLS >> 16 ) & 0xffff;
+    revisV = (int)( FileVersionLS >>  0 ) & 0xffff;
+
+    snprintf(version, sizeVerInfo, "%d.%d.%d.%d\n",majorV,minorV,buildV,revisV);;
+    todo_wine {
+        /* FIXME - Currently DllGetVersion isn't implemented correctly */
+        ok(strcmp(version,"0.0.0.0\n") != 0, "Cabinet struct doesn't contain correct version: Error = %ld.\n", GetLastError());
+    }
+}
+
+START_TEST(version)
+{
+    HMODULE libHandle;
+    libHandle = LoadLibraryA("Cabinet.dll");
+    ok(libHandle != NULL, "Cabinet.dll not found: Error = %ld.\n", GetLastError());
+    test_dllget(libHandle);
+}