We used to have an assert in mark_surface_used to protect us from this but it's gone now. Instead, we'll assert right before we try to memcpy the contents of surf_offset into the binding table. --- src/mesa/drivers/dri/i965/brw_binding_tables.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_binding_tables.c b/src/mesa/drivers/dri/i965/brw_binding_tables.c index 9ca841a..3c795ae 100644 --- a/src/mesa/drivers/dri/i965/brw_binding_tables.c +++ b/src/mesa/drivers/dri/i965/brw_binding_tables.c @@ -121,6 +121,8 @@ brw_upload_binding_table(struct brw_context *brw, &stage_state->bind_bo_offset); /* BRW_NEW_SURFACES and BRW_NEW_*_CONSTBUF */ + assert(prog_data->binding_table.size_bytes <= + sizeof(stage_state->surf_offset)); memcpy(bind, stage_state->surf_offset, prog_data->binding_table.size_bytes); } -- 2.5.0.400.gff86faf _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev