<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi Nikolay,</div><div dir="ltr"><br></div><div dir="ltr">Thanks for review!</div><div dir="ltr"><br>On Fri, Feb 15, 2019 at 8:34 AM Nikolay Sivov <<a href="mailto:nsivov@codeweavers.com" target="_blank">nsivov@codeweavers.com</a>> wrote:<br>><br>> > +HRESULT WINAPI PropVariantToBuffer(REFPROPVARIANT propvarIn, void *ret, UINT cb)<br>> > +{<br>> > +    TRACE("(%p, %p, %d)\n", propvarIn, ret, cb);<br>> > +<br>> > +    if(cb > propvarIn->u.caub.cElems)<br>> > +        return E_FAIL;<br>> > +<br>> > +    memcpy(ret, propvarIn->u.caub.pElems, cb);<br>> > +<br>> > +    return S_OK;<br>> > +}<br>><br>> This one should probably check variant type.<br>><br>Well, I've thought about this. From MSDN it said that it only supports  VT_VECTOR | VT_UI1 and VT_ARRRAY | VT_UI1. In order to verify this, we will need to test many other types. So it maybe better to just ignore type checking at this point and adding it in the future if a real application requests it?<br></div><div dir="ltr"><br></div><div dir="ltr"><br>--<br>Regards,<br>Jactry Zeng<br></div></div></div></div>