Re: [PATCH 0/6] MR2303: uiautomationcore: Implement a few COM UI Automation condition interfaces.

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/uiautomationcore/uia_com_client.c:
> - uia_not_condition_AddRef,
> - uia_not_condition_Release,
> - uia_not_condition_GetChild,
>
> +};
> -
>
> +static HRESULT create_uia_not_condition_iface(IUIAutomationCondition **out_cond, IUIAutomationCondition *in_cond)
> +{
> - struct uia_not_condition *uia_not_condition;
> -
> - uia_not_condition = heap_alloc_zero(sizeof(*uia_not_condition));
> - if (!uia_not_condition)
> -
>    return E_OUTOFMEMORY;
> -
> - uia_not_condition->IUIAutomationNotCondition_iface.lpVtbl = &uia_not_condition_vtbl;
> - uia_not_condition->condition.ConditionType = ConditionType_Not;
> - uia_not_condition->condition.pConditions = get_uia_condition_struct_from_iface(in_cond);

Should in_cond or the resulting UiaCondition* be checked for NULL?