Mesa (master): main: Checking for cube completeness in GetCompressedTextureImage.
Laura Ekstrand
ldeks at kemper.freedesktop.org
Mon Mar 9 20:34:45 UTC 2015
Module: Mesa
Branch: master
Commit: ff011340a499d9fe5c3ead6b46b917578bfeb533
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ff011340a499d9fe5c3ead6b46b917578bfeb533
Author: Laura Ekstrand <laura at jlekstrand.net>
Date: Tue Jan 6 14:05:41 2015 -0800
main: Checking for cube completeness in GetCompressedTextureImage.
v2: Review from Anuj Phogat
- Remove redundant copies of the cube map block comment
- Replace redundant "if (!texImage) return;" statements with
assert(texImage)
Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>
---
src/mesa/main/texgetimage.c | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/texgetimage.c b/src/mesa/main/texgetimage.c
index 9411db8..255d365 100644
--- a/src/mesa/main/texgetimage.c
+++ b/src/mesa/main/texgetimage.c
@@ -1303,7 +1303,16 @@ _mesa_GetCompressedTextureImage(GLuint texture, GLint level,
/* Must handle special case GL_TEXTURE_CUBE_MAP. */
if (texObj->Target == GL_TEXTURE_CUBE_MAP) {
- assert(texObj->NumLayers >= 6);
+
+ /* Make sure the texture object is a proper cube.
+ * (See texturesubimage in teximage.c for details on why this check is
+ * performed.)
+ */
+ if (!_mesa_cube_level_complete(texObj, level)) {
+ _mesa_error(ctx, GL_INVALID_OPERATION,
+ "glGetCompressedTextureImage(cube map incomplete)");
+ return;
+ }
/* Copy each face. */
for (i = 0; i < 6; ++i) {
More information about the mesa-commit
mailing list