Re: [PATCH xserver] xkb: Initialize pad bytes sent in replies of geometry requests.

2011-03-16 Thread Peter Hutterer
On Wed, Mar 16, 2011 at 10:35:22AM +0200, Rami Ylimäki wrote: > On 03/16/2011 02:43 AM, Peter Hutterer wrote: > > > >>-memcpy(&wire[2],str,len); > >>-wire+= ((2+len+3)/4)*4; > >>+paddedLen= pad_to_int32(sizeof(len)+len)-sizeof(len); > >>+strncpy(&wire[sizeof(len)],str,paddedLen); >

Re: [PATCH xserver] xkb: Initialize pad bytes sent in replies of geometry requests.

2011-03-16 Thread Daniel Stone
Hi, On Wed, Mar 16, 2011 at 10:35:22AM +0200, Rami Ylimäki wrote: > On 03/16/2011 02:43 AM, Peter Hutterer wrote: > > > >>-memcpy(&wire[2],str,len); > >>-wire+= ((2+len+3)/4)*4; > >>+paddedLen= pad_to_int32(sizeof(len)+len)-sizeof(len); > >>+strncpy(&wire[sizeof(len)],str,paddedLen

Re: [PATCH xserver] xkb: Initialize pad bytes sent in replies of geometry requests.

2011-03-16 Thread Rami Ylimäki
On 03/16/2011 02:43 AM, Peter Hutterer wrote: -memcpy(&wire[2],str,len); -wire+= ((2+len+3)/4)*4; +paddedLen= pad_to_int32(sizeof(len)+len)-sizeof(len); +strncpy(&wire[sizeof(len)],str,paddedLen); +wire+= sizeof(len)+paddedLen; looking at this again, does this really fix

Re: [PATCH xserver] xkb: Initialize pad bytes sent in replies of geometry requests.

2011-03-15 Thread Peter Hutterer
On Wed, Mar 09, 2011 at 03:45:40PM +0200, Rami Ylimäki wrote: > Valgrind complains about uninitialized data being written to clients. > > Reviewed-by: Erkki Seppälä > Signed-off-by: Rami Ylimäki > --- > xkb/xkb.c | 12 +--- > 1 files changed, 9 insertions(+), 3 deletions(-) > > diff

Re: [PATCH xserver] xkb: Initialize pad bytes sent in replies of geometry requests.

2011-03-09 Thread Peter Hutterer
On Wed, Mar 09, 2011 at 03:45:40PM +0200, Rami Ylimäki wrote: > Valgrind complains about uninitialized data being written to clients. > > Reviewed-by: Erkki Seppälä > Signed-off-by: Rami Ylimäki merged, thanks. Cheers, Peter > --- > xkb/xkb.c | 12 +--- > 1 files changed, 9 inser

Re: [PATCH xserver] xkb: Initialize pad bytes sent in replies of geometry requests.

2011-03-09 Thread Daniel Stone
On Wed, Mar 09, 2011 at 03:45:40PM +0200, Rami Ylimäki wrote: > Valgrind complains about uninitialized data being written to clients. > > Reviewed-by: Erkki Seppälä > Signed-off-by: Rami Ylimäki Reviewed-by: Daniel Stone signature.asc Description: Digital signature __

[PATCH xserver] xkb: Initialize pad bytes sent in replies of geometry requests.

2011-03-09 Thread Rami Ylimäki
Valgrind complains about uninitialized data being written to clients. Reviewed-by: Erkki Seppälä Signed-off-by: Rami Ylimäki --- xkb/xkb.c | 12 +--- 1 files changed, 9 insertions(+), 3 deletions(-) diff --git a/xkb/xkb.c b/xkb/xkb.c index 8d61e01..39dbab4 100644 --- a/xkb/xkb.c +++