From 5c631bd2c725856aef5a3229f34d151e013da281 Mon Sep 17 00:00:00 2001
From: Ani <115020168+drawbyperpetual@users.noreply.github.com>
Date: Wed, 19 Jun 2024 17:01:17 +0200
Subject: [PATCH] [Peek]Fixed crash caused by COM object double release in
 runner (#33427)

---
 src/modules/peek/peek/dllmain.cpp | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/src/modules/peek/peek/dllmain.cpp b/src/modules/peek/peek/dllmain.cpp
index d4874dee3f..cef7c1cf9e 100644
--- a/src/modules/peek/peek/dllmain.cpp
+++ b/src/modules/peek/peek/dllmain.cpp
@@ -187,7 +187,7 @@ private:
         }
 
         // Enumerate all Shell Windows to compare the window handle against.
-        IUnknownPtr spEnum{};
+        IUnknownPtr spEnum{}; // _com_ptr_t; no Release required.
         result = spShellWindows->_NewEnum(&spEnum);
         if (result != S_OK || spEnum == nullptr)
         {
@@ -195,7 +195,7 @@ private:
             return true; // Might as well assume it's possible it's an explorer window.
         }
 
-        IEnumVARIANTPtr spEnumVariant{};
+        IEnumVARIANTPtr spEnumVariant{}; // _com_ptr_t; no Release required.
         result = spEnum.QueryInterface(__uuidof(spEnumVariant), &spEnumVariant);
         if (result != S_OK || spEnumVariant == nullptr)
         {
@@ -219,8 +219,6 @@ private:
                     {
                         VariantClear(&variantElement);
                         spWebBrowserApp->Release();
-                        spEnumVariant->Release();
-                        spEnum->Release();
                         return true;
                     }
                 }
@@ -229,8 +227,6 @@ private:
             VariantClear(&variantElement);
         }
 
-        spEnumVariant->Release();
-        spEnum->Release();
         return false;
     }