Mesa (master): tgsi: add assertion to verify legal register file

Brian Paul brianp at kemper.freedesktop.org
Thu Jan 7 18:21:06 UTC 2010


Module: Mesa
Branch: master
Commit: 7335d8006fdc065c8dbe6b63007ffa77ffd53470
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7335d8006fdc065c8dbe6b63007ffa77ffd53470

Author: Brian Paul <brianp at vmware.com>
Date:   Thu Jan  7 11:18:04 2010 -0700

tgsi: add assertion to verify legal register file

This assertion fails with piglit glsl-vs-mov-after-deref test
because we're double freeing the memory.  It seems there's some
confusion between what's placed in the hash table and what isn't.

---

 src/gallium/auxiliary/tgsi/tgsi_sanity.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/src/gallium/auxiliary/tgsi/tgsi_sanity.c b/src/gallium/auxiliary/tgsi/tgsi_sanity.c
index 6aeb9ef..7f1c8e5 100644
--- a/src/gallium/auxiliary/tgsi/tgsi_sanity.c
+++ b/src/gallium/auxiliary/tgsi/tgsi_sanity.c
@@ -515,6 +515,7 @@ regs_hash_destroy(struct cso_hash *hash)
    while (!cso_hash_iter_is_null(iter)) {
       scan_register *reg = (scan_register *)cso_hash_iter_data(iter);
       iter = cso_hash_erase(hash, iter);
+      assert(reg->file < TGSI_FILE_COUNT);
       FREE(reg);
    }
    cso_hash_delete(hash);




More information about the mesa-commit mailing list