egl: don't allow eglGetConfigs to set num_configs param to a negative value
authorFrank Binns <frank.binns@imgtec.com>
Wed, 12 Aug 2015 15:35:59 +0000 (16:35 +0100)
committerEmil Velikov <emil.l.velikov@gmail.com>
Thu, 13 Aug 2015 16:36:06 +0000 (17:36 +0100)
commit21b2c6fd5ea5ec2a810945c3c61b14d93a53991d
tree1ebb7ce1a1a8410d7cd29e0b3fbf2c5db7370c66
parent9a4eae61c24858d69d731d63b141d2acaed40d69
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 <frank.binns@imgtec.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
src/egl/main/eglarray.c