i915: add a missing NULL pointer check
authorLubomir Rintel <lkundrak@v3.sk>
Wed, 28 May 2014 06:56:12 +0000 (08:56 +0200)
committerIan Romanick <ian.d.romanick@intel.com>
Sat, 31 May 2014 00:10:08 +0000 (17:10 -0700)
commit90b5747856dd116e1a7e0fd4a93781400f276c3e
tree83a45b299163b9ce15276644b9033d3e7dd582d0
parent7b1aeec9cd8c9b790d452eaf769ccbce51627908
i915: add a missing NULL pointer check

mesaVisual can be NULL with configless context since this commit:

    commit 551d459af421a2eb937e9e16301bb64da4624f89
    Author: Neil Roberts <neil@linux.intel.com>
    Date:   Fri Mar 7 18:05:47 2014 +0000

    Add the EGL_MESA_configless_context extension
...
    Previously the i965 and i915 drivers were explicitly creating a zeroed visual
    whenever 0 is passed for the EGLConfig.

We attempt to dereference the visual in i915 and now we don't create a
zeroed-out one one it crashes, breaking at least weston in an i915. There's
no point in doing so as it would be zero anyway.

v2: Fixed a typo in commit message.  Added some tags.

Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1100967
Cc: "10.2" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/mesa/drivers/dri/i915/intel_context.c