Fixed various queryInterface implementations (#6863)

* Fix: Improper implementation in RendererBase::queryInterface

In the case an arbitrary uuid was passed to RendererBase::QueryInterface it would return SLANG_OK while the outObject is null. This is improper and unexpected from an IUnkown implementation. Additionally, the function did not call addRef() when concerning an IDevice interface.

* Fix: DebugTransientResourceHeap::queryInterface returns wrong interface

When trying to query for the transient heap if the debug layer is enabled, queryInterface would set the outObject to the inner api specific heap (ex: vk::TransientResourceImpl) and NOT the debug heap. This causes a side effect when creating a command buffer that debug wrappers would not be used. The debug version will not be returned, and this snowballs causing an access violation when trying to bind a compute pipeline state. After this fix, debug wrappers for transient heaps, command buffers, encoders, etc... wil be used correctly.

* fix weird whitespace change

1 files changed, 7 insertions, 2 deletions

diff --git a/tools/gfx/renderer-shared.cpp b/tools/gfx/renderer-shared.cpp
index cb25079fb..d68ce4ee8 100644
--- a/tools/gfx/renderer-shared.cpp
+++ b/tools/gfx/renderer-shared.cpp
@@ -405,8 +405,13 @@ SlangResult RendererBase::queryInterface(SlangUUID const& uuid, void** outObject
         return SLANG_OK;
     }
 
-    *outObject = getInterface(uuid);
-    return SLANG_OK;
+    if (IDevice* device_ptr = getInterface(uuid))
+    {
+        *outObject = device_ptr;
+        addRef();
+        return SLANG_OK;
+    }
+    return SLANG_E_NO_INTERFACE;
 }
 
 IDevice* gfx::RendererBase::getInterface(const Guid& guid)