Re: patch for dlls/gdi32/dib.c: fixes crash

2011-07-29 Thread Wolfgang Walter
Am Donnerstag, 28. Juli 2011 schrieb Huw Davies: > On Thu, Jul 28, 2011 at 04:14:56PM +0200, Wolfgang Walter wrote: > > I think that > > > > bitmapinfo_from_user_bitmapinfo() > > > > is the real culprit. I think colors gets to big (> 256) and therefore the > > size for the memcpy. > > Hopefully

Re: patch for dlls/gdi32/dib.c: fixes crash

2011-07-28 Thread Huw Davies
On Thu, Jul 28, 2011 at 04:14:56PM +0200, Wolfgang Walter wrote: > I think that > > bitmapinfo_from_user_bitmapinfo() > > is the real culprit. I think colors gets to big (> 256) and therefore the > size > for the memcpy. Hopefully http://source.winehq.org/patches/data/77076 should fix th

Re: patch for dlls/gdi32/dib.c: fixes crash

2011-07-28 Thread Wolfgang Walter
Am Mittwoch, 27. Juli 2011 schrieben Sie: > On Wed, Jul 27, 2011 at 6:44 PM, Wolfgang Walter wrote: > > -    char src_bmibuf[FIELD_OFFSET( BITMAPINFO, bmiColors[256] )]; > > -    BITMAPINFO *src_info = (BITMAPINFO *)src_bmibuf; > > -    char dst_bmibuf[FIELD_OFFSET( BITMAPINFO, bmiColors[256] )];

Re: patch for dlls/gdi32/dib.c: fixes crash

2011-07-27 Thread Octavian Voicu
On Wed, Jul 27, 2011 at 6:44 PM, Wolfgang Walter wrote: > -    char src_bmibuf[FIELD_OFFSET( BITMAPINFO, bmiColors[256] )]; > -    BITMAPINFO *src_info = (BITMAPINFO *)src_bmibuf; > -    char dst_bmibuf[FIELD_OFFSET( BITMAPINFO, bmiColors[256] )]; > -    BITMAPINFO *dst_info = (BITMAPINFO *)dst_bm