Merge pull request #65 from markmaker/fix/double-release

vonnieda · web-flow · commit 982f21c25be6 · 2023-04-14T11:25:52.000-05:00
Fix / Avoid double Release() in FindCaptureDevice()
diff --git a/win/platformcontext.cpp b/win/platformcontext.cpp
@@ -453,7 +453,6 @@ HRESULT FindCaptureDevice(IBaseFilter** ppSrcFilter, const wchar_t* devicePath)
             (FAILED(hr) && strDevicePath == std::to_wstring(num_devices))) {
             VariantClear(&varName);
             hr = pMoniker->BindToObject(0, 0, IID_PPV_ARGS(ppSrcFilter));
-            pMoniker->Release();
             return hr;
         }
         VariantClear(&varName);

Original file line number	Diff line number	Diff line change
`@@ -453,7 +453,6 @@ HRESULT FindCaptureDevice(IBaseFilter** ppSrcFilter, const wchar_t* devicePath)`
`453`	`453`	`(FAILED(hr) && strDevicePath == std::to_wstring(num_devices))) {`
`454`	`454`	`VariantClear(&varName);`
`455`	`455`	`hr = pMoniker->BindToObject(0, 0, IID_PPV_ARGS(ppSrcFilter));`
`456`		`- pMoniker->Release();`
`457`	`456`	`return hr;`
`458`	`457`	`}`
`459`	`458`	`VariantClear(&varName);`