Re: winex11.drv: Fix uninitialised values. (valgrind) (try 2)

[Michael Stefaniuc]

Hello Austin,

Austin Lund wrote:
> ------------------------------------------------------------------------
> From 07bc18edc35c68c8e0da58a70ccd5c8f03584a1a Mon Sep 17 00:00:00 2001
> From: Austin Lund <austin.lund(a)gmail.com>
> Date: Tue, 10 Aug 2010 13:03:14 +1000
> Subject: [PATCH] winex11.drv: Fix uninitialised values. (valgrind)
>
> ---
>  dlls/winex11.drv/bitmap.c |    2 +-
>  dlls/winex11.drv/dib.c    |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/dlls/winex11.drv/bitmap.c b/dlls/winex11.drv/bitmap.c
> index 4683fd9..af6c00b 100644
> --- a/dlls/winex11.drv/bitmap.c
> +++ b/dlls/winex11.drv/bitmap.c
> @@ -351,7 +351,7 @@ LONG CDECL X11DRV_SetBitmapBits( HBITMAP hbitmap, const void *bits, LONG count )
>      wine_tsx11_lock();
>      image = XCreateImage( gdi_display, visual, physBitmap->pixmap_depth, ZPixmap, 0, NULL,
>                            bitmap.bmWidth, height, 32, 0 );
> -    if (!(image->data = HeapAlloc( GetProcessHeap(), 0, image->bytes_per_line * height )))
> +    if (!(image->data = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, image->bytes_per_line * height )))
>      {
>          WARN("No memory to create image data.\n");
>          XDestroyImage( image );
> diff --git a/dlls/winex11.drv/dib.c b/dlls/winex11.drv/dib.c
> index 11157f1..eda064e 100644
> --- a/dlls/winex11.drv/dib.c
> +++ b/dlls/winex11.drv/dib.c
> @@ -3535,7 +3535,7 @@ static int X11DRV_DIB_SetImageBits( const X11DRV_DIB_IMAGEBITS_DESCR *descr )
>      else {
>          bmpImage = XCreateImage( gdi_display, visual, descr->depth, ZPixmap, 0, NULL,
>       		 descr->infoWidth, lines, 32, 0 );
> -	bmpImage->data = HeapAlloc( GetProcessHeap(), 0, lines * bmpImage->bytes_per_line );
> +	bmpImage->data = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, lines * bmpImage->bytes_per_line );
>          if(bmpImage->data == NULL) {
>              ERR("Out of memory!\n");
>              XDestroyImage( bmpImage );

both functions are heavily used and you don't want to waste time
HEAP_ZERO_MEMORY just to please valgrind. I even wonder why valgrind
complains as e.g. in X11DRV_SetBitmapBits() only the default switch case
won't initialize the memory at all.

bye
	michael

winex11.drv: Fix uninitialised values. (valgrind) (try 2)

[Austin Lund]

On 10 August 2010 18:05, Michael Stefaniuc <mstefani(a)redhat.com> wrote:
> Hello Austin,
>
> Austin Lund wrote:
> > ------------------------------------------------------------------------
> both functions are heavily used and you don't want to waste time
> HEAP_ZERO_MEMORY just to please valgrind. I even wonder why valgrind
> complains as e.g. in X11DRV_SetBitmapBits() only the default switch case
> won't initialize the memory at all.

Hi Michael,

Thanks for the comment.  I ran winemine in valgrind to get this.
Somehow that must trigger the default case.

winex11.drv: Fix uninitialised values. (valgrind) (try 2)

[Michael Stefaniuc]

Hello Austin,

Austin Lund wrote:
> On 10 August 2010 18:05, Michael Stefaniuc <mstefani(a)redhat.com> wrote:
> > Hello Austin,
> >
> > Austin Lund wrote:
> > > ------------------------------------------------------------------------
> > both functions are heavily used and you don't want to waste time
> > HEAP_ZERO_MEMORY just to please valgrind. I even wonder why valgrind
> > complains as e.g. in X11DRV_SetBitmapBits() only the default switch case
> > won't initialize the memory at all.

> Thanks for the comment.  I ran winemine in valgrind to get this.
> Somehow that must trigger the default case.
> no, you would see a FIXME in that case.
> I wonder if Dan Kegel might know more on why valgrind triggers in that case.

bye
	michael