<div dir="ltr"><div><div>Agreed, a seperate patch could be written for that.<br><br></div>Cheers,<br></div>Aaryaman<br><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Apr 24, 2016 at 9:44 PM, Nikolay Sivov <span dir="ltr"><<a href="mailto:bunglehead@gmail.com" target="_blank">bunglehead@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On <a href="tel:24.04.2016%2018" value="+12404201618" target="_blank">24.04.2016 18</a>:50, Aaryaman Vasishta wrote:<br>
> This size of class name turns out to be equal to sizeof("Texture") which is strlen("Texture") + 1. This was recently discovered while testing the size returned by GetClassName.<br>
><br>
> Signed-off-by: Aaryaman Vasishta <<a href="mailto:jem456.vasishta@gmail.com" target="_blank">jem456.vasishta@gmail.com</a>><br>
> ---<br>
>  dlls/d3drm/texture.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/dlls/d3drm/texture.c b/dlls/d3drm/texture.c<br>
> index f81fc03..60374d3 100644<br>
> --- a/dlls/d3drm/texture.c<br>
> +++ b/dlls/d3drm/texture.c<br>
> @@ -796,7 +796,7 @@ static HRESULT WINAPI d3drm_texture3_GetClassName(IDirect3DRMTexture3 *iface, DW<br>
>  {<br>
>      TRACE("iface %p, size %p, name %p.\n", iface, size, name);<br>
><br>
> -    if (!size || *size < strlen("Texture") || !name)<br>
> +    if (!size || *size < sizeof("Texture") || !name)<br>
>          return E_INVALIDARG;<br>
><br>
>      strcpy(name, "Texture");<br>
><br>
<br>
</span>GetClassName() is duplicated for every object, it makes sense to<br>
introduce a helper function eventually.<br>
<br>
<br>
</blockquote></div><br></div></div></div></div>