Re: [PATCH v3 3/3] d2d1: Implement LoadVertexShader().

[Henri Verbeet]

On Fri, 29 Apr 2022 at 12:10, Ziqing Hui <zhui(a)codeweavers.com> wrote:
> static void d2d_effect_context_cleanup(struct d2d_effect_context *effect_context)
>  {
> +    unsigned int i;
> +
> +    for (i = 0; i < effect_context->shader_count; ++i)
> +    {
> +        if (effect_context->shaders[i].shader)
> +            IUnknown_Release(effect_context->shaders[i].shader);
> +    }
> +    heap_free(effect_context->shaders);
> +
>      ID2D1DeviceContext_Release(&effect_context->device_context->ID2D1DeviceContext_iface);
>  }

Can "effect_context->shaders[i].shader" legitimately be NULL above?