[Mesa-dev] [PATCH 1/3] egl/wayland: Ignore invalid modifiers

[Daniel Stone]

Hi Emil,

On 24 July 2017 at 12:18, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> On 21 July 2017 at 14:34, Daniel Stone <daniels at collabora.com> wrote:
>> @@ -370,6 +370,9 @@ get_back_bo(struct dri2_egl_surface *dri2_surf)
>>        return -1;
>>     }
>>
>> +   if (num_modifiers == 0)
>> +      modifiers = NULL;
>> +
> AFAICT this hunk is not needed - modifiers is accessed only when
> num_modifiers is non-zero.

It's divergent inside Gallium, where resource_create_with_modifiers
will be called rather than resource_create if modifiers != NULL.

Cheers,
Daniel