On Mon, May 1, 2017 at 5:06 PM, Tomasz Figa <tf...@chromium.org> wrote:
> On Sat, Apr 29, 2017 at 1:01 AM, Emil Velikov <emil.l.veli...@gmail.com>
> wrote:
>> From: Emil Velikov <emil.veli...@collabora.com>
>>
>> Annotate the array as static const and use C99 initialiser to populate
>> it.
>>
>> Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
>> ---
>> src/egl/drivers/dri2/egl_dri2.c | 68
>> +++++++++++------------------------------
>> 1 file changed, 18 insertions(+), 50 deletions(-)
>>
>> diff --git a/src/egl/drivers/dri2/egl_dri2.c
>> b/src/egl/drivers/dri2/egl_dri2.c
>> index 2cab7d00c1f..91456b025d5 100644
>> --- a/src/egl/drivers/dri2/egl_dri2.c
>> +++ b/src/egl/drivers/dri2/egl_dri2.c
>> @@ -102,56 +102,24 @@ const __DRIuseInvalidateExtension use_invalidate = {
>> .base = { __DRI_USE_INVALIDATE, 1 }
>> };
>>
>> -EGLint dri2_to_egl_attribute_map[] = {
>> - 0,
>> - EGL_BUFFER_SIZE, /* __DRI_ATTRIB_BUFFER_SIZE */
>> - EGL_LEVEL, /* __DRI_ATTRIB_LEVEL */
>> - EGL_RED_SIZE, /* __DRI_ATTRIB_RED_SIZE */
>> - EGL_GREEN_SIZE, /* __DRI_ATTRIB_GREEN_SIZE */
>> - EGL_BLUE_SIZE, /* __DRI_ATTRIB_BLUE_SIZE */
>> - EGL_LUMINANCE_SIZE, /* __DRI_ATTRIB_LUMINANCE_SIZE */
>> - EGL_ALPHA_SIZE, /* __DRI_ATTRIB_ALPHA_SIZE */
>> - 0, /* __DRI_ATTRIB_ALPHA_MASK_SIZE */
>> - EGL_DEPTH_SIZE, /* __DRI_ATTRIB_DEPTH_SIZE */
>> - EGL_STENCIL_SIZE, /* __DRI_ATTRIB_STENCIL_SIZE */
>> - 0, /* __DRI_ATTRIB_ACCUM_RED_SIZE */
>> - 0, /* __DRI_ATTRIB_ACCUM_GREEN_SIZE */
>> - 0, /* __DRI_ATTRIB_ACCUM_BLUE_SIZE */
>> - 0, /* __DRI_ATTRIB_ACCUM_ALPHA_SIZE */
>> - EGL_SAMPLE_BUFFERS, /* __DRI_ATTRIB_SAMPLE_BUFFERS */
>> - EGL_SAMPLES, /* __DRI_ATTRIB_SAMPLES */
>> - 0, /* __DRI_ATTRIB_RENDER_TYPE, */
>> - 0, /* __DRI_ATTRIB_CONFIG_CAVEAT */
>> - 0, /* __DRI_ATTRIB_CONFORMANT */
>> - 0, /* __DRI_ATTRIB_DOUBLE_BUFFER */
>> - 0, /* __DRI_ATTRIB_STEREO */
>> - 0, /* __DRI_ATTRIB_AUX_BUFFERS */
>> - 0, /* __DRI_ATTRIB_TRANSPARENT_TYPE */
>> - 0, /*
>> __DRI_ATTRIB_TRANSPARENT_INDEX_VALUE */
>> - 0, /* __DRI_ATTRIB_TRANSPARENT_RED_VALUE
>> */
>> - 0, /*
>> __DRI_ATTRIB_TRANSPARENT_GREEN_VALUE */
>> - 0, /* __DRI_ATTRIB_TRANSPARENT_BLUE_VALUE
>> */
>> - 0, /*
>> __DRI_ATTRIB_TRANSPARENT_ALPHA_VALUE */
>> - 0, /* __DRI_ATTRIB_FLOAT_MODE
>> (deprecated) */
>> - 0, /* __DRI_ATTRIB_RED_MASK */
>> - 0, /* __DRI_ATTRIB_GREEN_MASK */
>> - 0, /* __DRI_ATTRIB_BLUE_MASK */
>> - 0, /* __DRI_ATTRIB_ALPHA_MASK */
>> - EGL_MAX_PBUFFER_WIDTH, /* __DRI_ATTRIB_MAX_PBUFFER_WIDTH */
>> - EGL_MAX_PBUFFER_HEIGHT, /* __DRI_ATTRIB_MAX_PBUFFER_HEIGHT */
>> - EGL_MAX_PBUFFER_PIXELS, /* __DRI_ATTRIB_MAX_PBUFFER_PIXELS */
>> - 0, /* __DRI_ATTRIB_OPTIMAL_PBUFFER_WIDTH
>> */
>> - 0, /* __DRI_ATTRIB_OPTIMAL_PBUFFER_HEIGHT
>> */
>> - 0, /* __DRI_ATTRIB_VISUAL_SELECT_GROUP */
>> - 0, /* __DRI_ATTRIB_SWAP_METHOD */
>> - EGL_MAX_SWAP_INTERVAL, /* __DRI_ATTRIB_MAX_SWAP_INTERVAL */
>> - EGL_MIN_SWAP_INTERVAL, /* __DRI_ATTRIB_MIN_SWAP_INTERVAL */
>> - 0, /* __DRI_ATTRIB_BIND_TO_TEXTURE_RGB */
>> - 0, /* __DRI_ATTRIB_BIND_TO_TEXTURE_RGBA */
>> - 0, /* __DRI_ATTRIB_BIND_TO_MIPMAP_TEXTURE
>> */
>> - 0, /*
>> __DRI_ATTRIB_BIND_TO_TEXTURE_TARGETS */
>> - EGL_Y_INVERTED_NOK, /* __DRI_ATTRIB_YINVERTED */
>> - 0, /*
>> __DRI_ATTRIB_FRAMEBUFFER_SRGB_CAPABLE */
>> +static const EGLint dri2_to_egl_attribute_map[] = {
>
> Don't we need to specify the size explicitly now (or alternatively
> specify the value of last attribute explicitly, even if it's 0)?
> Looking at the code that reads from the array, it uses the key
> directly as the index, so for example reading
> __DRI_ATTRIB_FRAMEBUFFER_SRGB_CAPABLE will now trigger an out of
> bounds read.
Alternatively, the code that accesses the array could have array size
check added.
Best regards,
Tomasz
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
