From: Sven Baars <sven.wine@gmail.com>
Subject: Re: [PATCH] winex11.drv: Add missing biCompression initialization (Valgrind)
Message-Id: <862ba764-0a76-7b56-d4f6-ce611bc0a998@gmail.com>
Date: Tue, 18 Sep 2018 19:12:42 +0200
In-Reply-To: <874lenf5yu.fsf@winehq.org>
References: <20180917210908.16438-1-sven.wine@gmail.com> <874lenf5yu.fsf@winehq.org>

On 18-09-18 12:35, Alexandre Julliard wrote:
> Sven Baars <sven.wine@gmail.com> writes:
>
>> This was previously used uninitialized in bitmapinfoheader_from_user_bitmapinfo
> In theory GetDIBits() should be fixed, but there's no actual problem so
> it's probably not worth the trouble. We definitely don't want to change
> the caller to hide the bug.
>
Ah, I just checked the documentation and found that this is actually a
valid way to call GetDIBits indeed. And I also found out that there is
already a bug report about this specific issue. I'll check more
carefully next time. Sorry for the noise!

Sven