translate: do not clamp element index in generic_run
authorCharmaine Lee <charmainel@vmware.com>
Wed, 12 Apr 2023 00:24:40 +0000 (03:24 +0300)
committerMarge Bot <emma+marge@anholt.net>
Wed, 19 Apr 2023 20:53:27 +0000 (20:53 +0000)
commit13e885842a4b962cd02f57e140b348daf551dff3
tree02c3b2ec79d0f511051b4af9abdf3365d0deda19
parent5075e9f7c744dc092bf5e9ef8ee8f3a8704c1ff0
translate: do not clamp element index in generic_run

The buffer max_index value in translate_generic struct is relevant for
indexed draw only. So do not clamp the element index in generic_run() as it
is called for non-indexed draw only.
This patch passes index_size to the common generic_run_one function
so index clamping is only performed when a non-zero index_size is specified.

This fixes a text selection bug with kitty terminal emulator running on ARM
when it falls back to the generic translate path for unsigned byte vertex
array.

cc: mesa-stable

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22568>
src/gallium/auxiliary/translate/translate_generic.c