[PATCH v2 3/5] d2d1: Implement property parsing for RegisterEffectFromStream().
Nikolay Sivov
nsivov at codeweavers.com
Sun Jun 12 03:34:03 CDT 2022
On 6/12/22 08:31, Ziqing Hui wrote:
> + if (!wcscmp(name, L"DisplayName")
> + || !wcscmp(name, L"Author")
> + || !wcscmp(name, L"Category")
> + || !wcscmp(name, L"Description"))
> + {
> + if (type != D2D1_PROPERTY_TYPE_STRING)
> + return E_INVALIDARG;
> + ++reg->system_property_count;
> + }
That's not going to work for nested properties, that likely can have
same names as system ones.
> + if (j > entry->property_count)
> + {
> + hr = D2DERR_INVALID_PROPERTY;
> + goto done;
> + }
Should it be j == entry->property_count ?
More information about the wine-devel
mailing list