i965: Validate requested GLES context version in brwCreateContext

For GLES1 and GLES2, brwCreateContext neglected to validate the requested
context version received from the DRI layer. If DRI requested an OpenGL
ES2 context with version 3.9, we provided it one.

Before this fix, the switch statement that validated the requested GL
context flavor was an ugly #ifdef copy-paste mess. Instead of reproducing
the copy-past-mess for GLES1 and GLES2, I first refactored it.  Now the
switch statement is readable.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Signed-off-by: Chad Versace <chad.versace@linux.intel.com>

diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c
index 144896534a1e313e3ab27059915a46ff77253c29..f439cf20abce2fb45e2492d74c5dc1efb0659d80 100644 (file)
--- a/src/mesa/drivers/dri/i965/brw_context.c
+++ b/src/mesa/drivers/dri/i965/brw_context.c
@@ -84,53 +84,45 @@ brwCreateContext(int api,
    __DRIscreen *sPriv = driContextPriv->driScreenPriv;
    struct intel_screen *screen = sPriv->driverPrivate;
    struct dd_function_table functions;
+   const unsigned req_version = major_version * 10 + minor_version;
+   unsigned max_supported_version = 0;
    unsigned i;
 
-   /* Filter against the requested API and version.
-    */
-   switch (api) {
-   case API_OPENGL: {
 #ifdef TEXTURE_FLOAT_ENABLED
-      const unsigned max_version =
-         (screen->gen == 6 ||
-          (screen->gen == 7 && screen->kernel_has_gen7_sol_reset))
-         ? 30 : 21;
+   bool has_texture_float = true;
 #else
-      const unsigned max_version = 21;
+   bool has_texture_float = false;
 #endif
-      const unsigned req_version = major_version * 10 + minor_version;
 
-      if (req_version > max_version) {
-         *error = __DRI_CTX_ERROR_BAD_VERSION;
-         return false;
-      }
+   bool supports_gl30 = has_texture_float &&
+                        (screen->gen == 6 ||
+                         (screen->gen == 7 &&
+                          screen->kernel_has_gen7_sol_reset));
+
+   /* Determine max_supported_version. */
+   switch (api) {
+   case API_OPENGL:
+      max_supported_version = supports_gl30 ? 30 : 21;
       break;
-   }
    case API_OPENGLES:
+      max_supported_version = 11;
+      break;
    case API_OPENGLES2:
+      max_supported_version = 20;
       break;
-   case API_OPENGL_CORE: {
-#ifdef TEXTURE_FLOAT_ENABLED
-      const unsigned max_version =
-         (screen->gen == 6 ||
-          (screen->gen == 7 && screen->kernel_has_gen7_sol_reset))
-         ? 31 : 0;
-      const unsigned req_version = major_version * 10 + minor_version;
-
-      if (req_version > max_version) {
-         *error = (max_version == 0)
-            ? __DRI_CTX_ERROR_BAD_API : __DRI_CTX_ERROR_BAD_VERSION;
-         return false;
-      }
+   case API_OPENGL_CORE:
+      max_supported_version = supports_gl30 ? 31 : 0;
       break;
-#else
-      *error = __DRI_CTX_ERROR_BAD_API;
-      return false;
-#endif
-   }
    default:
+      break;
+   }
+
+   if (max_supported_version == 0) {
       *error = __DRI_CTX_ERROR_BAD_API;
       return false;
+   } else if (req_version > max_supported_version) {
+      *error = __DRI_CTX_ERROR_BAD_VERSION;
+      return false;
    }
 
    struct brw_context *brw = rzalloc(NULL, struct brw_context);