Commit a547efc0 authored by Rémi Bernon's avatar Rémi Bernon Committed by Alexandre Julliard

ole32: Reset IMoniker data pointer to NULL on failure.

To avoid double free when the caller releases the object.
parent 85fdc7d6
...@@ -161,6 +161,7 @@ static HRESULT get_moniker_comparison_data(IMoniker *pMoniker, MonikerComparison ...@@ -161,6 +161,7 @@ static HRESULT get_moniker_comparison_data(IMoniker *pMoniker, MonikerComparison
{ {
ERR("Failed to copy comparison data into buffer, hr = %#lx\n", hr); ERR("Failed to copy comparison data into buffer, hr = %#lx\n", hr);
HeapFree(GetProcessHeap(), 0, *moniker_data); HeapFree(GetProcessHeap(), 0, *moniker_data);
*moniker_data = NULL;
return hr; return hr;
} }
(*moniker_data)->ulCntData = size; (*moniker_data)->ulCntData = size;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment