[Mesa-stable] [Mesa-dev] [PATCH] gallivm: fix default cbuf info.

Juan A. Suarez Romero jasuarez at igalia.com
Mon May 27 09:39:10 UTC 2019


On Fri, 2019-05-24 at 03:08 +0200, sroland at vmware.com wrote:
> From: Roland Scheidegger <sroland at vmware.com>
> 
> The default null_output really needs to be static, otherwise the values
> we'll eventually get later are doubly random (they are not initialized,
> and even if they were it's a pointer to a local stack variable).
> VMware bug 2349556.


Shouldn't this be CC to @stable ?


> ---
>  src/gallium/auxiliary/gallivm/lp_bld_tgsi_info.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/gallium/auxiliary/gallivm/lp_bld_tgsi_info.c b/src/gallium/auxiliary/gallivm/lp_bld_tgsi_info.c
> index b4e3c2fbc8..9fc9b8c77e 100644
> --- a/src/gallium/auxiliary/gallivm/lp_bld_tgsi_info.c
> +++ b/src/gallium/auxiliary/gallivm/lp_bld_tgsi_info.c
> @@ -608,7 +608,7 @@ finished:
>      */
>  
>     for (index = 0; index < PIPE_MAX_COLOR_BUFS; ++index) {
> -      const struct lp_tgsi_channel_info null_output[4];
> +      static const struct lp_tgsi_channel_info null_output[4];
>        info->cbuf[index] = null_output;
>     }
>  



More information about the mesa-stable mailing list