diff options
author | Kenneth Graunke <[email protected]> | 2012-06-10 19:06:23 -0700 |
---|---|---|
committer | Ian Romanick <[email protected]> | 2012-08-17 09:14:36 -0700 |
commit | 63396ce4c0fe067f69d1b53d0408627a421c4678 (patch) | |
tree | 027897088205a1ba8992408a31a9d5c0a57f14d9 /src/mesa/program/sampler.cpp | |
parent | f69273f9522a0c1486cf1870e35b3a16516f7233 (diff) |
mesa: Add explicit target checking to Get[Compressed]TexImage().
Previously, they relied on _mesa_max_texture_levels() for texture target
error checking. This was somewhat dodgy, as _mesa_max_texture_levels()
is called in seven diferent places, not all of which necessarily accept
the same list of targets.
I copied the list of legal targets from _mesa_max_texture_levels() but
removed the proxy targets, as both functions explicitly rejected those
targets. This changes the order in which we check errors, which could
change whether we return INVALID_VALUE or INVALID_ENUM. However, it
shouldn't change the list of accepted targets.
Signed-off-by: Kenneth Graunke <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>
Diffstat (limited to 'src/mesa/program/sampler.cpp')
0 files changed, 0 insertions, 0 deletions