fix: load and unload command were returning non-zero status code on success

Author: ClementDreptin

.clang-format

@@ -1,5 +1,5 @@
 ---
-Language: Cpp
+Language: C
 BasedOnStyle: Microsoft
 AccessModifierOffset: -4
 AlignAfterOpenBracket: BlockIndent

src/Modules.c

@@ -208,7 +208,7 @@ HRESULT Load(const char *modulePath)
 
     LogSuccess("%s has been loaded.", modulePath);
 
-    return hr;
+    return S_OK;
 }
 
 HRESULT Unload(const char *modulePath)
@@ -266,7 +266,7 @@ HRESULT Unload(const char *modulePath)
 
     LogSuccess("%s has been unloaded.", modulePath);
 
-    return hr;
+    return S_OK;
 }
 
 HRESULT UnloadThenLoad(const char *modulePath)
@@ -290,5 +290,5 @@ HRESULT UnloadThenLoad(const char *modulePath)
     if (FAILED(hr))
         return E_FAIL;
 
-    return hr;
+    return S_OK;
 }

src/XDRPC.c

@@ -94,12 +94,10 @@ HRESULT XdrpcCall(const char *moduleName, uint32_t ordinal, XdrpcArgInfo *args,
 
     // Increase the size of the buffer to allow all arguments to fit
     for (i = 0; i < numberOfArgs; i++)
-    {
         if (args[i].Type == XdrpcArgType_String)
             bufferSize += SizeOfString(args[i].pData);
         else if (args[i].Type == XdrpcArgType_Integer)
             bufferSize += sizeof(uint64_t);
-    }
 
     // Increase the size of the buffer to fit the module name
     bufferSize += moduleNameSize;
@@ -253,7 +251,7 @@ HRESULT XdrpcCall(const char *moduleName, uint32_t ordinal, XdrpcArgInfo *args,
     // Close the XBDM connection
     DmCloseConnection(connection);
 
-    return hr;
+    return S_OK;
 }
 
 // ----------------------------------------------------------------