January 2012 - Wine-Commits

Francois Gouget : xcopy: Remove a duplicated space.
by Alexandre Julliard Jan. 23, 2012

Jan. 23, 2012

1 0

Stefan Dösinger : wined3d: Relax depth blit requirements.
by Alexandre Julliard Jan. 23, 2012

Jan. 23, 2012

Module: wine Branch: master Commit: 038e308a4153b84c101659611bc2da261e965cd4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=038e308a4153b84c101659611… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sun Jan 22 16:32:19 2012 +0100 wined3d: Relax depth blit requirements. --- dlls/d3d9/d3d9_private.h | 1 + dlls/d3d9/device.c | 63 ++++++++++++++++++++++++++++++++++++++++------ dlls/wined3d/surface.c | 32 ----------------------- 3 files changed, 56 insertions(+), 40 deletions(-) diff --git a/dlls/d3d9/d3d9_private.h b/dlls/d3d9/d3d9_private.h index 9feda66..5886618 100644 --- a/dlls/d3d9/d3d9_private.h +++ b/dlls/d3d9/d3d9_private.h @@ -173,6 +173,7 @@ typedef struct IDirect3DDevice9Impl unsigned int numConvertedDecls, declArraySize; BOOL notreset; + BOOL in_scene; } IDirect3DDevice9Impl; HRESULT device_init(IDirect3DDevice9Impl *device, IDirect3D9Impl *parent, struct wined3d *wined3d, UINT adapter, D3DDEVTYPE device_type, diff --git a/dlls/d3d9/device.c b/dlls/d3d9/device.c index 631ba5a..39cd740 100644 --- a/dlls/d3d9/device.c +++ b/dlls/d3d9/device.c @@ -1035,22 +1035,67 @@ static HRESULT WINAPI IDirect3DDevice9Impl_GetFrontBufferData(IDirect3DDevice9Ex return hr; } -static HRESULT WINAPI IDirect3DDevice9Impl_StretchRect(IDirect3DDevice9Ex *iface, IDirect3DSurface9 *pSourceSurface, - const RECT *pSourceRect, IDirect3DSurface9 *pDestSurface, const RECT *pDestRect, D3DTEXTUREFILTERTYPE Filter) +static HRESULT WINAPI IDirect3DDevice9Impl_StretchRect(IDirect3DDevice9Ex *iface, IDirect3DSurface9 *src_surface, + const RECT *src_rect, IDirect3DSurface9 *dst_surface, const RECT *dst_rect, D3DTEXTUREFILTERTYPE filter) { - IDirect3DSurface9Impl *src = unsafe_impl_from_IDirect3DSurface9(pSourceSurface); - IDirect3DSurface9Impl *dst = unsafe_impl_from_IDirect3DSurface9(pDestSurface); - HRESULT hr; + IDirect3DDevice9Impl *This = impl_from_IDirect3DDevice9Ex(iface); + IDirect3DSurface9Impl *src = unsafe_impl_from_IDirect3DSurface9(src_surface); + IDirect3DSurface9Impl *dst = unsafe_impl_from_IDirect3DSurface9(dst_surface); + HRESULT hr = D3DERR_INVALIDCALL; + struct wined3d_resource_desc src_desc, dst_desc; + struct wined3d_resource *wined3d_resource; TRACE("iface %p, src_surface %p, src_rect %p, dst_surface %p, dst_rect %p, filter %#x.\n", - iface, pSourceSurface, pSourceRect, pDestSurface, pDestRect, Filter); + iface, src_surface, src_rect, dst_surface, dst_rect, filter); wined3d_mutex_lock(); - hr = wined3d_surface_blt(dst->wined3d_surface, pDestRect, src->wined3d_surface, pSourceRect, 0, NULL, Filter); + wined3d_resource = wined3d_surface_get_resource(dst->wined3d_surface); + wined3d_resource_get_desc(wined3d_resource, &dst_desc); + + wined3d_resource = wined3d_surface_get_resource(src->wined3d_surface); + wined3d_resource_get_desc(wined3d_resource, &src_desc); + + if (src_desc.usage & WINED3DUSAGE_DEPTHSTENCIL) + { + if (This->in_scene) + { + WARN("Rejecting depth / stencil blit while in scene.\n"); + goto done; + } + + if (src_rect) + { + if (src_rect->left || src_rect->top || src_rect->right != src_desc.width + || src_rect->bottom != src_desc.height) + { + WARN("Rejecting depth / stencil blit with invalid source rect %s.\n", + wine_dbgstr_rect(src_rect)); + goto done; + } + } + if (dst_rect) + { + if (dst_rect->left || dst_rect->top || dst_rect->right != dst_desc.width + || dst_rect->bottom != dst_desc.height) + { + WARN("Rejecting depth / stencil blit with invalid destination rect %s.\n", + wine_dbgstr_rect(dst_rect)); + goto done; + } + } + if (src_desc.width != dst_desc.width || src_desc.height != dst_desc.height) + { + WARN("Rejecting depth / stencil blit with mismatched surface sizes.\n"); + goto done; + } + } + + hr = wined3d_surface_blt(dst->wined3d_surface, dst_rect, src->wined3d_surface, src_rect, 0, NULL, filter); if (hr == WINEDDERR_INVALIDRECT) hr = D3DERR_INVALIDCALL; - wined3d_mutex_unlock(); +done: + wined3d_mutex_unlock(); return hr; } @@ -1245,6 +1290,7 @@ static HRESULT WINAPI IDirect3DDevice9Impl_BeginScene(IDirect3DDevice9Ex *iface) wined3d_mutex_lock(); hr = wined3d_device_begin_scene(This->wined3d_device); + if (SUCCEEDED(hr)) This->in_scene = TRUE; wined3d_mutex_unlock(); return hr; @@ -1259,6 +1305,7 @@ static HRESULT WINAPI DECLSPEC_HOTPATCH IDirect3DDevice9Impl_EndScene(IDirect3DD wined3d_mutex_lock(); hr = wined3d_device_end_scene(This->wined3d_device); + if (SUCCEEDED(hr)) This->in_scene = FALSE; wined3d_mutex_unlock(); return hr; diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 023bf6a..e6b4cd8 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -1590,44 +1590,12 @@ HRESULT CDECL wined3d_surface_blt(struct wined3d_surface *dst_surface, const REC } else { - /* Accessing depth / stencil surfaces is supposed to fail while in - * a scene, except for fills, which seem to work. */ - if (device->inScene) - { - WARN("Rejecting depth / stencil access while in scene.\n"); - return WINED3DERR_INVALIDCALL; - } - if (src_ds_flags != dst_ds_flags) { WARN("Rejecting depth / stencil blit between incompatible formats.\n"); return WINED3DERR_INVALIDCALL; } - if (src_rect.top || src_rect.left - || src_rect.bottom != src_surface->resource.height - || src_rect.right != src_surface->resource.width) - { - WARN("Rejecting depth / stencil blit with invalid source rect %s.\n", - wine_dbgstr_rect(&src_rect)); - return WINED3DERR_INVALIDCALL; - } - - if (dst_rect.top || dst_rect.left - || dst_rect.bottom != dst_surface->resource.height - || dst_rect.right != dst_surface->resource.width) - { - WARN("Rejecting depth / stencil blit with invalid destination rect %s.\n", - wine_dbgstr_rect(&src_rect)); - return WINED3DERR_INVALIDCALL; - } - - if (scale) - { - WARN("Rejecting depth / stencil blit with mismatched surface sizes.\n"); - return WINED3DERR_INVALIDCALL; - } - if (SUCCEEDED(wined3d_surface_depth_blt(src_surface, &src_rect, dst_surface, &dst_rect))) return WINED3D_OK; }

1 0

Alexandre Julliard : winex11: Ignore rectangles with invalid coordinates when converting region data.
by Alexandre Julliard Jan. 23, 2012

Jan. 23, 2012

Module: wine Branch: master Commit: fe7d81dc1b1b27ba782676f60dca0aaef014d781 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fe7d81dc1b1b27ba782676f60… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Jan 23 11:32:26 2012 +0100 winex11: Ignore rectangles with invalid coordinates when converting region data. --- dlls/winex11.drv/graphics.c | 33 +++++++++++++++++++++++++-------- 1 files changed, 25 insertions(+), 8 deletions(-) diff --git a/dlls/winex11.drv/graphics.c b/dlls/winex11.drv/graphics.c index 910dfea..afe2ff1 100644 --- a/dlls/winex11.drv/graphics.c +++ b/dlls/winex11.drv/graphics.c @@ -161,13 +161,24 @@ RGNDATA *X11DRV_GetRegionData( HRGN hrgn, HDC hdc_lptodp ) { int j; /* need to start from the end */ + xrect += data->rdh.nCount; for (j = data->rdh.nCount-1; j >= 0; j--) { tmp = rect[j]; - xrect[j].x = max( min( tmp.left, SHRT_MAX), SHRT_MIN); - xrect[j].y = max( min( tmp.top, SHRT_MAX), SHRT_MIN); - xrect[j].width = max( min( tmp.right - xrect[j].x, USHRT_MAX), 0); - xrect[j].height = max( min( tmp.bottom - xrect[j].y, USHRT_MAX), 0); + if (tmp.left > SHRT_MAX) continue; + if (tmp.top > SHRT_MAX) continue; + if (tmp.right < SHRT_MIN) continue; + if (tmp.bottom < SHRT_MIN) continue; + xrect--; + xrect->x = max( min( tmp.left, SHRT_MAX), SHRT_MIN); + xrect->y = max( min( tmp.top, SHRT_MAX), SHRT_MIN); + xrect->width = max( min( tmp.right, SHRT_MAX ) - xrect->x, 0); + xrect->height = max( min( tmp.bottom, SHRT_MAX ) - xrect->y, 0); + } + if (xrect > (XRectangle *)data->Buffer) + { + data->rdh.nCount -= xrect - (XRectangle *)data->Buffer; + memmove( (XRectangle *)data->Buffer, xrect, data->rdh.nCount * sizeof(*xrect) ); } } else @@ -175,11 +186,17 @@ RGNDATA *X11DRV_GetRegionData( HRGN hrgn, HDC hdc_lptodp ) for (i = 0; i < data->rdh.nCount; i++) { tmp = rect[i]; - xrect[i].x = max( min( tmp.left, SHRT_MAX), SHRT_MIN); - xrect[i].y = max( min( tmp.top, SHRT_MAX), SHRT_MIN); - xrect[i].width = max( min( tmp.right - xrect[i].x, USHRT_MAX), 0); - xrect[i].height = max( min( tmp.bottom - xrect[i].y, USHRT_MAX), 0); + if (tmp.left > SHRT_MAX) continue; + if (tmp.top > SHRT_MAX) continue; + if (tmp.right < SHRT_MIN) continue; + if (tmp.bottom < SHRT_MIN) continue; + xrect->x = max( min( tmp.left, SHRT_MAX), SHRT_MIN); + xrect->y = max( min( tmp.top, SHRT_MAX), SHRT_MIN); + xrect->width = max( min( tmp.right, SHRT_MAX ) - xrect->x, 0); + xrect->height = max( min( tmp.bottom, SHRT_MAX ) - xrect->y, 0); + xrect++; } + data->rdh.nCount = xrect - (XRectangle *)data->Buffer; } return data; }

1 0

Francois Gouget : ddraw/tests: Fix compilation on systems that don' t support nameless unions.
by Alexandre Julliard Jan. 23, 2012

Jan. 23, 2012

Module: wine Branch: master Commit: 21036e13dff6f66f601bc3a219f9152bca2e9e0d URL: http://source.winehq.org/git/wine.git/?a=commit;h=21036e13dff6f66f601bc3a21… Author: Francois Gouget <fgouget(a)free.fr> Date: Sun Jan 22 19:26:47 2012 +0100 ddraw/tests: Fix compilation on systems that don't support nameless unions. --- dlls/ddraw/tests/ddraw1.c | 4 ++-- dlls/ddraw/tests/ddraw2.c | 4 ++-- dlls/ddraw/tests/ddraw4.c | 2 +- dlls/ddraw/tests/ddraw7.c | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/ddraw/tests/ddraw1.c b/dlls/ddraw/tests/ddraw1.c index 7981c84..c580e4b 100644 --- a/dlls/ddraw/tests/ddraw1.c +++ b/dlls/ddraw/tests/ddraw1.c @@ -137,7 +137,7 @@ static IDirect3DDevice *create_device(IDirectDraw *ddraw, HWND window, DWORD coo surface_desc.dwSize = sizeof(surface_desc); surface_desc.dwFlags = DDSD_CAPS | DDSD_ZBUFFERBITDEPTH | DDSD_WIDTH | DDSD_HEIGHT; surface_desc.ddsCaps.dwCaps = DDSCAPS_ZBUFFER; - surface_desc.dwZBufferBitDepth = z_depth; + U2(surface_desc).dwZBufferBitDepth = z_depth; surface_desc.dwWidth = 640; surface_desc.dwHeight = 480; hr = IDirectDraw_CreateSurface(ddraw, &surface_desc, &ds, NULL); @@ -408,7 +408,7 @@ static void test_clipper_blt(void) hr = IDirectDrawSurface_Lock(src_surface, NULL, &surface_desc, DDLOCK_WAIT, NULL); ok(SUCCEEDED(hr), "Failed to lock source surface, hr %#x.\n", hr); - ok(surface_desc.lPitch == 2560, "Got unexpected surface pitch %u.\n", surface_desc.lPitch); + ok(U1(surface_desc).lPitch == 2560, "Got unexpected surface pitch %u.\n", U1(surface_desc).lPitch); ptr = surface_desc.lpSurface; memcpy(&ptr[ 0], &src_data[ 0], 6 * sizeof(DWORD)); memcpy(&ptr[ 640], &src_data[ 6], 6 * sizeof(DWORD)); diff --git a/dlls/ddraw/tests/ddraw2.c b/dlls/ddraw/tests/ddraw2.c index 824ddb1..d6a79af 100644 --- a/dlls/ddraw/tests/ddraw2.c +++ b/dlls/ddraw/tests/ddraw2.c @@ -144,7 +144,7 @@ static IDirect3DDevice2 *create_device(IDirectDraw2 *ddraw, HWND window, DWORD c surface_desc.dwSize = sizeof(surface_desc); surface_desc.dwFlags = DDSD_CAPS | DDSD_ZBUFFERBITDEPTH | DDSD_WIDTH | DDSD_HEIGHT; surface_desc.ddsCaps.dwCaps = DDSCAPS_ZBUFFER; - surface_desc.dwZBufferBitDepth = z_depth; + U2(surface_desc).dwZBufferBitDepth = z_depth; surface_desc.dwWidth = 640; surface_desc.dwHeight = 480; hr = IDirectDraw2_CreateSurface(ddraw, &surface_desc, &ds, NULL); @@ -418,7 +418,7 @@ static void test_clipper_blt(void) hr = IDirectDrawSurface_Lock(src_surface, NULL, &surface_desc, DDLOCK_WAIT, NULL); ok(SUCCEEDED(hr), "Failed to lock source surface, hr %#x.\n", hr); - ok(surface_desc.lPitch == 2560, "Got unexpected surface pitch %u.\n", surface_desc.lPitch); + ok(U1(surface_desc).lPitch == 2560, "Got unexpected surface pitch %u.\n", U1(surface_desc).lPitch); ptr = surface_desc.lpSurface; memcpy(&ptr[ 0], &src_data[ 0], 6 * sizeof(DWORD)); memcpy(&ptr[ 640], &src_data[ 6], 6 * sizeof(DWORD)); diff --git a/dlls/ddraw/tests/ddraw4.c b/dlls/ddraw/tests/ddraw4.c index 68a478f..69dcef7 100644 --- a/dlls/ddraw/tests/ddraw4.c +++ b/dlls/ddraw/tests/ddraw4.c @@ -640,7 +640,7 @@ static void test_clipper_blt(void) hr = IDirectDrawSurface4_Lock(src_surface, NULL, &surface_desc, DDLOCK_WAIT, NULL); ok(SUCCEEDED(hr), "Failed to lock source surface, hr %#x.\n", hr); - ok(surface_desc.lPitch == 2560, "Got unexpected surface pitch %u.\n", surface_desc.lPitch); + ok(U1(surface_desc).lPitch == 2560, "Got unexpected surface pitch %u.\n", U1(surface_desc).lPitch); ptr = surface_desc.lpSurface; memcpy(&ptr[ 0], &src_data[ 0], 6 * sizeof(DWORD)); memcpy(&ptr[ 640], &src_data[ 6], 6 * sizeof(DWORD)); diff --git a/dlls/ddraw/tests/ddraw7.c b/dlls/ddraw/tests/ddraw7.c index d5477f0..e3f59dd 100644 --- a/dlls/ddraw/tests/ddraw7.c +++ b/dlls/ddraw/tests/ddraw7.c @@ -633,7 +633,7 @@ static void test_clipper_blt(void) hr = IDirectDrawSurface7_Lock(src_surface, NULL, &surface_desc, 0, NULL); ok(SUCCEEDED(hr), "Failed to lock source surface, hr %#x.\n", hr); - ok(surface_desc.lPitch == 2560, "Got unexpected surface pitch %u.\n", surface_desc.lPitch); + ok(U1(surface_desc).lPitch == 2560, "Got unexpected surface pitch %u.\n", U1(surface_desc).lPitch); ptr = surface_desc.lpSurface; memcpy(&ptr[ 0], &src_data[ 0], 6 * sizeof(DWORD)); memcpy(&ptr[ 640], &src_data[ 6], 6 * sizeof(DWORD));

1 0

Francois Gouget : po: Fix a formatting directive in the Dutch translation.
by Alexandre Julliard Jan. 23, 2012

Jan. 23, 2012

Module: wine Branch: master Commit: 224358e63464b2f6d0a236fbab34626c3a741f0f URL: http://source.winehq.org/git/wine.git/?a=commit;h=224358e63464b2f6d0a236fba… Author: Francois Gouget <fgouget(a)free.fr> Date: Mon Jan 23 00:39:49 2012 +0100 po: Fix a formatting directive in the Dutch translation. --- po/nl.po | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/po/nl.po b/po/nl.po index a9e1e9b..480bc82 100644 --- a/po/nl.po +++ b/po/nl.po @@ -10457,7 +10457,7 @@ msgid "" msgstr "" "%1\n" "Dit bestand bevat Unicode tekens die verloren zullen gaan\n" -"als dit bestand wordt opgeslagen in de %1 codering.\n" +"als dit bestand wordt opgeslagen in de %2 codering.\n" "Klik op Annuleren om deze tekens te behouden, en selecteer\n" "daarna een van de Unicode-opties in de coderingskeuzelijst.\n" "Wilt u doorgaan?"

1 0

Francois Gouget : po: Fix some formatting directives in the Korean translation.
by Alexandre Julliard Jan. 23, 2012

Jan. 23, 2012

Module: wine Branch: master Commit: 173d486df2f67cf9aff85ac6dd90ff0b75aac8cb URL: http://source.winehq.org/git/wine.git/?a=commit;h=173d486df2f67cf9aff85ac6d… Author: Francois Gouget <fgouget(a)free.fr> Date: Mon Jan 23 00:39:53 2012 +0100 po: Fix some formatting directives in the Korean translation. --- po/ko.po | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/po/ko.po b/po/ko.po index e6db6e8..591cc71 100644 --- a/po/ko.po +++ b/po/ko.po @@ -4147,7 +4147,7 @@ msgstr "재배치 체인이 %1의 세그맨트 제한 도달\n" #: winerror.mc:801 msgid "Infinite loop in relocation chain in %1\n" -msgstr "% 1에 재배치 체인의 무한 루프\n" +msgstr "%1에 재배치 체인의 무한 루프\n" #: winerror.mc:806 msgid "Environment variable not found\n" @@ -9864,7 +9864,7 @@ msgstr "현재 날짜는 %1임\n" #: cmd.rc:306 msgid "Current Time is %1\n" -msgstr "현재 시간은 %s임\n" +msgstr "현재 시간은 %1임\n" #: cmd.rc:307 msgid "Enter new date: " @@ -10348,7 +10348,7 @@ msgid "" "one of the Unicode options in the Encoding drop down list.\n" "Continue?" msgstr "" -"%s\n" +"%1\n" "유니코드 문자를 포함하고 있는 이 파일은 만약 당신이 %2 인코딩으로\n" "파일을 저장하면 문자의 손실이 있을 것입니다.\n" "이 문자를 보존하려면 취소를 클릭하고, 인코딩 드롭다운목록의 유니코드\n"

1 0

Eric Pouech : dbghelp: On x86-64 CPUs, add capability to interpret properly the epilog part.
by Alexandre Julliard Jan. 23, 2012

Jan. 23, 2012

Module: wine Branch: master Commit: 2ed88d7ac2cf5b2962c812e0673da67d99ddc077 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2ed88d7ac2cf5b2962c812e06… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sun Jan 22 13:00:06 2012 +0100 dbghelp: On x86-64 CPUs, add capability to interpret properly the epilog part. --- dlls/dbghelp/cpu_x86_64.c | 79 +++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 77 insertions(+), 2 deletions(-) diff --git a/dlls/dbghelp/cpu_x86_64.c b/dlls/dbghelp/cpu_x86_64.c index 6288d38..160590d 100644 --- a/dlls/dbghelp/cpu_x86_64.c +++ b/dlls/dbghelp/cpu_x86_64.c @@ -349,6 +349,82 @@ static BOOL is_inside_epilog(struct cpu_stack_walk* csw, DWORD64 pc) } } +static BOOL interpret_epilog(struct cpu_stack_walk* csw, ULONG64 pc, CONTEXT *context ) +{ + BYTE insn, val8; + WORD val16; + LONG val32; + DWORD64 val64; + + for (;;) + { + BYTE rex = 0; + + if (!sw_read_mem(csw, pc, &insn, 1)) return FALSE; + if ((insn & 0xf0) == 0x40) + { + rex = insn & 0x0f; /* rex prefix */ + if (!sw_read_mem(csw, ++pc, &insn, 1)) return FALSE; + } + + switch (insn) + { + case 0x58: /* pop %rax/r8 */ + case 0x59: /* pop %rcx/r9 */ + case 0x5a: /* pop %rdx/r10 */ + case 0x5b: /* pop %rbx/r11 */ + case 0x5c: /* pop %rsp/r12 */ + case 0x5d: /* pop %rbp/r13 */ + case 0x5e: /* pop %rsi/r14 */ + case 0x5f: /* pop %rdi/r15 */ + if (!sw_read_mem(csw, context->Rsp, &val64, sizeof(DWORD64))) return FALSE; + set_int_reg(context, insn - 0x58 + (rex & 1) * 8, val64); + context->Rsp += sizeof(ULONG64); + pc++; + continue; + case 0x81: /* add $nnnn,%rsp */ + if (!sw_read_mem(csw, pc + 2, &val32, sizeof(ULONG))) return FALSE; + context->Rsp += (LONG)val32; + pc += 2 + sizeof(LONG); + continue; + case 0x83: /* add $n,%rsp */ + if (!sw_read_mem(csw, pc + 2, &val8, sizeof(BYTE))) return FALSE; + context->Rsp += (signed char)val8; + pc += 3; + continue; + case 0x8d: + if (!sw_read_mem(csw, pc + 1, &insn, sizeof(BYTE))) return FALSE; + if ((insn >> 6) == 1) /* lea n(reg),%rsp */ + { + if (!sw_read_mem(csw, pc + 2, &val8, sizeof(BYTE))) return FALSE; + context->Rsp = get_int_reg( context, (insn & 7) + (rex & 1) * 8 ) + (signed char)val8; + pc += 3; + } + else /* lea nnnn(reg),%rsp */ + { + if (!sw_read_mem(csw, pc + 2, &val32, sizeof(ULONG))) return FALSE; + context->Rsp = get_int_reg( context, (insn & 7) + (rex & 1) * 8 ) + (LONG)val32; + pc += 2 + sizeof(LONG); + } + continue; + case 0xc2: /* ret $nn */ + if (!sw_read_mem(csw, context->Rsp, &val64, sizeof(DWORD64))) return FALSE; + if (!sw_read_mem(csw, pc + 1, &val16, sizeof(WORD))) return FALSE; + context->Rip = val64; + context->Rsp += sizeof(ULONG64) + val16; + return TRUE; + case 0xc3: /* ret */ + if (!sw_read_mem(csw, context->Rsp, &val64, sizeof(DWORD64))) return FALSE; + context->Rip = val64; + context->Rsp += sizeof(ULONG64); + return TRUE; + /* FIXME: add various jump instructions */ + } + FIXME("unsupported insn %x\n", insn); + return FALSE; + } +} + static BOOL default_unwind(struct cpu_stack_walk* csw, CONTEXT* context) { if (!sw_read_mem(csw, context->Rsp, &context->Rip, sizeof(DWORD64))) @@ -404,8 +480,7 @@ static BOOL interpret_function_table_entry(struct cpu_stack_walk* csw, prolog_offset = ~0; if (is_inside_epilog(csw, context->Rip)) { - FIXME("epilog management not fully done\n"); - /* interpret_epilog((const BYTE*)frame->AddrPC.Offset, context); */ + interpret_epilog(csw, context->Rip, context); return TRUE; } }

1 0

Eric Pouech : dbghelp: In x86-64 support code, fix RUNTIME_FUNCTION dumping .
by Alexandre Julliard Jan. 23, 2012

Jan. 23, 2012

Module: wine Branch: master Commit: 7c2a5e184d559520d59dd543661d834680d158d2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7c2a5e184d559520d59dd5436… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sun Jan 22 13:00:12 2012 +0100 dbghelp: In x86-64 support code, fix RUNTIME_FUNCTION dumping. --- dlls/dbghelp/cpu_x86_64.c | 61 +++++++++++++++++++++++++++++--------------- 1 files changed, 40 insertions(+), 21 deletions(-) diff --git a/dlls/dbghelp/cpu_x86_64.c b/dlls/dbghelp/cpu_x86_64.c index 160590d..8b101d2 100644 --- a/dlls/dbghelp/cpu_x86_64.c +++ b/dlls/dbghelp/cpu_x86_64.c @@ -2,7 +2,7 @@ * File cpu_x86_64.c * * Copyright (C) 1999, 2005 Alexandre Julliard - * Copyright (C) 2009 Eric Pouech. + * Copyright (C) 2009, 2011 Eric Pouech. * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -125,39 +125,45 @@ union handler_data ULONG handler; }; -static void dump_unwind_info(HANDLE hProcess, ULONG64 base, RUNTIME_FUNCTION *function) +static void dump_unwind_info(struct cpu_stack_walk* csw, ULONG64 base, RUNTIME_FUNCTION *function) { static const char * const reg_names[16] = { "rax", "rcx", "rdx", "rbx", "rsp", "rbp", "rsi", "rdi", "r8", "r9", "r10", "r11", "r12", "r13", "r14", "r15" }; - union handler_data *handler_data; + union handler_data handler_data; char buffer[sizeof(UNWIND_INFO) + 256 * sizeof(UNWIND_CODE)]; UNWIND_INFO* info = (UNWIND_INFO*)buffer; unsigned int i, count; - SIZE_T r; + RUNTIME_FUNCTION snext; + ULONG64 addr; TRACE("**** func %x-%x\n", function->BeginAddress, function->EndAddress); for (;;) { if (function->UnwindData & 1) { -#if 0 - RUNTIME_FUNCTION *next = (RUNTIME_FUNCTION*)((char*)base + (function->UnwindData & ~1)); + if (!sw_read_mem(csw, base + function->UnwindData, &snext, sizeof(snext))) + { + TRACE("Couldn't unwind RUNTIME_INFO\n"); + return; + } TRACE("unwind info for function %p-%p chained to function %p-%p\n", (char*)base + function->BeginAddress, (char*)base + function->EndAddress, - (char*)base + next->BeginAddress, (char*)base + next->EndAddress); - function = next; + (char*)base + snext.BeginAddress, (char*)base + snext.EndAddress); + function = &snext; continue; -#else - FIXME("NOT SUPPORTED\n"); -#endif } - ReadProcessMemory(hProcess, (char*)base + function->UnwindData, info, sizeof(*info), &r); - ReadProcessMemory(hProcess, (char*)base + function->UnwindData + FIELD_OFFSET(UNWIND_INFO, UnwindCode), - info->UnwindCode, 256 * sizeof(UNWIND_CODE), &r); + addr = base + function->UnwindData; + if (!sw_read_mem(csw, addr, info, FIELD_OFFSET(UNWIND_INFO, UnwindCode)) || + !sw_read_mem(csw, addr + FIELD_OFFSET(UNWIND_INFO, UnwindCode), + info->UnwindCode, info->CountOfCodes * sizeof(UNWIND_CODE))) + { + FIXME("couldn't read memory for UNWIND_INFO\n"); + return; + } TRACE("unwind info at %p flags %x prolog 0x%x bytes function %p-%p\n", - info, info->Flags, info->SizeOfProlog, + (char*)addr, info->Flags, info->SizeOfProlog, (char*)base + function->BeginAddress, (char*)base + function->EndAddress); if (info->FrameRegister) @@ -222,18 +228,31 @@ static void dump_unwind_info(HANDLE hProcess, ULONG64 base, RUNTIME_FUNCTION *fu } } - handler_data = (union handler_data*)&info->UnwindCode[(info->CountOfCodes + 1) & ~1]; + addr += FIELD_OFFSET(UNWIND_INFO, UnwindCode) + + ((info->CountOfCodes + 1) & ~1) * sizeof(UNWIND_CODE); if (info->Flags & UNW_FLAG_CHAININFO) { + if (!sw_read_mem(csw, addr, &handler_data, sizeof(handler_data.chain))) + { + FIXME("couldn't read memory for handler_data.chain\n"); + return; + } TRACE(" chained to function %p-%p\n", - (char*)base + handler_data->chain.BeginAddress, - (char*)base + handler_data->chain.EndAddress); - function = &handler_data->chain; + (char*)base + handler_data.chain.BeginAddress, + (char*)base + handler_data.chain.EndAddress); + function = &handler_data.chain; continue; } if (info->Flags & (UNW_FLAG_EHANDLER | UNW_FLAG_UHANDLER)) + { + if (!sw_read_mem(csw, addr, &handler_data, sizeof(handler_data.handler))) + { + FIXME("couldn't read memory for handler_data.handler\n"); + return; + } TRACE(" handler %p data at %p\n", - (char*)base + handler_data->handler, &handler_data->handler + 1); + (char*)base + handler_data.handler, (char*)addr + sizeof(handler_data.handler)); + } break; } } @@ -448,7 +467,7 @@ static BOOL interpret_function_table_entry(struct cpu_stack_walk* csw, /* FIXME: we have some assumptions here */ assert(context); - dump_unwind_info(csw->hProcess, sw_module_base(csw, context->Rip), function); + dump_unwind_info(csw, sw_module_base(csw, context->Rip), function); newframe = context->Rsp; for (;;) {

1 0

André Hentschel : po: Update German translation.
by Alexandre Julliard Jan. 23, 2012

Jan. 23, 2012

Module: wine Branch: master Commit: 861d475839cb892e25e7ded872a43d7099e7130e URL: http://source.winehq.org/git/wine.git/?a=commit;h=861d475839cb892e25e7ded87… Author: André Hentschel <nerv(a)dawncrow.de> Date: Fri Jan 20 19:44:31 2012 +0100 po: Update German translation. --- po/de.po | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/po/de.po b/po/de.po index 0b95e35..9373eb9 100644 --- a/po/de.po +++ b/po/de.po @@ -5,7 +5,7 @@ msgstr "" "Project-Id-Version: Wine\n" "Report-Msgid-Bugs-To: http://bugs.winehq.org\n" "POT-Creation-Date: N/A\n" -"PO-Revision-Date: 2012-01-18 21:56+0100\n" +"PO-Revision-Date: 2012-01-20 19:43+0100\n" "Last-Translator: André Hentschel\n" "Language-Team: none\n" "Language: German\n" @@ -1311,19 +1311,19 @@ msgstr "Zertifizierungsstellen Emittent" #: crypt32.rc:57 msgid "Certification Template Name" -msgstr "Zertifikationsvorlagenname" +msgstr "Zertifizierungsvorlagenname" #: crypt32.rc:58 msgid "Certificate Type" -msgstr "Zertifikationstyp" +msgstr "Zertifikatstyp" #: crypt32.rc:59 msgid "Certificate Manifold" -msgstr "Zertifikationsvervielfältigung" +msgstr "Zertifikatsvervielfältigung" #: crypt32.rc:60 msgid "Netscape Cert Type" -msgstr "Netscape Zertifikationstyp" +msgstr "Netscape Zertifikatstyp" #: crypt32.rc:61 msgid "Netscape Base URL" @@ -1339,7 +1339,7 @@ msgstr "Netscape CA Sperrungs-URL" #: crypt32.rc:64 msgid "Netscape Cert Renewal URL" -msgstr "Netscape Zertifikationserneuerungs-URL" +msgstr "Netscape Zertifikatserneuerungs-URL" #: crypt32.rc:65 msgid "Netscape CA Policy URL" @@ -1467,7 +1467,7 @@ msgstr "Richtlinieneinschränkungen" #: crypt32.rc:96 msgid "Cross-Certificate Distribution Points" -msgstr "Verteilungspunkte für Kreuzzertifikate" +msgstr "Zertifikatsübergreifende Verteilungspunkte" #: crypt32.rc:97 msgid "Application Policies"

1 0

André Hentschel : d3drm: Forward all IUnknown methods from IDirect3DRMFrame3.
by Alexandre Julliard Jan. 23, 2012

Jan. 23, 2012

Module: wine Branch: master Commit: dc700123cb8daa05899ea86089c8fa5c99cd9194 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dc700123cb8daa05899ea8608… Author: André Hentschel <nerv(a)dawncrow.de> Date: Sun Jan 22 18:35:53 2012 +0100 d3drm: Forward all IUnknown methods from IDirect3DRMFrame3. --- dlls/d3drm/frame.c | 14 ++------------ 1 files changed, 2 insertions(+), 12 deletions(-) diff --git a/dlls/d3drm/frame.c b/dlls/d3drm/frame.c index 7c0d97a..d91def8 100644 --- a/dlls/d3drm/frame.c +++ b/dlls/d3drm/frame.c @@ -939,23 +939,13 @@ static HRESULT WINAPI IDirect3DRMFrame3Impl_QueryInterface(IDirect3DRMFrame3* if static ULONG WINAPI IDirect3DRMFrame3Impl_AddRef(IDirect3DRMFrame3* iface) { IDirect3DRMFrameImpl *This = impl_from_IDirect3DRMFrame3(iface); - - TRACE("(%p)\n", This); - - return InterlockedIncrement(&This->ref); + return IDirect3DRMFrame2_AddRef(&This->IDirect3DRMFrame2_iface); } static ULONG WINAPI IDirect3DRMFrame3Impl_Release(IDirect3DRMFrame3* iface) { IDirect3DRMFrameImpl *This = impl_from_IDirect3DRMFrame3(iface); - ULONG ref = InterlockedDecrement(&This->ref); - - TRACE("(%p)\n", This); - - if (!ref) - HeapFree(GetProcessHeap(), 0, This); - - return ref; + return IDirect3DRMFrame2_Release(&This->IDirect3DRMFrame2_iface); } /*** IDirect3DRMObject methods ***/

1 0