Re: [PATCH v2 0/1] MR2141: vbscript: Fix memory leak in owned safearray iterator

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/vbscript/interp.c:
> case VT_VARIANT|VT_ARRAY|VT_BYREF: {
>          IEnumVARIANT *iter;
>
> -        hres = create_safearray_iter(V_ISBYREF(v.v) ? *V_ARRAYREF(v.v) : V_ARRAY(v.v), &iter);
> +        hres = create_safearray_iter(V_ISBYREF(v.v) ? *V_ARRAYREF(v.v) : V_ARRAY(v.v), v.owned, &iter);

I think we shouldn't destroy the array if it's a reference. Does changing `v.owned` to `v.owned && !V_ISBYREF(v.v)` work for you?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2141#note_23608