Re: [Mesa-dev] [PATCH] i915: Fix wrong sizeof argument in i915_update_tex_unit.

2012-11-21 Thread Ian Romanick
On 11/20/2012 09:51 PM, Vinson Lee wrote: The bug was found by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org Heh... since sizeof(state) and sizeof(*state) are the same on 32-bit platforms, this only could have cause problems for a small subset of users. This is probably part of

Re: [Mesa-dev] [PATCH] i915: Fix wrong sizeof argument in i915_update_tex_unit.

2012-11-21 Thread Matt Turner
On Wed, Nov 21, 2012 at 12:02 AM, Ian Romanick i...@freedesktop.org wrote: On 11/20/2012 09:51 PM, Vinson Lee wrote: The bug was found by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org Heh... since sizeof(state) and sizeof(*state) are the same on 32-bit platforms, this only

[Mesa-dev] [PATCH] i915: Fix wrong sizeof argument in i915_update_tex_unit.

2012-11-20 Thread Vinson Lee
The bug was found by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/drivers/dri/i915/i915_texstate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i915/i915_texstate.c b/src/mesa/drivers/dri/i915/i915_texstate.c index

[Mesa-dev] [PATCH] i915: Fix wrong sizeof argument in i915_update_tex_unit.

2012-11-13 Thread Vinson Lee
The bug was found by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/drivers/dri/i915/i915_texstate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i915/i915_texstate.c b/src/mesa/drivers/dri/i915/i915_texstate.c index