Brad King wrote: > Brian Paul wrote: >> Brad King wrote: >>> RenderingCxxTests: main/hash.c:133: _mesa_HashLookup: Assertion >>> `table' failed. >>> >>> Program received signal SIGABRT, Aborted. >>> [Switching to Thread 0x2adcc9934850 (LWP 18944)] >>> 0x00002adcc82f71d5 in raise () from /lib/libc.so.6 >>> (gdb) where >>> #0 0x00002adcc82f71d5 in raise () from /lib/libc.so.6 >>> #1 0x00002adcc82f8680 in abort () from /lib/libc.so.6 >>> #2 0x00002adcc82f075f in __assert_fail () from /lib/libc.so.6 >>> #3 0x00002adcc6492fe2 in _mesa_HashLookup (table=0x0, key=3) >>> at main/hash.c:133 >>> #4 0x00002adcc64e5842 in _mesa_lookup_texture (ctx=0x695ec0, id=3) >>> at main/texobj.c:58 >>> #5 0x00002adcc64e7133 in _mesa_DeleteTextures (n=1, >>> textures=0x7fffe97ca7a8) >>> at main/texobj.c:803 >>> #6 0x00002adcc1855c2f in (VTK Code) >> I don't see how this could be happening. The pointer to the texture >> object hash table is NULL. The table is freed at context destructions >> time, but even then, the pointer is not set to null. >> >> Maybe try with valgrind? > > I'm running it now, but it will take a few minutes. > > Meanwhile, I was just looking at 1b7e9098, the commit to which this > assertion failure bisects. There are some suspicious lines: > > +#ifdef FEATURE_OES_draw_texture > + case GL_TEXTURE_CROP_RECT_OES: > + params[0] = obj->CropRect[0]; > + params[0] = obj->CropRect[1]; > + params[0] = obj->CropRect[2]; > + params[0] = obj->CropRect[3]; > + break; > +#endif > ... > +#ifdef FEATURE_OES_draw_texture > + case GL_TEXTURE_CROP_RECT_OES: > + params[0] = obj->CropRect[0]; > + params[0] = obj->CropRect[1]; > + params[0] = obj->CropRect[2]; > + params[0] = obj->CropRect[3]; > + break; > +#endif > > Did you intend the patch below? It doesn't prevent the assertion > failure, but I thought I would mention it. > > -Brad > > diff --git a/src/mesa/main/texparam.c b/src/mesa/main/texparam.c > index 664adad..acddb66 100644 > --- a/src/mesa/main/texparam.c > +++ b/src/mesa/main/texparam.c > @@ -903,9 +903,9 @@ _mesa_GetTexParameterfv( GLenum target, GLenum > pname, GLfloat *params ) > #ifdef FEATURE_OES_draw_texture > case GL_TEXTURE_CROP_RECT_OES: > params[0] = obj->CropRect[0]; > - params[0] = obj->CropRect[1]; > - params[0] = obj->CropRect[2]; > - params[0] = obj->CropRect[3]; > + params[1] = obj->CropRect[1]; > + params[2] = obj->CropRect[2]; > + params[3] = obj->CropRect[3]; > break; > #endif > default: > @@ -1053,9 +1053,9 @@ _mesa_GetTexParameteriv( GLenum target, GLenum > pname, GLint *params ) > #ifdef FEATURE_OES_draw_texture > case GL_TEXTURE_CROP_RECT_OES: > params[0] = obj->CropRect[0]; > - params[0] = obj->CropRect[1]; > - params[0] = obj->CropRect[2]; > - params[0] = obj->CropRect[3]; > + params[1] = obj->CropRect[1]; > + params[2] = obj->CropRect[2]; > + params[3] = obj->CropRect[3]; > break; > #endif > default:
Yes, I'll fix that. Thanks. -Brian ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Mesa3d-dev mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
