Re: [Mesa-dev] [PATCH] gallium/tgsi: fix oob access in parse instruction

Hello, Ping! 2017-01-23 15:44 GMT+08:00 Li Qiang : > When parsing texture instruction, it doesn't stop if the > 'cur' is ',', the loop variable 'i' will also be increased > and be used to index the 'inst.TexOffsets' array. This can lead > an oob access issue. This patch avoid this. > > Signed-of

[Mesa-dev] [PATCH] gallium/tgsi: fix oob access in parse instruction

When parsing texture instruction, it doesn't stop if the 'cur' is ',', the loop variable 'i' will also be increased and be used to index the 'inst.TexOffsets' array. This can lead an oob access issue. This patch avoid this. Signed-off-by: Li Qiang --- src/gallium/auxiliary/tgsi/tgsi_text.c | 2 +

Re: [Mesa-dev] [PATCH] gallium/tgsi: fix oob access in parse instruction

Hi - Original Message - > When parsing texture instruction, it doesn't stop if the > 'cur' is ',', the loop variable 'i' will also be increased > and be used to index the 'inst.TexOffsets' array. This can lead > an oob access issue. This patch avoid this. > > Signed-off-by: Li Qiang > --

Re: [Mesa-dev] [PATCH] gallium/tgsi: fix oob access in parse instruction

2017-01-23 16:17 GMT+08:00 Marc-André Lureau : > Hi > > - Original Message - > > When parsing texture instruction, it doesn't stop if the > > 'cur' is ',', the loop variable 'i' will also be increased > > and be used to index the 'inst.TexOffsets' array. This can lead > > an oob access iss