diff options
| author | Ian Romanick <ian.d.romanick@intel.com> | 2012-07-27 16:07:19 -0700 |
|---|---|---|
| committer | Ian Romanick <ian.d.romanick@intel.com> | 2012-08-29 15:09:35 -0700 |
| commit | fb4f2d34256aee6b0005b18769a395dd066d3820 (patch) | |
| tree | 610f616cbe0f528eed4ebbe280f9d4780fcdf4a1 /src/mesa/main/es1_conversion.c | |
| parent | 8df3f9bd5f7626a4548e9901c0b7af33a21af8a6 (diff) | |
| download | external_mesa3d-fb4f2d34256aee6b0005b18769a395dd066d3820.zip external_mesa3d-fb4f2d34256aee6b0005b18769a395dd066d3820.tar.gz external_mesa3d-fb4f2d34256aee6b0005b18769a395dd066d3820.tar.bz2 | |
mesa/es: Validate glMaterial face and pname in Mesa code rather than the ES wrapper
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Diffstat (limited to 'src/mesa/main/es1_conversion.c')
| -rw-r--r-- | src/mesa/main/es1_conversion.c | 17 |
1 files changed, 5 insertions, 12 deletions
diff --git a/src/mesa/main/es1_conversion.c b/src/mesa/main/es1_conversion.c index c1bcfbe..9b13f13 100644 --- a/src/mesa/main/es1_conversion.c +++ b/src/mesa/main/es1_conversion.c @@ -635,18 +635,13 @@ _es_LoadMatrixx(const GLfixed *m) void GL_APIENTRY _es_Materialx(GLenum face, GLenum pname, GLfixed param) { - switch(face) { - case GL_FRONT_AND_BACK: - break; - default: + if (face != GL_FRONT_AND_BACK) { _mesa_error(_mesa_get_current_context(), GL_INVALID_ENUM, "glMaterialx(face=0x%x)", face); return; } - switch(pname) { - case GL_SHININESS: - break; - default: + + if (pname != GL_SHININESS) { _mesa_error(_mesa_get_current_context(), GL_INVALID_ENUM, "glMaterialx(pname=0x%x)", pname); return; @@ -662,14 +657,12 @@ _es_Materialxv(GLenum face, GLenum pname, const GLfixed *params) unsigned int n_params = 4; GLfloat converted_params[4]; - switch(face) { - case GL_FRONT_AND_BACK: - break; - default: + if (face != GL_FRONT_AND_BACK) { _mesa_error(_mesa_get_current_context(), GL_INVALID_ENUM, "glMaterialxv(face=0x%x)", face); return; } + switch(pname) { case GL_AMBIENT: case GL_DIFFUSE: |