wine-commits May 2012

wine-commits@winehq.org

2 participants
737 discussions

Henri Verbeet : d3d9: Properly retrieve an interface pointer in d3d9_texture_cube_GetCubeMapSurface ().
by Alexandre Julliard 01 May '12

01 May '12

Module: wine Branch: master Commit: 02cb4fe2946cc6420c313f85b67e5b29f3e401f9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=02cb4fe2946cc6420c313f85b… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Apr 30 21:30:19 2012 +0200 d3d9: Properly retrieve an interface pointer in d3d9_texture_cube_GetCubeMapSurface(). --- dlls/d3d9/texture.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/d3d9/texture.c b/dlls/d3d9/texture.c index bc39ed2..0a1212b 100644 --- a/dlls/d3d9/texture.c +++ b/dlls/d3d9/texture.c @@ -740,6 +740,7 @@ static HRESULT WINAPI d3d9_texture_cube_GetCubeMapSurface(IDirect3DCubeTexture9 { struct d3d9_texture *texture = impl_from_IDirect3DCubeTexture9(iface); struct wined3d_resource *sub_resource; + IDirect3DSurface9Impl *surface_impl; UINT sub_resource_idx; TRACE("iface %p, face %#x, level %u, surface %p.\n", iface, face, level, surface); @@ -752,7 +753,8 @@ static HRESULT WINAPI d3d9_texture_cube_GetCubeMapSurface(IDirect3DCubeTexture9 return D3DERR_INVALIDCALL; } - *surface = wined3d_resource_get_parent(sub_resource); + surface_impl = wined3d_resource_get_parent(sub_resource); + *surface = &surface_impl->IDirect3DSurface9_iface; IDirect3DSurface9_AddRef(*surface); wined3d_mutex_unlock();

1 0

Henri Verbeet : d3d9: Properly retrieve an interface pointer in d3d9_texture_2d_UnlockRect().
by Alexandre Julliard 01 May '12

01 May '12

Module: wine Branch: master Commit: c85f6c9419ca9ac9ea2c3e165c4f2d348c79f014 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c85f6c9419ca9ac9ea2c3e165… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Apr 30 21:30:18 2012 +0200 d3d9: Properly retrieve an interface pointer in d3d9_texture_2d_UnlockRect(). --- dlls/d3d9/texture.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/dlls/d3d9/texture.c b/dlls/d3d9/texture.c index bbe7808..bc39ed2 100644 --- a/dlls/d3d9/texture.c +++ b/dlls/d3d9/texture.c @@ -373,6 +373,7 @@ static HRESULT WINAPI d3d9_texture_2d_UnlockRect(IDirect3DTexture9 *iface, UINT { struct d3d9_texture *texture = impl_from_IDirect3DTexture9(iface); struct wined3d_resource *sub_resource; + IDirect3DSurface9Impl *surface_impl; HRESULT hr; TRACE("iface %p, level %u.\n", iface, level); @@ -381,7 +382,10 @@ static HRESULT WINAPI d3d9_texture_2d_UnlockRect(IDirect3DTexture9 *iface, UINT if (!(sub_resource = wined3d_texture_get_sub_resource(texture->wined3d_texture, level))) hr = D3DERR_INVALIDCALL; else - hr = IDirect3DSurface9_UnlockRect((IDirect3DSurface9 *)wined3d_resource_get_parent(sub_resource)); + { + surface_impl = wined3d_resource_get_parent(sub_resource); + hr = IDirect3DSurface9_UnlockRect(&surface_impl->IDirect3DSurface9_iface); + } wined3d_mutex_unlock(); return hr;

1 0

Henri Verbeet : d3d9: Properly retrieve an interface pointer in d3d9_texture_2d_LockRect().
by Alexandre Julliard 01 May '12

01 May '12

Module: wine Branch: master Commit: 61dd4fa851c72287130eaf9b69f1b9b2bcc05b8e URL: http://source.winehq.org/git/wine.git/?a=commit;h=61dd4fa851c72287130eaf9b6… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Apr 30 21:30:17 2012 +0200 d3d9: Properly retrieve an interface pointer in d3d9_texture_2d_LockRect(). --- dlls/d3d9/texture.c | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/d3d9/texture.c b/dlls/d3d9/texture.c index 6387e2d..bbe7808 100644 --- a/dlls/d3d9/texture.c +++ b/dlls/d3d9/texture.c @@ -350,6 +350,7 @@ static HRESULT WINAPI d3d9_texture_2d_LockRect(IDirect3DTexture9 *iface, { struct d3d9_texture *texture = impl_from_IDirect3DTexture9(iface); struct wined3d_resource *sub_resource; + IDirect3DSurface9Impl *surface_impl; HRESULT hr; TRACE("iface %p, level %u, locked_rect %p, rect %p, flags %#x.\n", @@ -359,8 +360,10 @@ static HRESULT WINAPI d3d9_texture_2d_LockRect(IDirect3DTexture9 *iface, if (!(sub_resource = wined3d_texture_get_sub_resource(texture->wined3d_texture, level))) hr = D3DERR_INVALIDCALL; else - hr = IDirect3DSurface9_LockRect((IDirect3DSurface9 *)wined3d_resource_get_parent(sub_resource), - locked_rect, rect, flags); + { + surface_impl = wined3d_resource_get_parent(sub_resource); + hr = IDirect3DSurface9_LockRect(&surface_impl->IDirect3DSurface9_iface, locked_rect, rect, flags); + } wined3d_mutex_unlock(); return hr;

1 0

Alexandre Julliard : kernel32: Fix another overflow in GetModuleFileNameExW .
by Alexandre Julliard 01 May '12

01 May '12

Module: wine Branch: master Commit: fd11023e4b44b18af7f9d44b216bcc254fe3b299 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fd11023e4b44b18af7f9d44b2… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue May 1 14:43:46 2012 +0200 kernel32: Fix another overflow in GetModuleFileNameExW. --- dlls/kernel32/module.c | 20 +++++++++++--------- 1 files changed, 11 insertions(+), 9 deletions(-) diff --git a/dlls/kernel32/module.c b/dlls/kernel32/module.c index 57a24ba..adf8a99 100644 --- a/dlls/kernel32/module.c +++ b/dlls/kernel32/module.c @@ -1255,18 +1255,20 @@ DWORD WINAPI K32GetModuleFileNameExW(HANDLE process, HMODULE module, return 0; len = ldr_module.FullDllName.Length / sizeof(WCHAR); - if (size <= len) - { - len = size; - size--; - } - if (!ReadProcessMemory(process, ldr_module.FullDllName.Buffer, - file_name, size * sizeof(WCHAR), NULL)) + file_name, min( len, size ) * sizeof(WCHAR), NULL)) return 0; - file_name[size] = 0; - return len; + if (len < size) + { + file_name[len] = 0; + return len; + } + else + { + file_name[size - 1] = 0; + return size; + } } /***********************************************************************

1 0

Alexandre Julliard : gdi32: Support drawing to a visible rectangle smaller than the whole bitmap in the DIB engine .
by Alexandre Julliard 01 May '12

01 May '12

Module: wine Branch: master Commit: 6a19d9f2520ae3805f43a7f2577bafb44699e6f2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6a19d9f2520ae3805f43a7f25… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 27 14:44:53 2012 +0200 gdi32: Support drawing to a visible rectangle smaller than the whole bitmap in the DIB engine. --- dlls/gdi32/dibdrv/bitblt.c | 20 ++- dlls/gdi32/dibdrv/dc.c | 10 +- dlls/gdi32/dibdrv/dibdrv.h | 1 + dlls/gdi32/dibdrv/graphics.c | 42 +++-- dlls/gdi32/dibdrv/objects.c | 17 ++- dlls/gdi32/dibdrv/primitives.c | 329 ++++++++++++++++++++-------------------- 6 files changed, 228 insertions(+), 191 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=6a19d9f2520ae3805f43a…

1 0

Alexandre Julliard : gdi32: Always pass the actual coordinates to the get_pixel_ptr functions.
by Alexandre Julliard 01 May '12

01 May '12

Module: wine Branch: master Commit: 025b58870ee9747d0f8473f0fe777d06f6038a07 URL: http://source.winehq.org/git/wine.git/?a=commit;h=025b58870ee9747d0f8473f0f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 27 14:24:43 2012 +0200 gdi32: Always pass the actual coordinates to the get_pixel_ptr functions. --- dlls/gdi32/dibdrv/primitives.c | 226 ++++++++++++++++++++-------------------- 1 files changed, 113 insertions(+), 113 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=025b58870ee9747d0f847…

1 0

Alexandre Julliard : gdi32: Add support for clipping source coordinates to the DC device rectangle.
by Alexandre Julliard 01 May '12

01 May '12

Module: wine Branch: master Commit: 41e5bc06d8928d6d9b56b8910164683cf66b4cd8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=41e5bc06d8928d6d9b56b8910… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Apr 30 21:17:00 2012 +0200 gdi32: Add support for clipping source coordinates to the DC device rectangle. --- dlls/gdi32/bitblt.c | 14 +++-------- dlls/gdi32/bitmap.c | 1 + dlls/gdi32/clipping.c | 55 +++++++++++++++++++++++++++------------------ dlls/gdi32/dc.c | 1 + dlls/gdi32/dib.c | 1 + dlls/gdi32/gdi_private.h | 2 + 6 files changed, 42 insertions(+), 32 deletions(-) diff --git a/dlls/gdi32/bitblt.c b/dlls/gdi32/bitblt.c index a97231c..6b99e43 100644 --- a/dlls/gdi32/bitblt.c +++ b/dlls/gdi32/bitblt.c @@ -146,13 +146,7 @@ static BOOL get_vis_rectangles( DC *dc_dst, struct bitblt_coords *dst, } get_bounding_rect( &rect, src->x, src->y, src->width, src->height ); - /* source is not clipped */ - if (dc_src->header.type == OBJ_MEMDC) - intersect_rect( &src->visrect, &rect, &dc_src->vis_rect ); - else - src->visrect = rect; /* FIXME: clip to device size */ - - if (is_rect_empty( &src->visrect )) return FALSE; + if (!clip_device_rect( dc_src, &src->visrect, &rect )) return FALSE; if (is_rect_empty( &dst->visrect )) return FALSE; return intersect_vis_rectangles( dst, src ); @@ -932,9 +926,9 @@ BOOL WINAPI GdiAlphaBlend(HDC hdcDst, int xDst, int yDst, int widthDst, int heig if (src.x < 0 || src.y < 0 || src.width < 0 || src.height < 0 || src.log_width < 0 || src.log_height < 0 || - (dcSrc->header.type == OBJ_MEMDC && - (src.width > dcSrc->vis_rect.right - dcSrc->vis_rect.left - src.x || - src.height > dcSrc->vis_rect.bottom - dcSrc->vis_rect.top - src.y))) + (!is_rect_empty( &dcSrc->device_rect ) && + (src.width > dcSrc->device_rect.right - dcSrc->vis_rect.left - src.x || + src.height > dcSrc->device_rect.bottom - dcSrc->vis_rect.top - src.y))) { WARN( "Invalid src coords: (%d,%d), size %dx%d\n", src.x, src.y, src.width, src.height ); SetLastError( ERROR_INVALID_PARAMETER ); diff --git a/dlls/gdi32/bitmap.c b/dlls/gdi32/bitmap.c index cee9204..68e9c9a 100644 --- a/dlls/gdi32/bitmap.c +++ b/dlls/gdi32/bitmap.c @@ -609,6 +609,7 @@ static HGDIOBJ BITMAP_SelectObject( HGDIOBJ handle, HDC hdc ) dc->vis_rect.top = 0; dc->vis_rect.right = bitmap->dib.dsBm.bmWidth; dc->vis_rect.bottom = bitmap->dib.dsBm.bmHeight; + dc->device_rect = dc->vis_rect; GDI_ReleaseObj( handle ); DC_InitDC( dc ); GDI_dec_ref_count( ret ); diff --git a/dlls/gdi32/clipping.c b/dlls/gdi32/clipping.c index c413e1a..1cb420c 100644 --- a/dlls/gdi32/clipping.c +++ b/dlls/gdi32/clipping.c @@ -29,15 +29,12 @@ WINE_DEFAULT_DEBUG_CHANNEL(clipping); -/* return the DC visible rectangle if not empty */ -static inline BOOL get_dc_visrect( DC *dc, RECT *rect ) +/* return the DC device rectangle if not empty */ +static inline BOOL get_dc_device_rect( DC *dc, RECT *rect ) { - if (dc->header.type != OBJ_MEMDC) return FALSE; - rect->left = 0; - rect->top = 0; - rect->right = dc->vis_rect.right - dc->vis_rect.left; - rect->bottom = dc->vis_rect.bottom - dc->vis_rect.top; - return TRUE; + *rect = dc->device_rect; + offset_rect( rect, -dc->vis_rect.left, -dc->vis_rect.top ); + return !is_rect_empty( rect ); } /*********************************************************************** @@ -64,6 +61,20 @@ static inline RECT get_clip_rect( DC * dc, int left, int top, int right, int bot } /*********************************************************************** + * clip_device_rect + * + * Clip a rectangle to the whole DC surface. + */ +BOOL clip_device_rect( DC *dc, RECT *dst, const RECT *src ) +{ + RECT clip; + + if (get_dc_device_rect( dc, &clip )) return intersect_rect( dst, src, &clip ); + *dst = *src; + return TRUE; +} + +/*********************************************************************** * clip_visrect * * Clip a rectangle to the DC visible rect. @@ -72,11 +83,9 @@ BOOL clip_visrect( DC *dc, RECT *dst, const RECT *src ) { RECT clip; - if (get_dc_visrect( dc, &clip )) intersect_rect( dst, src, &clip ); - else *dst = *src; - - if (GetRgnBox( get_dc_region(dc), &clip )) intersect_rect( dst, dst, &clip ); - return !is_rect_empty( dst ); + if (!clip_device_rect( dc, dst, src )) return FALSE; + if (GetRgnBox( get_dc_region(dc), &clip )) return intersect_rect( dst, dst, &clip ); + return TRUE; } /*********************************************************************** @@ -117,7 +126,7 @@ static inline void create_default_clip_region( DC * dc ) { RECT rect; - if (!get_dc_visrect( dc, &rect )) + if (!get_dc_device_rect( dc, &rect )) { rect.left = 0; rect.top = 0; @@ -361,7 +370,7 @@ BOOL WINAPI PtVisible( HDC hdc, INT x, INT y ) pt.y = y; LPtoDP( hdc, &pt, 1 ); update_dc( dc ); - ret = (!get_dc_visrect( dc, &visrect ) || + ret = (!get_dc_device_rect( dc, &visrect ) || (pt.x >= visrect.left && pt.x < visrect.right && pt.y >= visrect.top && pt.y < visrect.bottom)); if (ret && get_dc_region( dc )) ret = PtInRegion( get_dc_region( dc ), pt.x, pt.y ); @@ -385,7 +394,7 @@ BOOL WINAPI RectVisible( HDC hdc, const RECT* rect ) LPtoDP( hdc, (POINT *)&tmpRect, 2 ); update_dc( dc ); - ret = (!get_dc_visrect( dc, &visrect ) || intersect_rect( &visrect, &visrect, &tmpRect )); + ret = (!get_dc_device_rect( dc, &visrect ) || intersect_rect( &visrect, &visrect, &tmpRect )); if (ret && get_dc_region( dc )) ret = RectInRegion( get_dc_region( dc ), &tmpRect ); release_dc_ptr( dc ); return ret; @@ -406,7 +415,7 @@ INT WINAPI GetClipBox( HDC hdc, LPRECT rect ) if (get_dc_region( dc )) { ret = GetRgnBox( get_dc_region( dc ), rect ); - if (get_dc_visrect( dc, &visrect ) && !intersect_rect( rect, rect, &visrect )) + if (get_dc_device_rect( dc, &visrect ) && !intersect_rect( rect, rect, &visrect )) ret = NULLREGION; } else @@ -492,7 +501,6 @@ INT WINAPI GetMetaRgn( HDC hdc, HRGN hRgn ) INT WINAPI GetRandomRgn(HDC hDC, HRGN hRgn, INT iCode) { INT ret = 1; - RECT visrect; DC *dc = get_dc_ptr( hDC ); if (!dc) return -1; @@ -516,13 +524,16 @@ INT WINAPI GetRandomRgn(HDC hDC, HRGN hRgn, INT iCode) case SYSRGN: /* == 4 */ update_dc( dc ); if (dc->hVisRgn) + { CombineRgn( hRgn, dc->hVisRgn, 0, RGN_COPY ); - else if (get_dc_visrect( dc, &visrect )) - SetRectRgn( hRgn, visrect.left, visrect.top, visrect.right, visrect.bottom ); + /* On Windows NT/2000, the SYSRGN returned is in screen coordinates */ + if (!(GetVersion() & 0x80000000)) OffsetRgn( hRgn, dc->vis_rect.left, dc->vis_rect.top ); + } + else if (!is_rect_empty( &dc->device_rect )) + SetRectRgn( hRgn, dc->device_rect.left, dc->device_rect.top, + dc->device_rect.right, dc->device_rect.bottom ); else ret = 0; - /* On Windows NT/2000, the SYSRGN returned is in screen coordinates */ - if (ret && !(GetVersion() & 0x80000000)) OffsetRgn( hRgn, dc->vis_rect.left, dc->vis_rect.top ); break; default: WARN("Unknown code %d\n", iCode); diff --git a/dlls/gdi32/dc.c b/dlls/gdi32/dc.c index 33c1033..95dd3e2 100644 --- a/dlls/gdi32/dc.c +++ b/dlls/gdi32/dc.c @@ -698,6 +698,7 @@ HDC WINAPI CreateCompatibleDC( HDC hdc ) dc->vis_rect.top = 0; dc->vis_rect.right = 1; dc->vis_rect.bottom = 1; + dc->device_rect = dc->vis_rect; ret = dc->hSelf; diff --git a/dlls/gdi32/dib.c b/dlls/gdi32/dib.c index d8d283e..f7e08a5 100644 --- a/dlls/gdi32/dib.c +++ b/dlls/gdi32/dib.c @@ -1624,6 +1624,7 @@ static HGDIOBJ DIB_SelectObject( HGDIOBJ handle, HDC hdc ) dc->vis_rect.top = 0; dc->vis_rect.right = bitmap->dib.dsBm.bmWidth; dc->vis_rect.bottom = bitmap->dib.dsBm.bmHeight; + dc->device_rect = dc->vis_rect; GDI_ReleaseObj( handle ); DC_InitDC( dc ); GDI_dec_ref_count( ret ); diff --git a/dlls/gdi32/gdi_private.h b/dlls/gdi32/gdi_private.h index 20b522d..2589238 100644 --- a/dlls/gdi32/gdi_private.h +++ b/dlls/gdi32/gdi_private.h @@ -104,6 +104,7 @@ typedef struct tagDC SIZE virtual_res; /* Initially HORZRES,VERTRES. Changed by SetVirtualResolution */ SIZE virtual_size; /* Initially HORZSIZE,VERTSIZE. Changed by SetVirtualResolution */ RECT vis_rect; /* visible rectangle in screen coords */ + RECT device_rect; /* rectangle for the whole device */ FLOAT miterLimit; int flags; @@ -217,6 +218,7 @@ extern void free_brush_pattern( struct brush_pattern *pattern ) DECLSPEC_HIDDEN; extern BOOL get_brush_bitmap_info( HBRUSH handle, BITMAPINFO *info, void **bits, UINT *usage ) DECLSPEC_HIDDEN; /* clipping.c */ +extern BOOL clip_device_rect( DC *dc, RECT *dst, const RECT *src ) DECLSPEC_HIDDEN; extern BOOL clip_visrect( DC *dc, RECT *dst, const RECT *src ) DECLSPEC_HIDDEN; extern void update_dc_clipping( DC * dc ) DECLSPEC_HIDDEN;

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits May 2012