Re: [Mesa-dev] [PATCH] i830: Fix stack corruption

2012-07-20 Thread Ian Romanick
On 07/20/2012 04:18 PM, Matt Turner wrote: On Fri, Jul 20, 2012 at 3:45 PM, Chad Versace wrote: Found by compiler warning: i830_texstate.c:131:28: warning: argument to 'sizeof' in 'memset' call is the same expression as the destination; did you mean to dereference it?

Re: [Mesa-dev] [PATCH] i830: Fix stack corruption

2012-07-20 Thread Chad Versace
On 07/20/2012 04:18 PM, Matt Turner wrote: > On Fri, Jul 20, 2012 at 3:45 PM, Chad Versace > wrote: >> Found by compiler warning: >> i830_texstate.c:131:28: warning: argument to 'sizeof' in 'memset' call >> is the same expression as the destination; did you mean to >> deref

Re: [Mesa-dev] [PATCH] i830: Fix stack corruption

2012-07-20 Thread Matt Turner
On Fri, Jul 20, 2012 at 3:45 PM, Chad Versace wrote: > Found by compiler warning: > i830_texstate.c:131:28: warning: argument to 'sizeof' in 'memset' call > is the same expression as the destination; did you mean to > dereference it? [-Wsizeof-pointer-memaccess] >m

Re: [Mesa-dev] [PATCH] i830: Fix stack corruption

2012-07-20 Thread Chad Versace
On 07/20/2012 03:53 PM, Brian Paul wrote: > On 07/20/2012 04:45 PM, Chad Versace wrote: >> Found by compiler warning: >> i830_texstate.c:131:28: warning: argument to 'sizeof' in 'memset' call >>is the same expression as the destination; did you mean to >>dereference it?

Re: [Mesa-dev] [PATCH] i830: Fix stack corruption

2012-07-20 Thread Brian Paul
On 07/20/2012 04:45 PM, Chad Versace wrote: Found by compiler warning: i830_texstate.c:131:28: warning: argument to 'sizeof' in 'memset' call is the same expression as the destination; did you mean to dereference it? [-Wsizeof-pointer-memaccess] memset(state, 0

[Mesa-dev] [PATCH] i830: Fix stack corruption

2012-07-20 Thread Chad Versace
Found by compiler warning: i830_texstate.c:131:28: warning: argument to 'sizeof' in 'memset' call is the same expression as the destination; did you mean to dereference it? [-Wsizeof-pointer-memaccess] memset(state, 0, sizeof(state)); ~^