From: Frank Binns Date: Wed, 12 Aug 2015 15:35:59 +0000 (+0100) Subject: egl: don't allow eglGetConfigs to set num_configs param to a negative value X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=21b2c6fd5ea5ec2a810945c3c61b14d93a53991d;p=mesa.git egl: don't allow eglGetConfigs to set num_configs param to a negative value When a buffer is provided to eglGetConfigs it's supposed to set the value of the num_config parameter to the total number of configs that have been copied into this buffer. For some reason the EGL spec doesn't consider it to be an error to pass this function a buffer while specifying its size to be less than 0. Given this, one would expect this combination to result in the num_config parameter being set to 0 but this wasn't the case. This was due to the buffer size being copied straight into num_configs without being clamped to 0. This was causing the following dEQP EGL test to fail: dEQP-EGL.functional.query_config.get_configs.get_configs_bounds Signed-off-by: Frank Binns Reviewed-by: Matt Turner Reviewed-by: Emil Velikov --- diff --git a/src/egl/main/eglarray.c b/src/egl/main/eglarray.c index 3ccc8a649f0..d2f39af49a6 100644 --- a/src/egl/main/eglarray.c +++ b/src/egl/main/eglarray.c @@ -197,6 +197,9 @@ _eglFlattenArray(_EGLArray *array, void *buffer, EGLint elem_size, EGLint size, count = array->Size; if (buffer) { + /* clamp size to 0 */ + if (size < 0) + size = 0; /* do not exceed buffer size */ if (count > size) count = size;