diff options
| author | Ilia Mirkin <[email protected]> | 2016-02-12 12:17:18 -0500 |
|---|---|---|
| committer | Ilia Mirkin <[email protected]> | 2016-02-15 12:20:25 -0500 |
| commit | 4360ba0cafa53506297235637e13acc5f0879531 (patch) | |
| tree | a8c44b8bfd6bb7c91d5ea703c82c086a8d2e14e5 /src/mesa | |
| parent | 66c790720bc8720402e9739a132b8fb7d65d210f (diff) | |
mesa: need to check resource and set length even if bufSize is 0
This fixes a number of dEQP tests, such as:
dEQP-GLES31.functional.program_interface_query.buffer_limited_query.resource_query
It was expecting the length to be set even in the bufSize == 0 case.
Also _mesa_get_program_resourceiv does some error checking on the
resource which should probably happen even in the bufSize == 0 case as
well although there's no dEQP test for that.
Signed-off-by: Ilia Mirkin <[email protected]>
Reviewed-by: Tapani Pälli <[email protected]>
Diffstat (limited to 'src/mesa')
| -rw-r--r-- | src/mesa/main/program_resource.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/mesa/main/program_resource.c b/src/mesa/main/program_resource.c index 50659b0cd21..0d9f8aecf08 100644 --- a/src/mesa/main/program_resource.c +++ b/src/mesa/main/program_resource.c @@ -357,10 +357,6 @@ _mesa_GetProgramResourceiv(GLuint program, GLenum programInterface, return; } - /* No need to write any properties, user requested none. */ - if (bufSize == 0) - return; - _mesa_get_program_resourceiv(shProg, programInterface, index, propCount, props, bufSize, length, params); } |