September 2010 - Wine-Devel

Backtraces in crash window
by Jerome Leclanche Sept. 30, 2010

Sept. 30, 2010

Adding copypastable backtraces to the crash window might go a long way to prevent this sort of attachments: http://bugs2.winehq.org/attachment.cgi?id=31033 Maybe under a "Crash information" link or something similar. It wouldn't always do the trick (seeing as most of the time the error only shows in the fixmes, if not the traces), but at least it would get rid of some completely useless attachments, imho. Has this been discussed before? J. Leclanche

2 1

Need to retrieve VS_VERSION_INFO for 16, 32, 64 bit windows binaries
by Kapil Arora Sept. 30, 2010

Sept. 30, 2010

Hi, I've created a small executable on windows which calls GetFileVersionInfo() and VerQueryValue() to extract VS_VERSION_INFO from various windows binary files. I want to use this utility under wine on linux system. Everything works fine for most of the files but for some older dlls I'm not getting correct version information. In the wine sources I found some of the functions for previous (16- bit) version info extraction to be stubs. I tried replacing version.dll in $HOME/.wine/drive_c/windows/system32 with the one on my windows XP box but nothing happens. I still don't get complete information. Is there a way to achieve this. Thanks, Kapil.

2 1

Re: [PATCH] jscript: throw TypeError if T in 'new T' is not an object.
by Jacek Caban Sept. 30, 2010

Sept. 30, 2010

Hi Reece, On 9/30/10 12:20 AM, Reece Dunn wrote: > This addresses a fixme in the jscript code that is currently triggered > when running the Big Fish Games client, with a test. if(V_VT(&constr) != VT_DISPATCH) { - FIXME("throw TypeError\n"); + hres = throw_type_error(ctx->parser->script, ei, IDS_OBJECT_EXPECTED, NULL); VariantClear(&constr); - return E_FAIL; + return hres; It would be cleaner to clear constr before throw_type_error so that you can return its result directly, without storing it in hres. exception_test(function() {"test" in null;}, "TypeError", -2146823281); exception_test(function() {"test" in nullDisp;}, "TypeError", -2146823281); +exception_test(function() {new null;}, "TypeError", -2146823281); While you're at this, it would be nice to add a test (and fix) for nullDisp and test some other types (eg. undefined and number). Jacek

2 1

Re: Right-To-Left (RTL) languages and Wine
by James Mckenzie Sept. 30, 2010

Sept. 30, 2010

Alexandre Julliard <julliard(a)winehq.org> wrote: >Sent: Sep 29, 2010 11:25 AM >To: Shachar Shemesh <shachar(a)shemesh.biz> >Cc: Paul Vriens <paul.vriens.wine(a)gmail.com>, "'wine-devel(a)winehq.org'" <wine-devel(a)winehq.org> >Subject: Re: Right-To-Left (RTL) languages and Wine > >Shachar Shemesh <shachar(a)shemesh.biz> writes: > >> Did I mention that the automatic mirroring is a broken idea >> implemented in a broken way already? > >What do you consider broken about it? > >> Aside from notepad, for which the difference is very small, and most >> people would regard a default LTR control as the correct behavior >> anyway, what other applications are you concerned over? > >Pretty much all of them. For example, Wordpad has combo boxes in the >toolbars, those won't be RTL without changing the process layout. >Winefile and Regedit have treeviews that would need mirroring, etc. > And Microsoft Office appears different in RTL rather than LTR. I used to work with someone that had the Arabic version of these programs and it would switch from RTL when typing in Arabic to LTR when typing in a Latin-based language. As AJ has stated, there is more to do than just 'mirroring' to get RTL into Wine. I remember this was quite difficult to get into OpenOffice.org as well when I was testing it (I think I still have Hebrew and Arabic test files for Writer in OpenDocument (.odt) format.) James McKenzie

3 3

Re: [PATCH] jscript: throw TypeError if T in 'new T' is not an object.
by testbot＠testbot.winehq.org Sept. 30, 2010

Sept. 30, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5608 Your paranoid android. === W98SE (32 bit) === No test summary line found === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found

1 0

Re: msxml3/httprequest: Add basic bind callback for moniker binding
by testbot＠testbot.winehq.org Sept. 30, 2010

Sept. 30, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5605 Your paranoid android. === W98SE (32 bit domdoc) === domdoc.c:2917: Test failed: got 0x8000ffff domdoc.c:2979: Test failed: got 0x8000ffff === W2KPROSP4 (32 bit domdoc) === domdoc.c:2917: Test failed: got 0x8000ffff domdoc.c:2979: Test failed: got 0x8000ffff

1 0

Right-To-Left (RTL) languages and Wine
by Paul Vriens Sept. 29, 2010

Sept. 29, 2010

Hi, I've been in talks with a Hebrew translator for some time now and I'd like to have Wine being able to start doing more RTL stuff. I'm looking for example at http://bugs.winehq.org/show_bug.cgi?id=1158 If I run an English winmine.exe on an Arabic box (thanks to the testbot) the main window will shows as normal LTR window layout with the menu's left justified as well. So eventhough the locale is set to Arabic it doesn't turn 'everything' into RTL. If we don't change the resource files for the RTL languages, that same logic will give us LTR Wine applications or am I wrong here? -- Cheers, Paul.

4 16

Wanted: small C program to drop all capabilities but cap_sys_ptrace
by Scott Ritchie Sept. 29, 2010

Sept. 29, 2010

Ubuntu 10.10 is coming out soon, and its new kernel settings prevent Wine apps from looking at each others' memory. This breaks World of Warcraft, among other things. See: http://bugs.winehq.org/show_bug.cgi?id=24193 What's needed is a very small shim for Wine that can be setuid 0, but then release all capabilities except what Wine actually needs -- what a normal user has, and cap_sys_ptrace. On an Ubuntu system, this is very similar to what DHCP and PING do -- setuid 0, however they drop all privs except cap_net_rawio at the start. Existing code can be used: http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/dapper/dhcp3/dapper/ann… http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/dapper/dhcp3/dapper/ann… Basically, I need someone to write this shim for me. The long term solution is probably to just package Wine such that the wine binary itself has cap_sys_ptrace, however currently Ubuntu has no support for this kind of extended attribute in the packaging system so workarounds like the above for DHCP need to be done. I suspect other distros have similar issues. Thanks, Scott Ritchie

5 6

Re: [PATCH 1/5] d3dcompiler_43/tests: Added HLSL test suite
by Stefan Dösinger Sept. 29, 2010

Sept. 29, 2010

Am 26.09.2010 um 19:51 schrieb Travis Athougies: > + data = compute_shader_fullscreen9(device, vshader_passthru, pshader, quad_geometry, > + D3DFMT_A8R8G8B8, 1, 1); > + > + ok(data[0] == D3DCOLOR_ARGB(0, 0, 255, 255), > + "swizzle_test: Got color %08x (should be 0x0000FFFF)\n", data[0]); You leak the returned data here. On the big picture of this, I don't like the void * blob returned from compute_shader_fullscreen. As far as I can see the only position dependent aspect of HLSL is vpos(or rather, the HLSL equivalent). For all others it would be OK to return a ARGB color. > + /* The Direct3D 9 docs state that we cannot lock a render target surface, > + instead we must copy the render target onto this surface to lock it */ I think you can, if you create it with D3DUSAGE_DYNAMIC | D3DUSAGE_RENDERTARGET. If you want the backbuffer to be lockable you'll need some device creation flag whose name I forgot.

3 5

Re: winex11: Add support for animated cursors in X11 driver (try 2).
by Henri Verbeet Sept. 29, 2010

Sept. 29, 2010

I didn't really look at the patch at all, but that's not how HeapReAlloc() works.

2 1

mshtml and friends
by Reece Dunn Sept. 29, 2010

Sept. 29, 2010

Hi, The current state of support for the Internet Explorer components in Wine is good, but there are applications that don't work with it because they are dependent on more of the functionality provided by Internet Explorer via the WebBrowser control. I have had a scan through some of the mshtml code, and see various things not handled -- such as the onload handler of the HTMLBody element -- but I am not very familiar with this area. I am interested in helping out to improve this area -- my aim is to not require the `winetricks ie6` command to get some of these applications (specifically the Big Fish Games client) to a usable state. Therefore, I am wondering if anyone knows where the best place is to start looking (e.g. known areas of missing functionality) or how to debug applications (and interpret WINEDEBUG output) to identify where the issues are. Thanks, - Reece

2 7

Re: [PATCH] mshtml: implement ActiveScriptSite_OnScriptError.
by Jacek Caban Sept. 29, 2010

Sept. 29, 2010

On 9/29/10 9:39 AM, Reece Dunn wrote: > Hi, > > This reports any errors sent to the mshtml ActiveScriptSite > OnScriptError handler to the user and traces it to ERR to aid > debugging. static HRESULT WINAPI ActiveScriptSite_OnScriptError(IActiveScriptSite *iface, IActiveScriptError *pscripterror) { ScriptHost *This = ACTSCPSITE_THIS(iface); - FIXME("(%p)->(%p)\n", This, pscripterror); - return E_NOTIMPL; + EXCEPINFO excep; + HRESULT hr; + + TRACE("(%p)->(%p)\n", This, pscripterror); + + hr = IActiveScriptError_GetExceptionInfo(pscripterror,&excep); + + ERR("ActiveScript Error: %s source: %s\n", + debugstr_w(excep.bstrDescription), debugstr_w(excep.bstrSource)); ERR is not appropriate in this case. + MessageBoxW(NULL, excep.bstrDescription, excep.bstrSource, MB_OK|MB_ICONERROR); This is not how this function should work. If you want to implement it, please start with a test case. It probably should fire onerror event. Jacek

2 1

Re: wined3d: Keep track of textures projection state in compiled pixel shaders. [try 2]
by Henri Verbeet Sept. 29, 2010

Sept. 29, 2010

On 28 September 2010 23:07, Matteo Bruni <matteo.mystral(a)gmail.com> wrote: > You still need to use this state in the shader backends, instead of getting it from the stateblock.

1 0

winscard.dll
by viny Sept. 29, 2010

Sept. 29, 2010

Hi all, Is anyone working on winscard.dll ? If not, may I propose patches ? I would like to have a working winscard.dll in wine. -- Vincent

7 10

Wine and security
by Dan Kegel Sept. 29, 2010

Sept. 29, 2010

I keep seeing people asking about wine and security, e.g. http://bugs.winehq.org/show_bug.cgi?id=24550 or http://forum.winehq.org/viewtopic.php?t=9770 or https://answers.launchpad.net/ubuntu/+source/wine/+question/59148 ... It seems worth listing the things one can do to partially lock down wine, so I started a page at http://wiki.winehq.org/SecuringWine which is appropriately morose, yet lists a few things one could do if one really wanted to. Corrections or additions welcome.

2 1

Re: comctl32/tab: Fix DRAWITEMSTRUCT filling when extra item data of a custom size is used (try2)
by testbot＠testbot.winehq.org Sept. 29, 2010

Sept. 29, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5580 Your paranoid android. === W98SE (32 bit tab) === tab.c:1354: Test failed: got 0xdedede, expected 0xdededede === W7PROX64 (64 bit tab) === tab.c:1354: Test failed: got 0xdededede, expected 0xdededede

1 0

Re: user32: Unlike RedrawWindow, UpdateWindow doesn't accept a NULL hwnd +tests (try3)
by testbot＠testbot.winehq.org Sept. 29, 2010

Sept. 29, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5575 Your paranoid android. === WVISTAADM (32 bit win) === win.c:835: Test failed: wrong dwStyle: 84cb0000 != 94cb0000 for 000201C0 in hook HCBT_MINMAX win.c:3236: Test failed: Expected GetUpdateRect to return non-zero, got 0 win.c:3237: Test failed: Update rectangle is empty! win.c:3581: Test failed: pixel should be black, color is ffffffff win.c:3601: Test failed: pixel should be black, color is ffffffff win.c:3643: Test failed: wrong update region win.c:3656: Test failed: wrong update region win.c:3674: Test failed: wrong update region win.c:3684: Test failed: wrong update region in excessive scroll win.c:3706: Test failed: wrong update region win.c:3715: Test failed: wrong update region win.c:3724: Test failed: wrong update region win.c:3739: Test failed: wrong update region win.c:3819: Test failed: pixel should be black, color is ffffffff win.c:3823: Test failed: pixel should be black, color is ffffffff win.c:3831: Test failed: rects do not match (0,0-0,0) / (0,0-100,100) win.c:3838: Test failed: wrong update region win.c:3848: Test failed: wrong update region win.c:2322: Test failed: rgn right 0 not inside win rect -31700 win.c:2324: Test failed: rgn bottom 0 not inside win rect -31700 === W7PROX64 (32 bit win) === win.c:2684: Test failed: GetActiveWindow() = 00000000 win.c:2684: Test failed: GetFocus() = 00000000 win.c:2710: Test failed: GetActiveWindow() = 00000000 win.c:2710: Test failed: GetFocus() = 00000000 win.c:2769: Test failed: GetActiveWindow() = 00000000 win.c:2769: Test failed: GetFocus() = 00000000 === W7PROX64 (64 bit win) === win.c:2684: Test failed: GetActiveWindow() = 0000000000000000 win.c:2684: Test failed: GetFocus() = 0000000000000000 win.c:2710: Test failed: GetActiveWindow() = 0000000000000000 win.c:2710: Test failed: GetFocus() = 0000000000000000 win.c:2769: Test failed: GetActiveWindow() = 0000000000000000 win.c:2769: Test failed: GetFocus() = 0000000000000000

1 0

Re: [1/7] msvcrt: Initialize the environ global variables on DLL load.
by Alexandre Julliard Sept. 28, 2010

Sept. 28, 2010

Andrew Nguyen <anguyen(a)codeweavers.com> writes: > + ok( environ != NULL, "Expected the environ global variable to be initialized on startup\n" ); > + ok( _wenviron != NULL, "Expected the _wenviron global variable to be initialized on startup\n" ); This won't test what you want, importing variables is not supported on Wine, you have to load the variables explicitly with GetProcAddress, -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 3/3] winex11: Add support for animated cursors in X11 driver.
by Alexandre Julliard Sept. 28, 2010

Sept. 28, 2010

Erich Hoover <ehoover(a)mines.edu> writes: > @@ -508,35 +517,90 @@ static Cursor create_xcursor_cursor( HDC hdc, ICONINFO *icon, int width, int hei > info->bmiHeader.biYPelsPerMeter = 0; > info->bmiHeader.biClrUsed = 0; > info->bmiHeader.biClrImportant = 0; > - GetDIBits( hdc, icon->hbmColor, 0, height, image->pixels, info, DIB_RGB_COLORS ); > + hbmColor = CreateDIBSection( hdc, info, DIB_RGB_COLORS, (VOID **) &mem_bits, NULL, 0); > + if (!hbmColor) > + { > + ERR("Failed to create DIB section for image!\n"); > + goto cleanup; > + } > + SelectObject( hdc, hbmColor ); > + if (!DrawIconEx( hdc, 0, 0, icon, width, height, istep, NULL, DI_NORMAL )) > + { > + TRACE("Could not draw frame %d (walk past end of frames).\n", istep); > + goto cleanup; > + } > + memcpy(image->pixels, mem_bits, info->bmiHeader.biSizeImage); You should be reusing the info, DC and bitmap objects for all frames instead of always recreating everything from scratch. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: widl: Explicitly declare default function's calling convention to avoid win64 problems.
by Alexandre Julliard Sept. 28, 2010

Sept. 28, 2010

Jacek Caban <jacek(a)codeweavers.com> writes: > @@ -369,7 +369,7 @@ static void write_type_v(FILE *h, type_t *t, int is_field, int declonly, const c > if (type_get_type_detect_alias(pt) == TYPE_FUNCTION) { > int i; > const char *callconv = get_attrp(pt->attrs, ATTR_CALLCONV); > - if (!callconv && is_object_interface) callconv = "STDMETHODCALLTYPE"; > + if (!callconv) callconv = is_object_interface ? "STDMETHODCALLTYPE" : "__cdecl"; It may be useful to be able to declare functions that use the native calling convention, I don't think we want to prevent that. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [PATCH 2/6] d3dcompiler: Implement D3DGetBlobPart().(try 2)
by Henri Verbeet Sept. 28, 2010

Sept. 28, 2010

2010/9/27 Rico Schüller <kgbricola(a)web.de>: > !strcmp("unrecognized", debug_d3dcompiler_d3d_blob_part(part)) Oh, and please don't do that, I think it's very fragile.

1 0

Re: [PATCH 2/6] d3dcompiler: Implement D3DGetBlobPart().(try 2)
by Henri Verbeet Sept. 28, 2010

Sept. 28, 2010

2010/9/27 Rico Schüller <kgbricola(a)web.de>: > --- Wouldn't it be nicer if the "count" parameter to dxbc_init() was more of a hint? I.e., allow the sections array to grow if needed. For parse_dxbc() this works fine, because you know the number of sections in advance, but as you must have noticed, for creating the destination blob it's much more inconvenient.

1 0

Re: [PATCH 2/6] msxml3: ignore whitespace nodes
by Nikolay Sivov Sept. 28, 2010

Sept. 28, 2010

On 9/28/2010 05:37, Adam Martinson wrote: > From 1f8661342ae63290f212429e382b5776e07b9ebc Mon Sep 17 00:00:00 2001 > From: Adam Martinson<amartinson(a)codeweavers.com> > Date: Mon, 27 Sep 2010 19:29:04 -0500 > Subject: [PATCH 2/6] msxml3: ignore whitespace nodes > > --- > dlls/msxml3/msxml_private.h | 15 +++++++++++++++ > dlls/msxml3/node.c | 9 +++++++++ > dlls/msxml3/nodelist.c | 8 ++++++-- > dlls/msxml3/tests/domdoc.c | 14 ++++++++++++++ > 4 files changed, 44 insertions(+), 2 deletions(-) This should be affected by ::preserveWhiteSpace() property I guess.

2 1

Re: ws2_32(3/3): Implement WSAIoctl for SIO_ROUTING_INTERFACE_QUERY
by Alexandre Julliard Sept. 28, 2010

Sept. 28, 2010

Juan Lang <juan.lang(a)gmail.com> writes: > This one may cause test failures on systems that don't have a route to > the loopback address. As far as I can tell, Windows systems always > have such a route, whereas on *ix it isn't guaranteed. I'm not sure > how to handle that, as broken() isn't allowed for Wine, and such a > networking configuration is at least strange. If it's really always present on Windows and apps depend on it, you'll have to fake a route. Otherwise just don't test it. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: ws2_32/tests: Send zero byte to any connected IrDA device (resend)
by Alexandre Julliard Sept. 28, 2010

Sept. 28, 2010

Wolfgang Schwotzer <wolfgang.schwotzer(a)gmx.net> writes: > Test for bug #19397. > Test only executed, if remote IrDA device is present. > Otherwise test is skipped. (Proposal from Mike Kaplinskiy) I'm not sure this is very useful. It's unlikely to be run on any of the test boxes, and AFAICS this is a kernel bug anyway, it should be a Unix test case not a Wine one. -- Alexandre Julliard julliard(a)winehq.org

1 0