src/ivmr.c

@@ -1,9 +1,9 @@
 /**
- * @file interface.c
+ * @file ivmr.c
  * @author Vincent Burel, Onyx and Iris (code@onyxandiris.online)
  * @brief Functions for initializing the iVMR interface.
  * Defines a single public function that returns a pointer to the interface.
- * @version 0.11.0
+ * @version 0.10.0
  * @date 2024-07-06
  *
  * @copyright Vincent Burel(c)2015-2021 All Rights Reserved
@@ -14,7 +14,7 @@
  */
 
 #include <windows.h>
-#include "interface.h"
+#include "ivmr.h"
 #include "util.h"
 #include "log.h"
 
@@ -25,8 +25,10 @@
 #define PRAGMA_Pop \
     _Pragma("GCC diagnostic pop")
 
+static T_VBVMR_INTERFACE iVMR;
+
 static long initialize_dll_interfaces(PT_VMR vmr);
-static bool registry_get_voicemeeter_folder(char *dll_fullpath);
+static bool registry_get_voicemeeter_folder(char *szDir);
 
 /**
  * @brief Create an interface object
@@ -36,14 +38,10 @@ static bool registry_get_voicemeeter_folder(char *dll_fullpath);
  */
 PT_VMR create_interface()
 {
-    PT_VMR vmr = malloc(sizeof(T_VBVMR_INTERFACE));
+    PT_VMR vmr = &iVMR;
-    if (vmr == NULL)
+    int rep;
-    {
-        log_error("malloc failed to allocate memory");
-        return NULL;
-    }
 
-    LONG rep = initialize_dll_interfaces(vmr);
+    rep = initialize_dll_interfaces(vmr);
     if (rep < 0)
     {
         if (rep == -100)
@@ -54,8 +52,7 @@ PT_VMR create_interface()
         {
             log_fatal("Error loading Voicemeeter dll with code %d", rep);
         }
-        free(vmr);
+        return NULL;
-        vmr = NULL;
     }
 
     return vmr;
@@ -64,30 +61,26 @@ PT_VMR create_interface()
 /*******************************************************************************/
 /**                                GET DLL INTERFACE                          **/
 /*******************************************************************************/
-#define DLL_FULLPATH_SZ 1024
-#define DLL64_NAME "\\VoicemeeterRemote64.dll"
-#define DLL32_NAME "\\VoicemeeterRemote.dll"
-
 static long initialize_dll_interfaces(PT_VMR vmr)
 {
     HMODULE G_H_Module = NULL;
-    char dll_fullpath[DLL_FULLPATH_SZ];
+    char szDllName[1024];
     memset(vmr, 0, sizeof(T_VBVMR_INTERFACE));
 
     // get Voicemeeter installation directory
-    if (!registry_get_voicemeeter_folder(dll_fullpath))
+    if (!registry_get_voicemeeter_folder(szDllName))
     {
         // Voicemeeter not installed
         return -100;
     }
     // use right dll according to O/S type
     if (sizeof(void *) == 8)
-        strncat(dll_fullpath, DLL64_NAME, DLL_FULLPATH_SZ - strlen(DLL64_NAME) - 1);
+        strcat(szDllName, "\\VoicemeeterRemote64.dll");
     else
-        strncat(dll_fullpath, DLL32_NAME, DLL_FULLPATH_SZ - strlen(DLL32_NAME) - 1);
+        strcat(szDllName, "\\VoicemeeterRemote.dll");
 
     // Load Dll
-    G_H_Module = LoadLibrary(dll_fullpath);
+    G_H_Module = LoadLibrary(szDllName);
     if (G_H_Module == NULL)
         return -101;
 
@@ -132,33 +125,33 @@ static long initialize_dll_interfaces(PT_VMR vmr)
     if (vmr->VBVMR_Logout == NULL)
         return -2;
     if (vmr->VBVMR_RunVoicemeeter == NULL)
-        return -3;
+        return -2;
     if (vmr->VBVMR_GetVoicemeeterType == NULL)
-        return -4;
+        return -3;
     if (vmr->VBVMR_GetVoicemeeterVersion == NULL)
-        return -5;
+        return -4;
     if (vmr->VBVMR_IsParametersDirty == NULL)
-        return -6;
+        return -5;
     if (vmr->VBVMR_GetParameterFloat == NULL)
-        return -7;
+        return -6;
     if (vmr->VBVMR_GetParameterStringA == NULL)
-        return -8;
+        return -7;
     if (vmr->VBVMR_GetParameterStringW == NULL)
-        return -9;
+        return -8;
     if (vmr->VBVMR_GetLevel == NULL)
-        return -10;
+        return -9;
     if (vmr->VBVMR_SetParameterFloat == NULL)
-        return -11;
+        return -10;
     if (vmr->VBVMR_SetParameters == NULL)
-        return -12;
+        return -11;
     if (vmr->VBVMR_SetParametersW == NULL)
-        return -13;
+        return -12;
     if (vmr->VBVMR_SetParameterStringA == NULL)
-        return -14;
+        return -13;
     if (vmr->VBVMR_SetParameterStringW == NULL)
-        return -15;
+        return -14;
     if (vmr->VBVMR_GetMidiMessage == NULL)
-        return -16;
+        return -15;
 
     if (vmr->VBVMR_Output_GetDeviceNumber == NULL)
         return -30;
@@ -187,48 +180,50 @@ static long initialize_dll_interfaces(PT_VMR vmr)
 /**                           GET VOICEMEETER DIRECTORY                       **/
 /*******************************************************************************/
 
-#define INSTALLER_DIR_KEY "SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Uninstall"
 #define INSTALLER_UNINST_KEY "VB:Voicemeeter {17359A74-1236-5467}"
 
 #ifndef KEY_WOW64_32KEY
 #define KEY_WOW64_32KEY 0x0200
 #endif
 
-#define UNINSTALL_KEY_SZ 256
+static bool registry_get_voicemeeter_folder(char *szDir)
-#define UNINSTALL_PATH_SZ 1024
-
-static bool registry_get_voicemeeter_folder(char *dll_fullpath)
 {
+    char szKey[256];
+    char sss[1024];
+    DWORD nnsize = 1024;
+    HKEY hkResult;
+    LONG rep;
+    DWORD pptype = REG_SZ;
+    sss[0] = 0;
+    const char uninstDirKey[] = "SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Uninstall";
+
     // build Voicemeeter uninstallation key
-    char uninstall_key[UNINSTALL_KEY_SZ];
+    strcpy(szKey, uninstDirKey);
-    snprintf(uninstall_key, UNINSTALL_KEY_SZ, "%s\\%s", INSTALLER_DIR_KEY, INSTALLER_UNINST_KEY);
+    strcat(szKey, "\\");
+    strcat(szKey, INSTALLER_UNINST_KEY);
 
     // open key
-    HKEY result;
+    rep = RegOpenKeyEx(HKEY_LOCAL_MACHINE, szKey, 0, KEY_READ, &hkResult);
-    LONG rep = RegOpenKeyEx(HKEY_LOCAL_MACHINE, uninstall_key, 0, KEY_READ, &result);
     if (rep != ERROR_SUCCESS)
     {
         // if not present we consider running in 64bit mode and force to read 32bit registry
-        rep = RegOpenKeyEx(HKEY_LOCAL_MACHINE, uninstall_key, 0, KEY_READ | KEY_WOW64_32KEY, &result);
+        rep = RegOpenKeyEx(HKEY_LOCAL_MACHINE, szKey, 0, KEY_READ | KEY_WOW64_32KEY, &hkResult);
     }
     if (rep != ERROR_SUCCESS)
         return false;
-
     // read uninstall path from registry
-    DWORD pptype = REG_SZ;
+    rep = RegQueryValueEx(hkResult, "UninstallString", 0, &pptype, (unsigned char *)sss, &nnsize);
-    DWORD len_uninstall_path = UNINSTALL_PATH_SZ;
+    RegCloseKey(hkResult);
-    char uninstall_path[UNINSTALL_PATH_SZ] = {0};
-    rep = RegQueryValueEx(result, "UninstallString", 0, &pptype, (unsigned char *)uninstall_path, &len_uninstall_path);
-    RegCloseKey(result);
 
     if (pptype != REG_SZ)
         return false;
     if (rep != ERROR_SUCCESS)
         return false;
-
     // remove name to get the path only
-    remove_last_part_of_path(uninstall_path);
+    remove_last_part_of_path(sss);
-    snprintf(dll_fullpath, DLL_FULLPATH_SZ, uninstall_path);
+    if (nnsize > 512)
+        nnsize = 512;
+    strncpy(szDir, sss, nnsize);
 
     return true;
 }

src/util.c

@@ -2,7 +2,7 @@
  * @file util.c
  * @author Onyx and Iris (code@onyxandiris.online)
  * @brief Utility functions.
- * @version 0.11.0
+ * @version 0.10.0
  * @date 2024-07-06
  *
  * @copyright Copyright (c) 2024

src/vmrcli.c

@@ -2,7 +2,7 @@
  * @file vmrcli.c
  * @author Onyx and Iris (code@onyxandiris.online)
  * @brief A Voicemeeter Remote Command Line Interface
- * @version 0.11.0
+ * @version 0.10.0
  * @date 2024-07-06
  *
  * @copyright Copyright (c) 2024
@@ -13,7 +13,7 @@
 #include <stdlib.h>
 #include <getopt.h>
 #include <windows.h>
-#include "interface.h"
+#include "ivmr.h"
 #include "wrapper.h"
 #include "log.h"
 #include "util.h"
@@ -60,7 +60,6 @@ struct result
 
 static bool vflag = false;
 
-static void terminate(PT_VMR vmr, char *msg);
 static void usage();
 static enum kind set_kind(char *kval);
 static void interactive(PT_VMR vmr, bool with_prompt);
@@ -159,9 +158,10 @@ int main(int argc, char *argv[])
     if (rep != 0)
     {
         if (rep == -2)
-            terminate(vmr, "Timeout logging into the API.");
+            log_fatal("Timeout logging into the API.");
         else
-            terminate(vmr, "Error logging into the Voicemeeter API");
+            log_fatal("Error logging into the Voicemeeter API");
+        exit(EXIT_FAILURE);
     }
 
     if (mflag)
@@ -200,25 +200,14 @@ int main(int argc, char *argv[])
     rep = logout(vmr);
     if (rep != 0)
     {
-        terminate(vmr, "Error logging out of the Voicemeeter API");
+        log_fatal("Error logging out of the Voicemeeter API");
+        return EXIT_FAILURE;
     }
-
+    else
+    {
         log_info("Successfully logged out of the Voicemeeter API");
-    free(vmr);
         return EXIT_SUCCESS;
     }
-
-/**
- * @brief Write fatal error log, free dyn allocated memory then exit
- *
- * @param vmr Pointer to the iVMR interface
- * @param msg Fatal error message
- */
-static void terminate(PT_VMR vmr, char *msg)
-{
-    log_fatal(msg);
-    free(vmr);
-    exit(EXIT_FAILURE);
 }
 
 /**

src/wrapper.c

@@ -2,7 +2,7 @@
  * @file wrapper.c
  * @author Onyx and Iris (code@onyxandiris.online)
  * @brief Provides public functions that wrap the iVMR calls
- * @version 0.11.0
+ * @version 0.10.0
  * @date 2024-07-06
  *
  * @copyright Copyright (c) 2024