Re: [PATCH v3 1/2] oleacc: Implement AccessibleObjectFromEvent.

[Piotr Caban]

Hi Connor,

On 8/30/21 4:10 PM, Connor McAdams wrote:
> - if (SUCCEEDED(hr) && child)
> - {
> -
>    IAccessible_Release(acc);
> -
>    hr = IDispatch_QueryInterface(child, &IID_IAccessible, (void **)&acc);
> -
>    if (FAILED(hr))
> -
>        return hr;

You're leaking child here. It can be released before QueryInterface 
return code check.
> -
> -
>    variant_init_i4(&child_id_variant, CHILDID_SELF);
> -
>    IDispatch_Release(child);

Thanks,
Piotr