softpipe: Fix segfault with fbo-cubemap.
authorOlivier Galibert <galibert@pobox.com>
Thu, 19 Jul 2012 16:55:14 +0000 (18:55 +0200)
committerBrian Paul <brianp@vmware.com>
Thu, 19 Jul 2012 19:19:14 +0000 (13:19 -0600)
commitfbe3fa74e5cdaf02f3738da7d6052e1f1fd7bbf2
treec80eb020d0163a873da477fd71779ef1ed61b3ca
parentd7522ed13052a3d30bc4faedce04685263f57933
softpipe: Fix segfault with fbo-cubemap.

The cube sampler generates two-dimensional texture coordinates and
hence passes NULL for the array for the third one.  The actual 2D
sampler, lower in the pipe, knew not to used that array since it
didn't need it.  But the samplers have become single-texel and the
coordinate array dereference has been moved up one step, to a level
where the code does not know only two coordinates are used.  Hence the
segfault.

The simplest fix by far is to add a third dummy coordinate array in
the call to the next pipe step, which will be dereferenced to an
harmless 0 which then will be happily ignored by the sampler.

Fixes https://bugs.freedesktop.org/show_bug.cgi?id=52250

Signed-off-by: Olivier Galibert <galibert@pobox.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
src/gallium/drivers/softpipe/sp_tex_sample.c