February 2013 - wine-devel

Error handling in RpcBindingVectorFree (dlls/rpcrt4/rpc_binding.c)
by Gerald Pfeifer 22 Nov '20

22 Nov '20

Looking at RPC_STATUS WINAPI RpcBindingVectorFree( RPC_BINDING_VECTOR** BindingVector ) { RPC_STATUS status; ULONG c; TRACE("(%p)\n", BindingVector); for (c=0; c<(*BindingVector)->Count; c++) { status = RpcBindingFree(&(*BindingVector)->BindingH[c]); } HeapFree(GetProcessHeap(), 0, *BindingVector); *BindingVector = NULL; return RPC_S_OK; } we currently always ignore the outcome of RpcBindingFree and return RPC_S_OK. However, there is one case where RpcBindingFree returns something different (which is if *Binding is null when RPC_S_INVALID_BINDING is returned). What is the proper way of handling this? Just keeping the code as is and removing the unused status variable? Breaking the loop once RpcBindingFree returns something different from RPC_S_OK? Continuing and returning the first / the last status different from RPC_S_OK? Gerald

1 1

ntoskrnl.exe: unimplemented function ntoskrnl.exe.IoGetDeviceInterfaces
by Qian Hong 28 Sep '18

28 Sep '18

Dear all, While test another online bank with wine ActiveX, I got an unimplemented fuction of ntoskrnl: IoGetDeviceInterfaces, I found it listed in http://source.winehq.org/WineAPI/ntoskrnl.html as a stup, so I can't understand this log: wine: Unimplemented function ntoskrnl.exe.IoGetDeviceInterfaces called at address 0x7b839552 (thread 0022), starting debugger... Grateful for any explain! env: wine1.3.12 on Ubuntu 10.04 Here are the steps: 1. install an ActiveX from https://e.bank.ecitic.com/perbank5/plugs/CNCBSecPkg_EN.exe $ rm -rf ~/.wine $ winetricks -q mfc42 $ wine CNCBSecPkg_EN.exe fixme:ole:DllRegisterServer stub fixme:win:DisableProcessWindowsGhosting : stub fixme:msg:ChangeWindowMessageFilter c057 00000001 fixme:msg:ChangeWindowMessageFilter c057 00000001 fixme:msg:ChangeWindowMessageFilter c057 00000001 fixme:ole:CoCreateInstance no instance created for interface {ea1afb91-9e28-4b86-90e9-9e9f8a5eefaf} of class {56fdf344-fd6d-11d0-958a-006097c9a090}, hres is 0x80004002 fixme:sfc:SfcIsFileProtected ((nil), L"C:\\Program Files\\\4e2d\4fe1\94f6\884c\7f51\94f6\5b89\5168\63a7\4ef6\\unins000.exe") stub fixme:win:WINNLSEnableIME hUnknown1 0x1011a bUnknown2 0: stub! fixme:win:WINNLSEnableIME hUnknown1 0x1011a bUnknown2 -1: stub! fixme:win:WINNLSEnableIME hUnknown1 0x1011a bUnknown2 0: stub! wine: Call from 0x7b839552 to unimplemented function ntoskrnl.exe.IoGetDeviceInterfaces, aborting wine: Unimplemented function ntoskrnl.exe.IoGetDeviceInterfaces called at address 0x7b839552 (thread 002b), starting debugger... wine: Call from 0x7b839552 to unimplemented function ntoskrnl.exe.IoGetDeviceInterfaces, aborting wine: Call from 0x7b839552 to unimplemented function ntoskrnl.exe.IoGetDeviceInterfaces, aborting 2. open the online bank entry with wine builtin IE, then IE will crash: $ wine iexplore https://e.bank.ecitic.com/perbank5/signIn.do Please checkout the full log here: http://pastebin.com/rbAg7gwj Should I file a singel bug in ntoskrnl component , or separate bugs, one for ntoskrnl and one for the IE crashing? Generalliy what component should I switch while file a bug about IE crashing? Many thanks! -- Regards, Qian Hong - Sent from Ubuntu http://www.ubuntu.com/

5 6

ODBC Databases: Jet/Access success
by Barry Bird 14 Jun '18

14 Jun '18

Good Afternoon. In section 3.3.6.2 of your User Guide you ask readers to report successes with databases other than MS SQL. Well here's one (I know Access 2000 can work with Wine, but this doesn't require Access): How to set up Wine to enable Windows programs that read and write to Jet (Access) databases using ODBC. I write such programs in C and use the API defined in ODBC API Reference http://msdn.microsoft.com/en-us/library/ms714562%28VS.85%29.aspx They work fine on Windows and only require Mingw to be installed to compile and link them, both for console programs and those using Windows SDK. You then can write interactive programs in C which use a full-featured SQL database which comes bundled with Windows. No need to purchase Access. With the set-up below they will also run on Linux (Kubuntu 9.04) and Wine 1.1.26. You need to update the registry to install the Access drivers. Under Windows, export from the Registry to *.reg files the registry entries - HKLM\Software\ODBC - HKLM\Software\Microsoft\Jet all subsidiary keys and values come with them. You can carefully edit these files to remove drivers and DSNs you don't need. Import these files using the registry editor in wine: wine regedit.exe. You need to bring across from Windows to Windows\System32 under wine: clbcatq.dll comres.dll expsrv.dll msjet40.dll mswstr10.dll msjter40.dll msjint40.dll msjtes40.dll msrd3x40.dll odbc32.dll odbccp32.dll odbcji32.dll odbcjt32.dll odbcad32.exe odbcint.dll odbctrac.dll vbajet32.dll Register this server in wine: wine regsvr32.exe msjtes40.dll No need to install MDAC. To use Windows ODBC drivers, you have to override Wine's odbccp32.dll and odbc32.dll with the native versions because the Wine versions are currently wired directly to Linux's unixodbc. This can be done by setting up the ODBC Data Source Administrator odbcad32.exe using winecfg: - Add the program to the Applications tab - then in the libraries tab, pick from 'New override for library' drop-down odbc32.dll and odbccp32.dll add them and edit them to be Native for Windows. Then if you do wine odbcad32.exe this brings up the ODBC Data Source Administrator window as in the Windows Control Panel If needed, set up (System) DSNs using this program. Bring across the programs and *.mdb database files from Windows. Using winecfg, you need to set up each program you want to run with overrides for odbc32.dll and odbccp32.dll (Applications and Libraries tabs) as above. The programs should then run as they did in Windows but perhaps a bit slower with: wine Odbc-prog.exe This has been tested twice on a clean .wine install. I cannot vouch for support of all API and SQL facilities however. I hope someone finds this useful. Barry Bird

5 4

IMM / IME work
by Aric Stewart 16 Jun '16

16 Jun '16

Hello all, I know this will only interest a small portion of you but thought i would give a quick update on the state of IMM32 since I have brought it to a major milestone. All the main patches are in which now separate IMM32 and IMEs. There is still more work to do but the major framework is in place. X11 XIM processing should be unchanged. However wine can now begin to load native windows IMEs as well. I have tested with windows ATOK20 (a popular Japanese IME) and successfully had text processing in a fully IME aware application. There are still clear issues to resolve in many aspects of this processing but we have forward progress. ImmInstallIME does not work yet, nor does switching keyboards. So to get the native IME to work you need to add this registry key. [System\\CurrentControlSet\\Control\\Keyboard Layouts\\<keyboard layout>] "Ime File"=<IME filename> so for example for ATOK20 in Japanese i used. [System\\CurrentControlSet\\Control\\Keyboard Layouts\\e0010411] "Ime File"="ATOK20W.IME" I would love to hear how well things work. I am sure using native IMEs will quickly show us many places where IMM32 needs to be improved. One issues I am going to investigate next is that sometimes non x11drv ime initialization, if occurring too early, causes x11drv to fail to create windows. I have not investigated with the latest changes to xim.c (which may already correct this problem) but if you see this problem this patch may help and i believe the IME_UpdateAssociation(NULL) is already unneeded. diff --git a/dlls/winex11.drv/xim.c b/dlls/winex11.drv/xim.c index d4df9f7..0c98136 100644 --- a/dlls/winex11.drv/xim.c +++ b/dlls/winex11.drv/xim.c @@ -475,7 +475,6 @@ static void X11DRV_OpenIM(Display *display, XPointer ptr, XP XUnregisterIMInstantiateCallback(display, NULL, NULL, NULL, X11DRV_OpenIM, wine_tsx11_unlock(); IME_XIMPresent(TRUE); - IME_UpdateAssociation(NULL); } thanks, -aric

5 7

Re: XInput [1/4]: Implement XInputGetState
by Stefan Dösinger 05 Jul '15

05 Jul '15

Am 09.07.2009 um 02:58 schrieb Michael Gruber: > This patch series implements parts of the XInput library. You can use > this to play games that support Xbox 360 Controllers. To be able to > use this you will need to have your Controller working on Linux via > the xpad kernel driver and the event interface. It will not work on > any OS other than Linux. It supports gamepads, should work with > guitars and wheels, but will not work with dance-mats. > <1-XInputGetState.patch> Is it possible to implement XInput on top of DirectInput? DirectInput already has some internal abstraction layers for differnet joystick APIs, and has basic support for OSX joysticks. Does DInput have enough capabilities to access all the Xbox controller's features? My gut says no, otherwise MS wouldn't have created a new lib. I guess its worth checking though.

5 6

Re: configure: Add -linotify when present, for non-Linux platforms.
by Alexandre Julliard 23 Aug '14

23 Aug '14

Gerald Pfeifer <gerald(a)pfeifer.com> writes: > A FreeBSD user ran into this when sys/inotify.h was present via an > extra package and Wine went ahead using that, only to fail at link- > time. We'd want to use kqueue instead on FreeBSD. -- Alexandre Julliard julliard(a)winehq.org

2 1

What happened to the Fedora packages?
by Neal Gompa 02 Aug '14

02 Aug '14

What happened to the Fedora packages? They have not been updated since 0.9.2!!!! Right now it is at 0.9.10!!! Nearly every other Linux distro supported has the up to date packages!!! And why does the Red Hat packages site not go to the SourceForge site as it does for SUSE packages and the others?? I have not really had the guts to ask until now, because I thought that maybe there was a slump, but now, its getting annoying!! And Fedora just released Fedora Core 5 yesterday!!! Please tell me new packages will be ready soon!!! Compiling WINE always crashes my computer, so I prefer to use the RPMs...

10 18

winspool: ERROR_INVALID_NAME in GetDefaultPrinter() ?
by Detlef Riekenberg 09 Oct '13

09 Oct '13

Hi. >From which configuration does the "ERROR_INVALID_NAME" came from, when calling GetDefaultPrinter(NULL, &size) and no Printer is installed? This Test is Present in the current "dlls/winspool/tests/info.c". MSDN told us, that we receive an "ERROR_FILE_NOT_FOUND", if no Printer is installed: http://msdn.microsoft.com/library/en-us/gdi/prntspol_0hma.asp I get the "ERROR_FILE_NOT_FOUND" on win98se, winme, w2k and win2003 in this Situation. -- By By ... ... Detlef

3 3

Re: Web based translation tool
by Michal Čihař 18 Jun '13

18 Jun '13

Hi Dne Mon, 12 Mar 2012 12:02:49 +0200 Yaron Shahrabani <sh.yaron(a)gmail.com> napsal(a): > The Wine project is thinking about using Weblate in order to translate > Wine, you can look at the following discussion and see for yourself. That sounds great! > Looking good although a litle immature, not sure about authentication as > wel... For authentication you can basically use anything what can be used in Django. Any specific concerns on this topic? > I'll talk to Michal Cihar and ask him for a roadmap for this project. Well I don't have any real roadmap right now. I urgently needed replacement for Pootle because we got hit by too many bugs there and did not manage to understand the code there to be able to fix them myself. So far I think it is in pretty good shape for day to day work (with today's 0.5 release fixing few annoying bugs). Of course there are some features missing, you can check what I plan for 1.0 release here: https://github.com/nijel/weblate/issues?state=open&milestone=1 I currently hope to have 1.0 in a month or so, but I really don't want to commit to any specific date. PS: I'm not subscribed to mailing list so please CC me on reply. -- Michal Čihař | http://cihar.com | http://blog.cihar.com

7 16

[PATCH v3 0/7] Add O_DENY* support for VFS and CIFS/NFS
by Pavel Shilovsky 04 Apr '13

04 Apr '13

This patchset adds support of O_DENY* flags for Linux fs layer. These flags can be used by any application that needs share reservations to organize a file access. VFS already has some sort of this capability - now it's done through flock/LOCK_MAND mechanis, but that approach is non-atomic. This patchset build new capabilities on top of the existing one but doesn't bring any changes into the flock call semantic. These flags can be used by NFS (built-in-kernel) and CIFS (Samba) servers and Wine applications through VFS (for local filesystems) or CIFS/NFS modules. This will help when e.g. Samba and NFS server share the same directory for Windows and Linux users or Wine applications use Samba/NFS share to access the same data from different clients. According to the previous discussions the most problematic question is how to prevent situations like DoS attacks where e.g /lib/liba.so file can be open with DENYREAD, or smth like this. That's why one extra flag O_DENYMAND is added. It indicates to underlying layer that an application want to use O_DENY* flags semantic. It allows us not affect native Linux applications (that don't use O_DENYMAND flag) - so, these flags (and the semantic of open syscall that they bring) are used only for those applications that really want it proccessed that way. So, we have four new flags: O_DENYREAD - to prevent other opens with read access, O_DENYWRITE - to prevent other opens with write access, O_DENYDELETE - to prevent delete operations (this flag is not implemented in VFS and NFS part and only suitable for CIFS module), O_DENYMAND - to switch on/off three flags above. Also, this version of the patchset fixes the problem of data races on newely created files: open with O_CREAT can return the -ETXTBSY error for a successfully created file if this files was locked with a deny lock by another task. The #1 patch adds flags to fcntl, the #2 patch implements VFS part. The patches #3, #4, #5 are related to CIFS-specific changes, #6 and #7 describe NFS and NFSD parts. The preliminary patch for Samba that replaces the existing use of flock/LOCK_MAND mechanism with O_DENY* flags: http://git.etersoft.ru/people/piastry/packages/?p=samba.git;a=commitdiff;h=… The future part of open man page patch: ----- O_DENYMAND - used to inforce a mandatory share reservation scheme of the file access. If this flag is passed, the open fails with -ETXTBSY in following cases: 1) if O_DENYREAD flag is specified and there is another open with O_DENYMAND flag and READ access to the file; 2) if O_DENYWRITE flag is specified and there is another open with O_DENYMAND flag and WRITE access to the file; 3) if READ access is requested and there is another open with O_DENYMAND and O_DENYREAD flags; 4) if WRITE access is requested and there is another open with O_DENYMAND and O_DENYWRITE flags. If O_DENYDELETE flag is specified and the open succeded, any further unlink operation will fail with -ETXTBSY untill this open is closed. Now this flag is processed by CIFS filesystems only. This mechanism is done through flocks. If O_DENYMAND is specified and flock syscall is processed after the open. The share modes are translated into flock according the following rules: 1) O_DENYMAND -> LOCK_MAND 2) !O_DENYREAD -> LOCK_READ 3) !O_DENYWRITE -> LOCK_WRITE ----- Pavel Shilovsky (7): fcntl: Introduce new O_DENY* open flags vfs: Add O_DENYREAD/WRITE flags support for open syscall CIFS: Add O_DENY* open flags support CIFS: Use NT_CREATE_ANDX command for forcemand mounts CIFS: Translate SHARING_VIOLATION to -ETXTBSY error code for SMB2 NFSv4: Add O_DENY* open flags support NFSD: Pass share reservations flags to VFS fs/cifs/cifsacl.c | 6 +- fs/cifs/cifsglob.h | 12 +++- fs/cifs/cifsproto.h | 9 +-- fs/cifs/cifssmb.c | 47 ++++++++-------- fs/cifs/dir.c | 14 +++-- fs/cifs/file.c | 18 ++++-- fs/cifs/inode.c | 11 ++-- fs/cifs/link.c | 10 ++-- fs/cifs/readdir.c | 2 +- fs/cifs/smb1ops.c | 15 ++--- fs/cifs/smb2file.c | 10 ++-- fs/cifs/smb2inode.c | 4 +- fs/cifs/smb2maperror.c | 2 +- fs/cifs/smb2ops.c | 10 ++-- fs/cifs/smb2pdu.c | 6 +- fs/cifs/smb2proto.h | 14 +++-- fs/fcntl.c | 5 +- fs/locks.c | 117 +++++++++++++++++++++++++++++++++++---- fs/namei.c | 44 ++++++++++++++- fs/nfs/nfs4xdr.c | 24 ++++++-- fs/nfsd/nfs4state.c | 46 ++++++++++++++- include/linux/fs.h | 6 ++ include/uapi/asm-generic/fcntl.h | 14 +++++ 23 files changed, 345 insertions(+), 101 deletions(-) -- 1.8.1.2

8 36

Re: [PATCH 2/2] winmm: MCI_BREAK_HWND handling now matches Wine's 1999 MCI_DefYieldProc.
by Alexandre Julliard 05 Mar '13

05 Mar '13

<Joerg-Cyril.Hoehle(a)t-systems.com> writes: > @@ -2063,9 +2063,10 @@ static DWORD MCI_Break(UINT wDevID, DWORD dwFlags, LPMCI_BREAK_PARMS lpParms) > switch (dwFlags & (MCI_BREAK_KEY | MCI_BREAK_OFF)) { > case MCI_BREAK_KEY: > wmd->lpfnYieldProc = MCI_DefYieldProc; > - wmd->dwYieldData = lpParms->nVirtKey; > - if (dwFlags & MCI_BREAK_HWND) > - FIXME("(%x) vkey %02X on window %p\n", dwFlags, lpParms->nVirtKey, lpParms->hwndBreak); > + /* Our MCI_DefYieldProc from 1999-04-11 expects this DWORD splitting, > + * yet tests on w2k-w7 prove wmd->dwYieldData = lpParms->nVirtKey; */ It doesn't make sense to add workarounds for Wine code, it should be fixed instead. -- Alexandre Julliard julliard(a)winehq.org

2 1

Use accented letters in their names.
by Tae Wong 04 Mar '13

04 Mar '13

You will want to use accented letters in their authors names. Caolan McNamara Ferenc Wagner Marko Nikolic Every once and a while the GNOME Live website is down with an error on Netscape called “The operation timed out when attempting to contact .” - This paragraph should have a website name. You want to be unbanned in Amaterasu Translations and the DOS ain’t dead forums.

2 2

Re: gdiplus: GdipDrawString should handle alignment in both directions same way.
by Vincent Povirk 28 Feb '13

28 Feb '13

> - if (format) halign = format->align; > - else halign = StringAlignmentNear; > - How are you handling the case where format is NULL?

1 1

GSoC 2013 guidelines.
by Tejas Nikumbh 28 Feb '13

28 Feb '13

Hi, I am Tejas Nikumbh, an undergrad at Indian Institute of Technology Bombay. I will be participating in GSoC this year. [GSoC 2013]. In order to imporve my chances for getting selected this year, I would like to start contributing early on to Open Source development via Wine. Here's a little background info : *Languages proficient in : *Java, Python, Javascript [+jQuery] , HTML[+HTML5 Canvas], CSS [+CSS3]. C++ should be on the list soon. *CS Theory :* Discrete Math, Probablity and Random Processes, Data Structures and Algorithms. *SVN : *Github, Mercurial . *Basic Machine learning [Regression algos and some classification algos]* Based on the above info, are there any projects which might suit me? Also, Picking up something reasonable quickly shouldn't be a problem. Thanks, -- Tejas Nikumbh, Third Year Undergraduate, Electrical Engineering Department, IIT Bombay.

1 0

Re: ieframe: Fixed QI tests on IE10, where IStdMarshalInfo is supported by WebBrowser object.
by Marvin 28 Feb '13

28 Feb '13

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=24550 Your paranoid android. === WNT4WSSP6 (32 bit webbrowser) === Timeout === W2KPROSP4 (32 bit webbrowser) === webbrowser.c:2607: Test failed: expected GetOverridesKeyPath webbrowser.c:2612: Test failed: expected Invoke_SETSECURELOCKICON webbrowser.c:2613: Test failed: expected Invoke_FILEDOWNLOAD webbrowser.c:3041: Test failed: doc_disp == NULL webbrowser: unhandled exception c0000005 at 00402F21

1 0

Re: msvcrt/tests: Mark win8 behaviour as broken (try 2)
by Marvin 27 Feb '13

27 Feb '13

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=24548 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Fix name order in the authors file.
by Tae Wong 27 Feb '13

27 Feb '13

You’ve found Kazuyuki Sato in inverted name order. Apostrophes on the O-section are not ignored. O'Callahan, Robert O'Neill, Damien O'Regan, Mike Oberhumer, Stefan Oberndorfer, Peter Ogburn, Walt Ognyanoff, Damyan Olsen, Henrik Olsen, Magnus Olson, Roger Olver, Neil Ortiz, Jose Manuel Ferrer Osmundsen, Knut St. It should be ignored so the section should be sorted as: Oberhumer, Stefan Oberndorfer, Peter O'Callahan, Robert Ogburn, Walt Ognyanoff, Damyan Olsen, Henrik Olsen, Magnus Olson, Roger Olver, Neil O'Neill, Damien O'Regan, Mike Ortiz, Jose Manuel Ferrer Osmundsen, Knut St.

1 0

Re: d3dx9: Avoid expensive computations
by Rico Schüller 27 Feb '13

27 Feb '13

On 25.02.2013 06:03, Nozomi Kodama wrote: > + out.u.m[2][i] = v.z / signed_det; > + out.u.m[3][i] = v.w / signed_det; > } > > *pout = out; While you are at it, you may fix the indentation of out*, "}", "*pout = out;" and "return pout;". > signed_det = (i % 2)? -det: det; Couldn't you just use something like "det = -det;" instead of the modulo? This should be a little bit faster. I did some small tests for speed with the following results. You may also avoid such a lot of variable assignments like *pout = out and you may use 4 vecs instead. This should save ~48 assignments and it should also improve the speed a bit more (~10%). Though, native is still 40% faster than that. With the change above it should look like: int i; D3DXVECTOR4 v, vec[4]; FLOAT det; ... for (i = 0; i < 4; i++) { vec[i].x = pm->u.m[i][0]; vec[i].y = pm->u.m[i][1]; vec[i].z = pm->u.m[i][2]; vec[i].w = pm->u.m[i][3]; } for (i = 0; i < 4; i++) { switch (i) { case 0: D3DXVec4Cross(&v, &vec[1], &vec[2], &vec[3]); break; case 1: D3DXVec4Cross(&v, &vec[0], &vec[2], &vec[3]); break; case 2: D3DXVec4Cross(&v, &vec[0], &vec[1], &vec[3]); break; case 3: D3DXVec4Cross(&v, &vec[0], &vec[1], &vec[2]); break; } pout->u.m[0][i] = v.x / det; pout->u.m[1][i] = v.y / det; pout->u.m[2][i] = v.z / det; pout->u.m[3][i] = v.w / det; det = -det; } return pout; Maybe we could reuse some calculations from the D3DXVec4Cross function ... Cheers Rico

5 7

Re: secur32: Report SecPkgContext_ConnectionInfo in bits, not bytes.
by Marvin 27 Feb '13

27 Feb '13

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=24544 Your paranoid android. === W2KPROSP4 (32 bit schannel) === schannel.c:762: Test failed: conn_info.dwHashStrength = 128 === WXPPROSP3 (32 bit schannel) === schannel.c:762: Test failed: conn_info.dwHashStrength = 128 === W2K3R2SESP2 (32 bit schannel) === schannel.c:762: Test failed: conn_info.dwHashStrength = 128

1 0

Re: mmdevapi: Prevent 64 bit overflow within a few days of audio device use.
by Michael Stefaniuc 27 Feb '13

27 Feb '13

Hello Joerg, On 02/27/2013 02:46 PM, Joerg-Cyril.Hoehle(a)t-systems.com wrote: > The idea to replace X * numerator / denominator > by X / den * mul + remainder from euclidian division > came from can you please use an inline function for that? > http://blog.airsource.co.uk/index.php/2010/03/15/quelle-heure-est-il/ thanks bye michael

1 0

Re: [PATCH 2/4] vbscript: Added color constants.
by Marvin 27 Feb '13

27 Feb '13

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=24538 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [1/3] wininet: Remember that headers were not received.
by Jacek Caban 27 Feb '13

27 Feb '13

Hi Ričardas, On 02/27/13 00:07, Ričardas Barkauskas wrote: > @@ -5708,6 +5708,7 @@ static INT HTTP_GetResponseHeaders(http_request_t *request, BOOL clear) > > heap_free(request->rawHeaders); > request->rawHeaders = heap_strdupW(szDefaultHeader); > + request->flags |= INTERNET_REQFLAG_NO_HEADERS; I don't think we want to maintain valid request->flags for that particular flag. Checking for request->rawHeaders should be enough. If we avoided setting rawHeaders here, we'd achieve the same behaviour. Jacek

1 0

Re: msvcrt/tests: Mark win8 behaviour as broken
by Piotr Caban 27 Feb '13

27 Feb '13

On 02/26/13 23:57, André Hentschel wrote: > diff --git a/dlls/msvcrt/tests/time.c b/dlls/msvcrt/tests/time.c > index 30d9fa7..3d00769 100644 > --- a/dlls/msvcrt/tests/time.c > +++ b/dlls/msvcrt/tests/time.c > @@ -181,8 +181,8 @@ static void test_gmtime(void) > err = p_gmtime32_s(&gmt_tm_s, &gmt); > - ok(gmt_tm_s.tm_year==-1 || broken(gmt_tm_s.tm_year==70 && gmt_tm->tm_sec<0), > - "tm_year = %d\n", gmt_tm_s.tm_year); > + ok(gmt_tm_s.tm_year==-1 || broken(gmt_tm_s.tm_year==70), > + "tm_year = %d, tm_sec = %d\n", gmt_tm_s.tm_year, gmt_tm->tm_sec); This test was checking wrong structure. It should use gmt_tm_s struct for testing seconds instead of gmt_tm.

1 0

Re: [1/3] wininet: Remember that headers were not received.
by Marvin 26 Feb '13

26 Feb '13

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=24535 Your paranoid android. === W7PROX64 (32 bit http) === http.c:3337: Test failed: HttpSendRequest failed: 12002

1 0

Re: gdi32: Fix the GdiGetCodePage() support ANSI_CHARSET font associated charset
by Akihiro Sagawa 26 Feb '13

26 Feb '13

On Sun, 24 Feb 2013 09:55:59 +0900, Byeongsik Jeon wrote: Could you consider adding "(try N)" at the end of subject line? The rule is documented here: http://wiki.winehq.org/SubmittingPatches#head-fea51bfdf6a29105a9a6fc6f5886a… > diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c > index b274b83..3a99d35 100644 > --- a/dlls/gdi32/font.c > +++ b/dlls/gdi32/font.c > @@ -590,11 +590,75 @@ HFONT WINAPI CreateFontW( INT height, INT width, INT esc, This part looks good. > diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c (snip) > +static void update_font_association_info(UINT current_ansi_codepage) > +{ > + static const WCHAR font_assoc_reg_keyW[] = {'S','y','s','t','e','m','\\', > + 'C','u','r','r','e','n','t','C','o','n','t','r','o','l','S','e','t','\\', > + 'C','o','n','t','r','o','l','\\','F','o','n','t','A','s','s','o','c','\0'}; It is nice introducing separate function for this work. However using WCHAR is a bit overwork, because ANSI APIs are still used in update_font_info() especially set_value_key(). Please follow the style of surrounding code. > @@ -3615,6 +3675,10 @@ static void update_font_info(void) > } > if (!done) > FIXME("there is no font defaults for codepages %u,%u\n", ansi_cp, oem_cp); > + > + /* update locale dependent font association info in registry */ > + if (strcmp(buf, cpbuf)) > + update_font_association_info(ansi_cp); > } > > static BOOL init_freetype(void) This condition should be rewritten. buf is not initialized when RegQueryValueExA is failed in line 3542. And strcmp(buf, cpbuf) has already done in line 3544. > diff --git a/dlls/gdi32/tests/font.c b/dlls/gdi32/tests/font.c > index 943f584..9d57806 100644 > --- a/dlls/gdi32/tests/font.c > +++ b/dlls/gdi32/tests/font.c Why are there no tests against OEM_CHARSET and SYMBOL_CHARSET? Regards, Akihiro Sagawa

2 2

Re: riched20: Remove redundant 'return;' statements.
by Dmitry Timoshkov 26 Feb '13

26 Feb '13

This patch is marked as 'Rejected', and I can't find an e-mail to wine-devel with an explanation why. -- Dmitry.

2 1

Re: comctl32/listview: fix mouse message sequences
by Nikolay Sivov 26 Feb '13

26 Feb '13

On 2/24/2013 01:33, Daniel Jelinski wrote: > @@ -10178,6 +10139,19 @@ static LRESULT LISTVIEW_LButtonDown(LISTVIEW_INFO *infoPtr, WORD wKey, INT x, IN > > if (infoPtr->dwLvExStyle & LVS_EX_ONECLICKACTIVATE) > if(lvHitTestInfo.iItem != -1) notify_itemactivate(infoPtr,&lvHitTestInfo); > + if (LISTVIEW_TrackMouse(infoPtr, pt)) > + { > + NMLISTVIEW nmlv; > + > + ZeroMemory(&nmlv, sizeof(nmlv)); > + nmlv.iItem = lvHitTestInfo.iItem; > + nmlv.ptAction = infoPtr->ptClickPos; > + > + notify_listview(infoPtr, LVN_BEGINDRAG, &nmlv); > + infoPtr->bDragging = TRUE; > + } > + else > + LISTVIEW_LButtonUp(infoPtr,0,x,y); This doesn't look very clean. I mean invoking *up handler from a *down one. Is this something that could be resolved with message loop like rbutton dragging was fixed?

2 3

Re: winetest: Add a link to the test summary page that specifies the test configuration details, the dll versions, etc.
by Alexandre Julliard 25 Feb '13

25 Feb '13

Francois Gouget <fgouget(a)codeweavers.com> writes: > Clarify two other link names to avoid confusion. > --- > > I did not test this patch so it should be verified carefully before > committing. I recently found myself annoyed that I could not go from a > page such as this one: > > http://test.winehq.org/data/209b58c64e6181c9bf448cb4f3130d3ba705c5d2/win7_n… > > to what I call that test's summary page: > > http://test.winehq.org/data/209b58c64e6181c9bf448cb4f3130d3ba705c5d2/win7_n… > > Of course 'summary' referred instead to the page with all the 'build > results' which I found very confusing. So I renamed it to just that > ('build results', not 'confusing' <g>). Then I renamed the 'index' link > to 'home' since it points to the winetest site's landing page (aka its > 'home' page). I'm not sure that's better. Particularly since that summary page is helpfully called "Main summary", it's not that confusing. The version info is obtained on other pages by clicking on the test tag, so we could potentially display the test tag on the individual test page too. -- Alexandre Julliard julliard(a)winehq.org

1 0

A new demangler test?
by Max TenEyck Woodbury 25 Feb '13

25 Feb '13

Would it be appropriate to add a test to the name demangler that: 1) Scans all '.dll' and '.spac' files for mangled names, and 2) Tries to decode those names. 3) Prints the mangled and decoded names and where they occur. Success would be that all names decode without the decoder blowing up or failing.

4 6

Re: [2/3] gdiplus: Round up when converting from 32-bit ARGB to PARGB.
by Vincent Povirk 25 Feb '13

25 Feb '13

Are you sure about this? Other tests have suggested that native is not this exact when dealing with color values, and different OS versions have produced slightly different results. Is there an application that relies on this, or is there a reason this is generally more correct behavior when premultiplying alpha?

1 1

Re: user32/tests: Test invalidating during SetWindowPos() (resend).
by Sergey Guralnik 25 Feb '13

25 Feb '13

On 2013-02-21 15:44, Sergey Guralnik wrote: > This patch marked as "Pending" without any comment at wine-devel. Is something wrong with it? (Maybe there should be more test cases?) -- Sergey

2 1

Re: [1/3] gdiplus: Add some tests for ARGB conversions.
by Nikolay Sivov 25 Feb '13

25 Feb '13

On 2/25/2013 09:29, Dmitry Timoshkov wrote: It seems your system time is an hour ahead. Probably old timezone info?

1 0

Why complain about X1R5G5B5?
by Francois Gouget 24 Feb '13

24 Feb '13

If I run Wine's conformance tests in a QEmu VM using Spice's QXL driver I get the following errors: d3d8:device device.c:1004: Test failed: Unexpected display mode returned for mode 0: 0x18 d3d9:device device.c:1366: Test failed: EnumAdapterModes(D3DFMT_X1R5G5B5) did not return D3DERR_INVALIDCALL (got 00000000)! In essence our tests say that the driver must not support X1R5G5B5. Why? Further notes: * If I then call Direct3D9::CheckDeviceType() for this format I get NOTAVAILABLE which is inconsistent. But then that's exactly what Microsoft's Standard VGA driver does for the R5G6B5 and X8R8G8B8 formats. http://www.winehq.org/pipermail/wine-patches/2013-February/122388.html * See the exact list of formats returned by IDirect3D8::EnumAdapterModes() http://newtestbot.winehq.org/JobDetails.pl?Key=243&log_103=1#k103 Test executable and source: https://bugs.freedesktop.org/attachment.cgi?id=75248 * See the list of formats that IDirect3D8::EnumAdapterModes() claims are supported, and how CheckDeviceType() refuses them. http://newtestbot.winehq.org/JobDetails.pl?Key=249&log_103=1#k103 Test executable and source: https://bugs.freedesktop.org/attachment.cgi?id=75260 * I think it's important to get Spice's QXL driver and Wine's conformance tests to play nice together. First they are both open-source so if one is wrong we should work to resolve that. But also the new WineTestBot is uses QEmu and the QXL driver looks like it will be _the_ paravirtualized(*) graphics driver there. (*) I.e. high performance and most used driver. * See also QXL bug 61227: https://bugs.freedesktop.org/show_bug.cgi?id=61227 -- Francois Gouget <fgouget(a)codeweavers.com>

3 4

Re: A new demangler test?
by Nikolay Sivov 24 Feb '13

24 Feb '13

On 2/24/2013 15:13, Eric Pouech wrote: > Le 24/02/2013 10:18, Nikolay Sivov a écrit : >> On 2/24/2013 10:07, Max TenEyck Woodbury wrote: >>> On 02/23/2013 02:54 AM, Eric Pouech wrote: >>>> Le 21/02/2013 14:33, Max TenEyck Woodbury a écrit : >>>>> Would it be appropriate to add a test to the name demangler that: >>>>> >>>>> 1) Scans all '.dll' and '.spec' files for mangled names, and >>>>> >>>>> 2) Tries to decode those names. >>>>> >>>>> 3) Prints the mangled and decoded names and where they occur. >>>>> >>>>> Success would be that all names decode without the decoder blowing up >>>>> or failing. >>>>> >>>>> >>>>> >>>> adding tests for demangler is good >>>> but you just need to have a set of mangled/unmangled strings for >>>> the test >>>> as core demangler in msvcrt, test should be added here... >>>> (but grabbing the mangled strings shall be left out of the test >>>> itself) >>>> A+ >>>> >>> One of the important aspects of name demangling is that it should work >>> on _all_ the names in the current system. The current test does >>> demangle a list of known strings, but that list was incomplete with >>> respect to all the features used in real names the last time I dug into >>> the details (which was none too recently). >>> >>> Scanning for all the real mangled names not only makes sure that the >>> demangler is working properly, it also assures that all the names used >>> are properly formed and decode to their intended values. >> I don't see a point to extend test data that much, it's enough to add >> some most common cases, >> and add more when something breaks because of it. Otherwise it >> doesn't deserve that much of attention imho. >> >> By the way, do you see something broken or it's just something you >> want to work on? >>> >>> > there are lots of broken stuff (in msvcrt), and demangler in winedump > is ante-deluvian (and doesn't share its code with msvcrt) > A+ Okay, don't know if winedump part is of any use, I mean it tries to provide a sane C name for decorated names, and usually such names are hand-crafted anyway later. Msvcrt being broken is important of course (at least for dbghelp).

1 0

Re: [0/13] Patch series description
by Saulius Krasuckas 23 Feb '13

23 Feb '13

* On Wed, 20 Feb 2013, André Hentschel wrote: > this patch series removes the broken SPARC support. > Reasons: > * It's broken, i wasted enough time already trying to fix that It's sad to hear. Can you be more verbose on brokenness, please? Eg. namely what Sparc platform were you using for development? > * Nobody cares - I sent an e-mail to everyone ever related to > Wine/SPARC based on addresses from our mailing lists - no answer in a > week (and cw also has no customers for it) You didn't catch this then: http://www.winehq.org/pipermail/wine-devel/2004-October/030355.html I care :) > * If someone _would_ care the right thing to do would be a clean SPARC > 64-bit port So, the old port was 32-bit or mixed or what? > * It really can confuse people (more than a ARM/ARM64 port xD) You mean random readers about host platforms Wine runs on;)? > * SPARC is dead on the Desktop :( > And last but not least: I had a bad experience with the Solaris > community regarding Wine/SPARC Oh. Is there any related public messages on the net? It would interesting to read and know those stories. S.

2 6

Re: gdiplus: Remove debug traces from GdipBitmapGetPixel and GdipBitmapSetPixel.
by Vincent Povirk 22 Feb '13

22 Feb '13

> alpha_blend_bmp_pixels() calls them while copying bitmap bits with an > alpha channel. This makes debug logs huge, hard to analyze, and completely > unmanagable in size. GDI+ should be fixed so it doesn't call these functions internally. It's just not an efficient way to render to bitmaps.

1 0

Re: gdi32: Fix the GdiGetCodePage() support ANSI_CHARSET font associated charset.
by Akihiro Sagawa 22 Feb '13

22 Feb '13

On Fri, 22 Feb 2013 11:21:20 +0900, Byeongsik Jeon wrote: > This patch fix a original topic of the wine-bugs #16325. Thanks for your work on this. > +static DWORD get_associated_charset_info(void) > +{ > + static DWORD associated_charset = -1; > + > + if (associated_charset == -1) > + { > + const char *assoc_charset_reg_key = > + "System\\CurrentControlSet\\Control\\FontAssoc\\Associated Charset"; Please use WCHARs and W version of APIs in gdi32/font.c, like: const WCHAR yesW[] = { 'Y', 'E', 'S', 0 }; and so on. > @@ -3615,6 +3615,33 @@ static void update_font_info(void) > } > if (!done) > FIXME("there is no font defaults for codepages %u,%u\n", ansi_cp, oem_cp); > + > + if (RegCreateKeyA( HKEY_LOCAL_MACHINE, > + "System\\CurrentControlSet\\Control\\FontAssoc\\Associated Charset", As far as I know, in SBCS locale, this key doesn't exist. Therefore it should delete in SBCS locale initialization and create the key in DBCS locale initialization. Regards, Akihiro Sagawa

1 0

newtestbot timings
by Joerg-Cyril.Hoehle＠t-systems.com 22 Feb '13

22 Feb '13

Hi, I've looked at some of newtestbot timing results. http://newtestbot.winehq.org/JobDetails.pl?Key=256 It's not too bad, but it's not superb either. >render.c:633: event after Stop.4 average 40.000ms deviation 0.06 from 100 samples That's exactly the expected 40ms, great! >render.c:618: Test failed: Wait iteration 40 gave 102 >render.c:633: event after Stop.1 average 10.925ms deviation 0.36 from 40 samples That's funny. I recently discovered that the Vista-w7 machines downgrade their event rate from 10 to 40.0ms exactly 2 seconds following IAC_Stop. Newtestbot's w8 results indicate that w8 continues sending 10ms events for 400ms after IAC_Stop, then ceases. Current wine*.drv immediately stop sending events after IAC_Stop, so Wine almost implemented w8 behaviour before it existed :-) >render.c:189: Returned periods: 10.1587 ms 3.0000 ms >render.c:633: event after Start.2 average 10.955ms deviation 0.16 from 100 samples >render.c:2200: Released 44800=100x448 -448 frames at 44100 worth 1006ms in 1087ms >render.c:2206: event average 10.878ms deviation 0.16 from 100 samples So the machines do not manage to deliver the expected 10.1587ms period. Well, 10.9 is not that far away. By comparison, a native box meets the average: < event after Start.4 average 10.159ms deviation 0.01 from 100 samples < Released 44800=100x448 -896 frames at 44100 worth 996ms in 1018ms < event average 10.161ms deviation 0.03 from 100 samples The exclusive mode test shows some degradation: >render.c:928: Testing exclusive mode at 44100 >render.c:948: Latency: 15.0113 ms >render.c:967: BufferSize 22528 estimated fragment 662 x 33 = 21846 >render.c:991: Clock Frequency 44100 >render.c:1060: Test failed: Position 5696 too far after stop 100ms -- 129ms 30% off, quite a lot. >render.c:1070: padding 0 past sleep #2 >render.c:1089: padding 0 position 22528 past stop #2 This is worrying. Padding 0 means that within 200ms, the driver emptied its whole half second buffer. That's not physical. Do you have PulseAudio running on the UNIX VM host? It's known from winealsa to eat lots of samples when starting up. By contrast, a native box yields: < padding 13308 past sleep #2 < padding 13251 position 9277 past stop #2 -- 210ms >render.c:1148: position 6208 past 100ms sleep #3 -- 141ms >render.c:1152: Test failed: Position 6208 too far after playing 100ms >render.c:1168: padding 16256 position 6272 past stop #3 -- 142ms >render.c:1173: Test failed: Position 6272 too far after stop 100ms Again, a native box yields values fully in range: < position 4757 past 100ms sleep #3 -- 108ms < padding 17706 position 4822 past stop #3 -- 109ms Interestingly, only exclusive mode is affected by huge deviations, upto 50%. Regarding shared mode, oddly w8 machines exhibit some failures. They are quite small (I think we should raise the margins to fit): render.c:1060: Test failed: Position 39424 too far after stop 100ms -- 111ms render.c:1060: Test failed: Position 43008 too far after stop 100ms -- 121ms render.c:1094: Test failed: Position 78848 too far after playing 200ms --223ms One machine stalled for 44 seconds: >render.c:633: event after Stop.2 average 444.809ms deviation 4294.73 from 100 samples That would have caused a test failure if it had happened at another location within the tests. Regards, Jörg Höhle

1 0

winealsa: Have the MIDI recorder wait in poll(), not snd_seq_event_input().
by Joerg-Cyril.Hoehle＠t-systems.com 22 Feb '13

22 Feb '13

Hi, Johannes Krol wrote: >Joerg, if you want the complete log I can send it to you by PM, it's >about 1.7M. I don't need that for now. What I need are the 119 missing bytes between 17.015:003b:trace:midi:modLongData dwBufferLength=125 ! 17.015:003b:trace:midi:modLongData F0 42 40 ... 00 00 00 Please copy&paste & adapt this code to produce a full hex dump: http://source.winehq.org/source/dlls/winmm/winmm.c#L1066 In particular, I want to know if there's some F7 amid that data (remember that your original patch precisely stopped at a F7...) and what the bytes following that F7 look like. >This could be interpreted to mean that Windows will add the 0xF7 >marker, if it was missing, when any other MIDI message is sent. That's a possible interpretation, however your log snippet shows no sign of sending another message. Do you have MIDI/ALSA debugging tools that can show you whether ALSA truly sends out the 125 bytes? That too would help identify the problem. Thank you, Jörg Höhle

2 6

Re: vbscript: Added support for a few more constants.
by Marvin 22 Feb '13

22 Feb '13

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=24500 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: comctl32/treeview: Protect TVM_GETITEM from invalid item pointers
by Alexandre Julliard 22 Feb '13

22 Feb '13

Nikolay Sivov <nsivov(a)codeweavers.com> writes: > @@ -2101,9 +2102,17 @@ TREEVIEW_GetItemT(const TREEVIEW_INFO *infoPtr, LPTVITEMEXW tvItem, BOOL isW) > { > if (!item) return FALSE; > > - TRACE("got item from different tree %p, called from %p\n", item->infoPtr, infoPtr); > - infoPtr = item->infoPtr; > - if (!TREEVIEW_ValidItem(infoPtr, item)) return FALSE; > + __TRY > + { > + infoPtr = item->infoPtr; > + TRACE("got item from different tree %p, called from %p\n", item->infoPtr, infoPtr); > + if (!TREEVIEW_ValidItem(infoPtr, item)) return FALSE; You can't return from a try block. Also have you verified that this is not just masking some other Wine bug? -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: server/advapi32: Report administrator ownership by default for registry objects (try 4).
by Alexandre Julliard 22 Feb '13

22 Feb '13

"Erich E. Hoover" <ehoover(a)mymail.mines.edu> writes: > + else > + { > + struct object_type *type = obj->ops->get_type( obj ); > + static const WCHAR key_type[] = {'K','e','y'}; > + const WCHAR *type_name = NULL; > + data_size_t type_len = 0; > + > + if (!type) goto done; > + type_name = get_object_name( &type->obj, &type_len ); > + release_object( type ); > + if (memcmp(type_name, key_type, type_len) != 0) goto done; > + } You clearly need to spend more time studying how the server works. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: gdi32: Fix the GdiGetCodePage() support ANSI_CHARSET font associated charset.
by Qian Hong 22 Feb '13

22 Feb '13

On Fri, Feb 22, 2013 at 10:21 AM, Byeongsik Jeon <bsjeon(a)hanmail.net> wrote: > This patch fix a original topic of the wine-bugs #16325. Thanks for the great work! I've been waiting for this fix for a really long time :) Out basic Wine test VM does not include Chinese/Japanese Windows, so I submit Byeongsik's patch to the testbot again with Chinese/Japanese Windows included: https://testbot.winehq.org/JobDetails.pl?Key=24496 All tests related to this patch passed (Those failed tests are not related to this patch) I also manually test the patch on Windows with Korean locale and the test case passed as well. Hopes that helps :) -- Regards, Qian Hong - http://www.winehq.org

1 0

Re: [12/13] winedump: Remove SPARC support
by André Hentschel 21 Feb '13

21 Feb '13

Am 21.02.2013 20:55, schrieb Eric Pouech: > Le 20/02/2013 21:11, André Hentschel a écrit : >> - case IMAGE_FILE_MACHINE_SPARC: return "SPARC"; >> >> > Actually this one should be kept, so that dumping such would print the relevant information. > A+ Thought about that for a second, too. But winedump only parses PE dlls, not the winelib ones and there are no SPARC PE dlls out there (Windows was never ported to it). -- Best Regards, André Hentschel

1 0

prelink --reloc-only versus ld -Ttext-segment
by Tijl Coosemans 21 Feb '13

21 Feb '13

Hi, I'm looking into supporting winegcc -Wl,--image-base on FreeBSD. On Linux prelink --reloc-only is used for that but rather than porting prelink it seems easier to pass -Wl,-Ttext-segment to the compiler instead. Is there anything obviously wrong with that? Winedbg shows kernel32 and ntdll are loaded at their respective addresses: ELF 7b800000-7ba41000 Dwarf kernel32<elf> \-PE 7b820000-7ba41000 \ kernel32 ELF 7bc00000-7bccd000 Dwarf ntdll<elf> \-PE 7bc10000-7bccd000 \ ntdll The output of readelf -l ntdll.dll.so looks good too: Elf file type is DYN (Shared object file) Entry point 0x7bc20200 There are 5 program headers, starting at offset 52 Program Headers: Type Offset VirtAddr PhysAddr FileSiz MemSiz Flg Align LOAD 0x000000 0x7bc00000 0x7bc00000 0xafce4 0xafce4 R E 0x1000 LOAD 0x0afce4 0x7bcb0ce4 0x7bcb0ce4 0x0941c 0x1c2b4 RW 0x1000 DYNAMIC 0x0b0000 0x7bcb1000 0x7bcb1000 0x000e8 0x000e8 RW 0x4 GNU_EH_FRAME 0x0960bc 0x7bc960bc 0x7bc960bc 0x04144 0x04144 R 0x4 GNU_STACK 0x000000 0x00000000 0x00000000 0x00000 0x00000 RW 0x4

1 0

prelink --reloc-only versus ld -Ttext-segment
by Tijl Coosemans 21 Feb '13

21 Feb '13

Hi, I'm looking into supporting winegcc -Wl,--image-base on FreeBSD. On Linux prelink --reloc-only is used for that but rather than porting prelink it seems easier to pass -Wl,-Ttext-segment to the compiler instead. Is there anything obviously wrong with that? Winedbg shows kernel32 and ntdll are loaded at their respective addresses: ELF 7b800000-7ba41000 Dwarf kernel32<elf> \-PE 7b820000-7ba41000 \ kernel32 ELF 7bc00000-7bccd000 Dwarf ntdll<elf> \-PE 7bc10000-7bccd000 \ ntdll The output of readelf -l ntdll.dll.so looks good too: Elf file type is DYN (Shared object file) Entry point 0x7bc20200 There are 5 program headers, starting at offset 52 Program Headers: Type Offset VirtAddr PhysAddr FileSiz MemSiz Flg Align LOAD 0x000000 0x7bc00000 0x7bc00000 0xafce4 0xafce4 R E 0x1000 LOAD 0x0afce4 0x7bcb0ce4 0x7bcb0ce4 0x0941c 0x1c2b4 RW 0x1000 DYNAMIC 0x0b0000 0x7bcb1000 0x7bcb1000 0x000e8 0x000e8 RW 0x4 GNU_EH_FRAME 0x0960bc 0x7bc960bc 0x7bc960bc 0x04144 0x04144 R 0x4 GNU_STACK 0x000000 0x00000000 0x00000000 0x00000 0x00000 RW 0x4

1 0

Re: webservices: Add new dll
by Hans Leidekker 21 Feb '13

21 Feb '13

On Thu, 2013-02-21 at 18:47 +1100, Alistair Leslie-Hughes wrote: > +#include "wine/debug.h" > + > +WINE_DEFAULT_DEBUG_CHANNEL(wincodecs); That channel is already taken :)

1 0

Re: comctl32/listview: fix selection in ownerdata listview
by Nikolay Sivov 21 Feb '13

21 Feb '13

On 2/21/2013 02:19, Daniel Jelinski wrote: > /* disable per item notifications on LVS_OWNERDATA style > FIXME: single LVN_ODSTATECHANGED should be used */ > bOldChange = infoPtr->bDoChangeNotify; > - if (infoPtr->dwStyle & LVS_OWNERDATA) infoPtr->bDoChangeNotify = FALSE; > + /*if (infoPtr->dwStyle & LVS_OWNERDATA) infoPtr->bDoChangeNotify = FALSE;*/ What's a problem in fixing this properly? This change could break something potentially, it was disabled for a purpose of fixing some real application as I remember.

2 2

Re: Implement pinned memory for ATI where buffer regions are slow
by Henri Verbeet 20 Feb '13

20 Feb '13

On 17 February 2013 07:17, Stanislaw Halik <sthalik(a)misaki.pl> wrote: > This one should be better. Sorry about indentation. Much, much FPS, and > supposedly no loss of functionality. > I didn't really review this, but I think any extension that allows "... including program or even system termination." from regular user space is terrible. (And yes, your formatting is also messed up.) I think Stefan has said this before, but what really needs to happen is that you (or someone else) figure out where the actual issue with ARB_map_buffer_range is, and either get that fixed in Wine, or file a bug for fglrx.

3 5

Failing tests in the Japanese and Hebrew languages
by Francois Gouget 20 Feb '13

20 Feb '13

I have noticed some test failures that only happen when I set the locale and display language to Japanese and/or Hebrew in Windows 7. Let me know if I should create bugs for these. * comctl32:listview (Hebrew and Japanese) listview.c:3822: Test failed: Expected 96, got 112 listview.c:4751: Test failed: Expected 96, got 112 Both correspond to the use of GetDeviceCaps(LOGPIXELSX) so something must be wrong there: todo_wine expect(((GetDeviceCaps(hdc, LOGPIXELSX) + 15) / 16) * 16, rect.right); todo_wine expect(((GetDeviceCaps(hdc, LOGPIXELSX) + 15) / 16) * 16, ret); * riched20:editor (Hebrew only) editor.c:586: Test failed: EM_POSFROMCHAR reports x=5, expected 8 editor.c:609: Test failed: EM_POSFROMCHAR reports x=5, expected 8 editor.c:640: Test failed: pt.x = 117 All three are related to testing positions past the end of text. Here the fact that Hebrew is RTL probably breaks the test. * rpcrt4:rpc (Japanese only) rpc.c:171: Test failed: DceErrorInqTextA(deviation...) Based on the comment it does not seem surprising that this would be language-specific: /* One for which FormatMessage should succeed but * DceErrorInqText should "fail" * 3814 is generally quite a long message */ * shell32:progman_dde (Hebrew and Japanese) progman_dde.c:377: Test failed: Window "?????" was not created in 45 seconds - assumed failure. [S_G:11] Based on the 'S_G:11' this would correspond to this line: ShowGroupTest(instance, hConv, "[ShowGroup(Startup,0)]", DMLERR_NO_ERROR, "Startup", StartupTitle, TRUE, DDE_TEST_SHOWGROUP|testnum++); * shlwapi:ordinal (Hebrew only) ordinal.c:1721: Test failed: expected (ùáú 16 ôáøåàø 2013), got (??? 16 ?????? 2013) ordinal.c:1729: Test failed: expected (ùáú 16 ôáøåàø 2013), got (??? 16 ?????? 2013) ordinal.c:1743: Test failed: expected (ùáú 16 ôáøåàø 2013) got (??? 16 ?????? 2013) for date part ordinal.c:1757: Test failed: expected (ùáú 16 ôáøåàø 2013) got (??? 16 ?????? 2013) for date part ordinal.c:1929: Test failed: expected (L"\05e9\05d1\05ea 16 \05e4\05d1\05e8\05d5\05d0\05e8 2013") got (L"\200f\05e9\05d1\05ea \200f16 \200f\05e4\05d1\05e8\05d5\05d0\05e8 \200f") for date part ordinal.c:1955: Test failed: expected (L"\05e9\05d1\05ea 16 \05e4\05d1\05e8\05d5\05d0\05e8 2013") got (L"\200f\05e9\05d1\05ea \200f16 \200f\05e4\05d1\05e8\05d5\05d0\05e8 \200f") for date part Note that while the above test was run with a Hebrew locale and display language, I get similar errors when running the test in the Japanese locale with the French display language: ordinal.c:1721: Test failed: expected (2013N216ú), got (2013?2?16?) ordinal.c:1729: Test failed: expected (2013N216ú), got (2013?2?16?) ordinal.c:1743: Test failed: expected (2013N216ú) got (2013?2?16?, 8) for date part ordinal.c:1757: Test failed: expected (2013N216ú) got (2013?2?16?, 8) for date part -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ E-Voting: It's not the people who vote that count. It's the people who count the votes.

3 6

Re: advapi32: Report administrator ownership by default for registry objects (try 3).
by Alexandre Julliard 19 Feb '13

19 Feb '13

"Erich E. Hoover" <ehoover(a)mymail.mines.edu> writes: > The attached patch fixes an issue where the Opera installer expects > registry keys to return ownership information (Bug #32904), concluding > the cleanup of [Get|Set]NamedSecurityInfo. This patch addresses the > issue by ensuring that NULL owner and group information is never > returned for registry keys and instead returns the Administrators SID > (S-1-5-32-544) when this information is unavailable, which is > sufficient to keep the Opera installer from encountering an error. > The included test shows that this behavior is the case for system keys > on Windows, which returns the Administrators SID for system registry > keys (such as HKLM\Software). Surely this is not specific to GetNamedSecurityInfo. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [14/14] gdiplus: Reimplement GdipDrawPath.
by Vincent Povirk 19 Feb '13

19 Feb '13

> Please don't apply this [14/14] patch, it causes a test failure. I'm sending > it to show how this is going to look like eventually. Yeah, GdipWidenPath isn't good enough for this yet. But I agree this is the way to go, and the rest of the series looks good.

2 2

Re: [PATCH 5/8] vbscript: Added RegExp.Execute tests
by Marvin 19 Feb '13

19 Feb '13

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=24454 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

2 1

Re: [PATCH 7/8] vbscript: Added tests for invalid parameters in regular expression functions
by Marvin 19 Feb '13

19 Feb '13

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=24455 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

2 1

Re: [PATCH 5/8] vbscript: Added RegExp.Execute tests
by Marvin 19 Feb '13

19 Feb '13

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=24454 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

IE 6 related urlmon:protocol failures
by Francois Gouget 18 Feb '13

18 Feb '13

I have investigated the following test failure on the new WineTestBot: urlmon: protocol.c:1051: Test failed: hrResult = 800401e4, expected: 00000000 protocol.c:1056: Test failed: dwError == ERROR_SUCCESS protocol.c:613: Test failed: expected ReportData The results from test.winehq.org show that they only happen on systems that have urlmon.dll version 6.0, which I guess corresponds to Internet Explorer 6.0. More specifically: * The test detects systems that have Internet Explorer 5.5 or older, based on missing functions, declares too old and skips the tests: 5.0.3700.6705 wtb-w2kassp4 5.0.3888.1400 wtb-w2kprosp4 5.50.4807.2300 fg-win2000-ie55 * The test runs and fails on these systems: 6.0.2800.1649 fg-win2000-up1 6.0.2800.1649 fg-win2000-uplast 6.0.2900.3562 fg-winxp-2sp 6.0.2900.3698 wtb-wxpmce 6.0.2900.6049 wtb-wxphmnlsp3 * The test runs and succeeds on systems that have urlmon 7.0 or greater: 7.0.6000.16850 fg-winxp-2spie7 7.0.6000.16850 fg-winxp-3sp 8.0.6001.18806 fg-winxp-3spie8 8.0.6001.19100 fg-winxp-last So I will be applying the latest updates to the new TestBot's Windows XP VM and that should solve the issue in that case. However I think the test should still not fail. So I'm looking for a volunteer to either get the test to succeed with urlmon 6.0, or find a way to detect it and skip these three tests. -- Francois Gouget <fgouget(a)codeweavers.com>

2 4

Re: [PATCH 1/2] wininet: Address string should never be converted to UNICODE in status notifications
by Marvin 18 Feb '13

18 Feb '13

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=24441 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

2 1

Re: msi: Don't ignore failure to remove an existing product.
by Alexandre Julliard 18 Feb '13

18 Feb '13

Hans Leidekker <hans(a)codeweavers.com> writes: > --- > dlls/msi/action.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) It fails here: ../../../tools/runtest -q -P wine -M msi.dll -T ../../.. -p msi_test.exe.so action.c && touch action.ok action.c:6397: Test failed: Expected ERROR_SUCCESS, got 2 action.c:6400: Test failed: Expected ERROR_SUCCESS, got 2 make[1]: *** [action.ok] Error 2 -- Alexandre Julliard julliard(a)winehq.org

1 0

winmm: More compatible waveIn/Out[Un]Prepare WHDR_* flag handling.
by Joerg-Cyril.Hoehle＠t-systems.com 18 Feb '13

18 Feb '13

Hi, One thing to note is a slight change when Prepare is called twice in a row. When the ACM is used, the former Wine code would prepare the ACM again, leaking the memory of the previous preparation. Now that code is by-passed as the header is prepared exactly once, the first time. What if the app changes dwBufferLength in between? We don't know what exactly native does. My tests only show that *some* code is by-passed, because when WHDR_PREPARE is found already set, the other flags are not repeatedly cleared. All calls return NOERROR. The 16 bit code has a comment http://source.winehq.org/source/dlls/mmsystem.dll16/message16.c#L430 > /* dwBufferLength can be reduced between prepare & write */ > if (... && wh32->dwBufferLength < wh16->dwBufferLength) { I've checked neither - what native does in that case when the ACM is involved, nor - how Wine drives the ACM in that case. It's reasonable to assume that only dwBufferLength at the time of waveOutWrite should be relevant. Let's hope that dwBufferLength from the time of Prepare doesn't matter in Wine's 32 bit ACM. Regards, Jörg Höhle

1 0

Re: [PATCH 1/2] wined3d: Make (wined3d_)surface_depth_blt_fbo handle locations other than SFLAG_INTEXTURE.
by Henri Verbeet 18 Feb '13

18 Feb '13

On 17 February 2013 18:10, Matteo Bruni <mbruni(a)codeweavers.com> wrote: > @@ -6168,18 +6172,10 @@ HRESULT surface_load_location(struct wined3d_surface *surface, DWORD location, c > > if (surface->resource.usage & WINED3DUSAGE_DEPTHSTENCIL) > { > - if (location == SFLAG_INTEXTURE) > - { > - struct wined3d_context *context = context_acquire(device, NULL); > - surface_load_ds_location(surface, context, location); > - context_release(context); > - return WINED3D_OK; > - } > - else > - { > - FIXME("Unimplemented location %s for depth/stencil buffers.\n", debug_surflocation(location)); > - return WINED3DERR_INVALIDCALL; > - } > + struct wined3d_context *context = context_acquire(device, NULL); > + surface_load_ds_location(surface, context, location); > + context_release(context); > + return WINED3D_OK; > } This is probably wrong (and the original code is probably questionable too), surface_load_ds_location() is only really meant for onscreen <-> offscreen blits. The original code is meant for the case where AlwaysOffscreen is disabled and a depth stencil is used as texture. The implication there is that the only possible locations are SFLAG_INTEXTURE and SFLAG_INDRAWABLE, it doesn't really work with multisampling. Fortunately you can't directly texture from multisampled depth stencil surfaces either. In principle we could make surface_load_ds_location() the generic function for depth / stencil location transfers, but currently it isn't. If you're hitting it with the RESZ code it probably only works by accident.

2 2

Re: Wine release 1.5.24
by Gennady Telegin 17 Feb '13

17 Feb '13

Hi, I want to unsubscribe from this list, how to do it? Best, Gennady Telegin On Sat, Feb 16, 2013 at 12:30 AM, Alexandre Julliard <julliard(a)winehq.org>wrote: > The Wine development release 1.5.24 is now available. > > What's new in this release (see below for details): > - Keyboard and mouse wheel support in the Mac driver. > - Regular expression support in VB Script. > - Many RichEdit code cleanups. > - Various bug fixes. > > The source is available from the following locations: > > http://prdownloads.sourceforge.net/wine/wine-1.5.24.tar.bz2 > http://mirrors.ibiblio.org/wine/source/1.5/wine-1.5.24.tar.bz2 > > Binary packages for various distributions will be available from: > > http://www.winehq.org/download > > You will find documentation on http://www.winehq.org/documentation > > You can also get the current source directly from the git > repository. Check http://www.winehq.org/git for details. > > Wine is available thanks to the work of many people. See the file > AUTHORS in the distribution for the complete list. > > ---------------------------------------------------------------- > > Bugs fixed in 1.5.24 (total 38): > > 6512 power-tab-editor freezes at end of a song > 8598 msvcrt file IO functions do not handle unicode properly in text > mode > 12908 Battle Zone I does not start. > 16685 redraw problem in AIM_675 > 17380 CListCtrl: wrong icon spacing > 17762 Citavi: Application is very slow > 17763 Citavi: mouse doesn't catch links > 18079 AutoCorect: does not properly display custom checkboxes made by > Alcinoe > 20294 sigma photo pro crashes in X11DRV_DIB_DeleteDIBSection > 24089 EverQuest: Escape to Norrath: login screen is blank without > native ie6 > 24315 Verizon Media Manager crashes on exit (VZMediaagent.exe > 24361 Crashday: game is unusably slow during gameplay > 24474 Simpsons Hit & Run sound bug > 24554 Black screen in Everquest 2 (lighting issue?) > 25576 Microsoft Flight Simulator X: Demo/Deluxe Edition, menu blank > without native ie7 > 25584 Racedriver:GRID sound hardware acceleration not working > 25958 DreamStation 1 free music tracker visually freezes under some > conditions > 27011 Lucent Heart: patcher window is blank > 27905 HUNTED The Demon Forge: Sound does not work in the game (in > movies this works) > 29718 IE4 setup wants wininet.dll.LoadUrlCacheContent > 29873 Guild Wars: Water graphic is missing > 30008 Resource Hacker logo on about window has black background > 30183 Fchart installation crashes > 30246 EA Origin:Crashes when using openssl 1.0.1 > 30839 BSTR cache corrupts most of cached BSTR entries > 31308 Remote Tools for Visual Studio 2012 RC installer for Windows on > ARM (WoA) crashes because TPIDRURW (user TLS register) needs to be set to > TEB address > 31706 Sacred Underworld: Some models glow like a Christmas tree > 31977 YoudaoDict crash at start > 32520 EVE Online and other games want d3d11.dll.D3D11CreateDevice > 32669 Ghost Master: invisible mouse pointer unless 'Enhanced Cursor' > option selected > 32808 installer of PPTV: needs unimplemented > atl100.dll.AtlComModuleRegisterClassObjects > 32818 Adrenalin Extreme Show: launcher.exe has repainting issues > 32842 TurboTax 2012 needs shlwapi.dll IsInternetESCEnabled stub > 32858 Crash dialog Details button Russian text doesn't fit > 32862 Microsoft Expression Design 4 (Free Version) needs > WindowsCodecsExt.dll (Microsoft Windows Codecs Extended Library) > 32882 Grand Theft Auto IV doesn't start, aborts with a GLX error. > 32909 QQDownload 3.9 needs unimplemented msvcr80.dll._wstat32i64 > 32929 Microsoft Expression Design 4 (Free Version) needs > windowscodecsext.dll.WICCreateColorTransform_Proxy > > ---------------------------------------------------------------- > > Changes since 1.5.23: > > Akihiro Sagawa (1): > gdi32: Ensure a fixed-pitch full-width character has double advance > of a half-width character. > > Alexander Morozov (1): > ole32: Avoid a deadlock when a being loaded DLL calls > CoRegisterClassObject from its DLL_PROCESS_ATTACH handler. > > Alexandre Julliard (28): > winemac: Don't move off-screen windows to a random position. > gdi32: Return the correct module handle for the initial display > driver load. > explorer: Retrieve the graphics driver module from gdi32. > explorer: Return a simple boolean instead of a window in the > wine_create_desktop entry point. > wintab32: Retrieve the graphics driver module from gdi32. > imm32: Retrieve the graphics driver module from gdi32. > winex11: Ignore color key and exposures when using the null surface. > user32: Avoid releasing a potentially null pointer. > server: Also exclude the top-level client rectangle for windows that > have a pixel format. > server: Return the window paint flags in the get_visible_region > request. > user32: Don't paint to the surface if the window has a pixel format, > regardless of the state of the top parent. > wininet: Don't reset the available size after the async has been > queued. > user32: Prefer loading color cursors in LoadImage. > ntdll: Add a helper function to check that a fault address lies in a > known virtual memory view. > ntdll: Check for invalid %gs value in 32-bit code. > winex11: Preserve the client window when changing the window visual. > server: Don't support debug registers in ptrace if sys/user.h is > missing. > loader: Define our own auxv structure. > loader: Rename the elf_hash function to avoid conflicts. > odbccu32: Remove an unnecessary source file. > twain_32: Remove an unnecessary DllMain entry point. > cryptnet: Remove an unnecessary DllMain entry point. > dpnhpast: Remove an unnecessary DllMain entry point. > iphlpapi: Remove an unnecessary DllMain entry point. > ktmw32: Remove an unnecessary DllMain entry point. > sensapi: Remove an unnecessary DllMain entry point. > usp10: Remove an unnecessary DllMain entry point. > wtsapi32: Remove an unnecessary DllMain entry point. > > Alistair Leslie-Hughes (1): > fusion: Return correct error for null public token. > > Andrew Talbot (8): > jscript: Avoid signed-unsigned integer comparisons. > mapi32: Avoid signed-unsigned integer comparisons. > kernel32: Avoid signed-unsigned integer comparisons. > mscoree: Avoid signed-unsigned integer comparisons. > mmdevapi: Avoid signed-unsigned integer comparisons. > mlang: Avoid signed-unsigned integer comparisons. > msdaps: Avoid signed-unsigned integer comparisons. > msctf: Avoid signed-unsigned integer comparisons. > > André Hentschel (4): > ntdll: Set TPIDRURW to TEB on thread init for ARM. > make_xftmpl: Avoid comparing a char with EOF. > winedbg: Add PState to cpu_register_map for ARM64. > ntdll: Implement RtlUnwind for ARM. > > Aric Stewart (6): > usp10: Fix typo with Script_Khmer_Numeric. > usp10: Split the core Itemize function out to be able to support > differences between the normal and OpenType versions. > usp10: Update diacritical script handling for non OpenType > Itemization. > usp10: Free default_language items when freeing script cache. > usp10: Fix memory leak when a font has 0 languages, scripts or > features. > usp10: Correct error with ScriptLayout log2vis. > > Austin English (4): > windowscodecsext: Add a stub dll. > shlwapi: Add a stub for IsInternetESCEnabled. > d3d11: Add a stub for D3D11CreateDevice. > wininet: Add a stub for LoadUrlCacheContent. > > Daniel Jelinski (6): > comctl32/listview: Correct icon spacing calculation when set to 0 or > -1. > comctl32/listview: Always use large icon size when calculating icon > spacing. > comctl32/listview: Do not touch icon spacing if set explicitly. > user32: Fix CopyImage implementation when changing cursor to icon. > comctl32/listview: Better handling of custom colors in report view. > comctl32/listview: Fix LVM_SETICONSPACING on 64bit machines. > > Detlef Riekenberg (2): > advapi32/tests: Skip tests on non-English systems. > kernel32/tests: Skip tests on non-English systems. > > Dmitry Timoshkov (14): > wbemprox: Free name in create_signature (Coverity). > comctl32: Test appropriate tool window for active state. > gdi32: Add support for 16-bit QUERYESCSUPPORT queries. > gdi32: Add a bunch of tests for CreateDC/ResetDC with a custom > DEVMODE. > winspool.drv: Turn macros into helper functions. > wineps.drv: Add DEVMODE dumping to make debugging device state > easier. > winspool.drv: There is no need to speculate about wineps.drv default > DEVMODE. > gdi32: Make sure that actual clipping region is updated for a EMF DC. > gdi32: Add more EMF clipping tests. > gdi32: OffsetClipRgn should update actual clipping region for a EMF > DC. > gdi32: IntersectClipRect should update actual clipping region for a > EMF DC. > gdi32: ExcludeClipRect should update actual clipping region for a > EMF DC. > gdi32: SelectClipPath should update actual clipping region for a EMF > DC. > kernel32: There is no need for lock/unlock heap in GlobalAlloc. > > Erich Hoover (2): > advapi32: Support registry objects in GetNamedSecurityInfo. > advapi32: Support registry objects in SetNamedSecurityInfo. > > Francois Gouget (5): > kernel32/tests: Reduce the size of the console buffer to speed up > the tests. > rpcrt4/tests: Trace the GUID when it is wrong. > d3d11: Make debug_d3d_driver_type() static. > ws2_32/tests: Fix the GetAcceptExSockaddrs() test. > winetest: Trace the time it takes to run each test. > > Frédéric Delanoy (1): > msi: Fix memory leak (coverity). > > Hans Leidekker (12): > windowscodecs: Implement IWICColorContext::InitializeFromFilename. > windowscodecs: Add a stub IWICColorTransform implementation. > windowscodecs: Convert the source bitmap to the requested pixel > format in IWICColorTransform::Initialize. > windowscodecs: Implement > IWICComponentFactory::CreateDecoderFromFileHandle. > wbemprox: Trace the qualifier value in IWbemClassObject::GetNames. > windowscodecsext: Implement WICCreateColorTransform_Proxy and > IWICColorTransform_Initialize_Proxy. > windowscodecs: Implement IWICBitmapFrameDecode::GetColorContexts for > TIFF images. > msiexec: Add /unregserver support. > msiexec: Convert DoRegServer to Unicode. > wininet/tests: Fix a test failure. > msi/tests: Move a couple of tests from install.c to msi.c. > msi: Don't remove products if msidbUpgradeAttributesOnlyDetect is > set. > > Henri Verbeet (2): > wined3d: Add WINED3DFMT_INST to the format table. > configure: Include Xlib.h before Xrandr.h. > > Huw Davies (23): > riched20: Actually pass the ANSI string to the break proc. > riched20: Pass the character ptr and string length directly to the > break proc helper. > riched20: Pass a character ptr and length to SplitParagraph. > riched20: Pass character ptrs to the whitespace finding functions. > riched20: Move the text to the paragraph level. > riched20: Move SplitRun to wrap.c and make it static. > riched20: Move the split point calculation to wrap.c. > riched20: Move run extent calculation to wrap.c. > riched20: The run width is always passed so we don't need to > calculate it again. > riched20: Pass the run to the text drawing routine. > riched20: Move underline pen creation to a helper function. > riched20: Move y offset calculation to a helper function. > riched20: Move text color determination to a helper function. > riched20: Move the white space painting operations to a common > function. > riched20: Remove the special case tab handling now that we handle > that in draw_space. > riched20: Make it possible to perform point -> char conversion while > holding a context. > riched20: The string len parameter is no longer necessary. > riched20: Use the point from char helpers to calculate the selection > rectangle. > riched20: Simplify selection painting by drawing a selected run > twice with appropriate clipping. > riched20: Move underline drawing to a common function. > riched20: Move the range clamping to the helper now that the range > isn't used later on. > riched20: Move the password string hack to just before it's used. > riched20: Use ME_PointFromChar to calculate the caret position. > > Jacek Caban (13): > atl100: Added AtlComModuleRegisterClassObjects implementation (based > on AtlModuleRegisterClassObjects). > oleaut32: COM clean up of ITypeLib2 interface implementation. > mshtml: Use VT_ERROR as return type in DispCallFunc call. > oleaut32: Return error if VT_HRESULT is used as return type in > DispCallFunc. > urlmon: Clean up handling tested URLs. > urlmon: Fixed QueryInfo tests during BINDSTATUS_PROXYDETECTING > notification. > urlmon: Skip reported_url tests when binding to object. > urlmon: Fixed tests on IEs that report BINDSTATUS_DECODING. > oleaut32: Don't free a string in SysFreeString if it's already in > cache. > oleaut32: Added some SysFreeString tests. > urlmon: Allow available_bytes to be set before protocol_continue > call. > wininet: Set available bytes in InternetQueryDataAvailable even if > it ends up in async call. > vbscript: Added support for a few more constants. > > Jactry Zeng (4): > msvcr100: Added _wstat32i64 implementation. > msvcr100: Added _stat32i64 implementation. > msvcr80: Sync _wstat32i64 and _stat32i64 implementation. > msvcr90: Sync _wstat32i64 and _stat32i64 implementation. > > Jeff Latimer (2): > wmvcore: Add stub implementation for WMCreateEditor. > wmvcore: Fix a typo. > > Juan Lang (2): > iphlpapi: Fix HRESULT/win32 error code confusion. > cryptui: Use add_usage_to_oid correctly. > > Julian Rüger (1): > po: Update German translation. > > Jörg Höhle (4): > ntdll: Use the monotonic time counter also for timer queues. > winmm: More compatible midiIn/Out[Un]Prepare MHDR_* flag handling. > winmm: More compatible waveIn/Out[Un]Prepare WHDR_* flag handling. > winmm: Handle GetTickCount overflow in MIDI player. > > Ken Thomases (25): > winemac: Implement ScrollDC(). > winemac: Implement support for minimizing windows. > winemac: Build a map from Mac virtual key codes to Win32 vkeys and > scan codes based on Mac keyboard layout. > winemac: Rebuild key map when Mac keyboard layout changes. > winemac: Generate KEY_PRESS/RELEASE events from Cocoa key events. > user32: In TranslateMessage, handle ToUnicode returning 2 or more > chars. > winemac: Implement ToUnicodeEx(). > winemac: Ignore Cocoa's window frame adjustments for off-screen > windows. > winemac: Implement Beep(). > winemac: Implement MapVirtualKeyEx(). > winemac: Implement VkKeyScanEx(). > winemac: Implement GetKeyNameText(). > winemac: Implement fake support for the active Windows keyboard > layout. > winemac: Implement SysCommand() to prevent Alt keypresses from > activating system menu. > winemac: Add WineApplication methods to convert from Cocoa to Win32 > coordinate space. > winemac: Implement MOUSE_MOVED(_ABSOLUTE) events. > winemac: Process clicks in an inactive window in addition to > activating it. > secur32: On Mac, add support for cipher suites defined in 10.8 SDK. > user32: Improve mouse wheel scrolling in edit control. > user32: Improve mouse wheel scrolling in listbox control. > comctl32: Eliminate redundant local variable (gcWheelDelta == > -wheelDelta). > comctl32: Improve mouse wheel scrolling in listview control. > comctl32: Improve mouse wheel scrolling in treeview control. > winemac: Implement MOUSE_SCROLL events. > winemac: Prevent Cocoa from consuming certain key-down events. > > Ludger Sprenker (1): > windowscodecs: Add enum type WICTiffCompressionOption. > > Marcus Meissner (1): > comdlg32: Free files on errorpath (Coverity). > > Michael Stefaniuc (20): > ddraw/tests: Use assignment instead of memcpy to copy structs. > crypt32: Use assignment instead of memcpy to copy structs. > wined3d: Remove ERR() on HeapAlloc failure for small sizes known at > compile time. > ieframe: Let C look like C. > krnl386.exe16: Let C look like C. > gphoto2.ds: Let C look like C. > iphlpapi: Let C look like C. > qedit: Let C look like C. > d3drm: Let C look like C. > riched20: Let C look like C. > usp10: Let C look like C. > d3d10core: Remove ERR() on HeapAlloc failure for small sizes known > at compile time. > d3d8: Remove ERR() on HeapAlloc failure for small sizes known at > compile time. > d3d9: Remove ERR() on HeapAlloc failure for small sizes known at > compile time. > d3dx9_36: Remove ERR() on HeapAlloc failure for small sizes known at > compile time. > d3drm: Remove ERR() on HeapAlloc failure for small sizes known at > compile time. > amstream: Remove ERR() on HeapAlloc failure for small sizes known at > compile time. > d3dcompiler_43: Remove ERR() on HeapAlloc failure for small sizes > known at compile time. > dmime: Remove ERR() on HeapAlloc failure for small sizes known at > compile time. > wineoss.drv: Use offsetof to calculate the size of a struct with > variable length array. > > Nikolay Sivov (3): > oledb32: Add a table of init properties, support Persist Security > Info switch in init string. > xmllite: Store start data pointer for comments. > xmllite: Basic test for chunk value read vs full value read. > > Nozomi Kodama (1): > d3dx9: Use const instead of CONST. > > Piotr Caban (11): > jscript: Rename jsheap_t to heap_pool_t. > jscript: Don't use jsstr_t in JSRegExp structure. > jscript: Don't use jsstr_t in js_NewRegExp. > jscript: Move code operating on JSRegExp out of do_regexp_match_next > function. > jscript: Make context generic in internal regular expression > functions. > jscript: Moved generic regular expressions code to separate file. > vbscript: Added IRegExp2 stub. > vbscript: Added IRegExp2_QueryInterface tests. > vbscript: Added partial implementation of IDispatch methods in > IRegExp2. > vbscript: Added IRegExp2::Pattern implementation. > vbscript: Rename vbsheap to heap_pool. > > Roman Dadkov (3): > ole32/tests: Add test for SetClientSite. > mpr/tests: Test to check the return values of WNetGetUniversalNameA. > ole32/tests: Add test for OleRun. > > Stefan Dösinger (2): > wined3d: Use the ARBfp ffp pipeline only if ARBfp is supported. > wined3d: Reserve one vs constant on r200. > > Vijay Kiran Kamuju (1): > msvcr90: Update the manifest version. > > -- > Alexandre Julliard > julliard(a)winehq.org > >

2 1

Re: [PATCH] Support a protection scheme which reset the GS selector - v2
by Sergey Guralnik 17 Feb '13

17 Feb '13

On 2013-02-07 17:00, Alessandro Pignotti wrote: >+ opcode=(unsigned char*)context->Eip; >+ if (*opcode==0x65 && /* GS segment instruction prefix */ >+ context->SegGs!=ntdll_get_thread_data()->gs) Segment-override prefix may be preceded by repeat or operand-size override prefixes. So instruction that refers to GS-segment doesn't always starts with 0x65. -- Sergey

3 2

Re: [PATCH] ntdll: Support a protection scheme which reset the GS selector - v3
by Ken Thomases 17 Feb '13

17 Feb '13

On Feb 17, 2013, at 4:18 AM, Alessandro Pignotti wrote: > Third attempt > > ---------------- > > Hi everyone, > > I've found two different games: > > -) Of orcs and men > -) The testament of sherlock holmes > > which are using a protection scheme which reset the GS segment selector, > possibly to confuse virtual machines. Since on linux the GS selector is > used internally by glibc the game would crash as soon as a native glibc > method is called by wine. The proposed patch tries to workaround this > problem by restoring the GS to a known working value in wine's SEGFAULS > handler and try the instruction again. > > Regards, > Alessandro Pignotti > > <0001-ntdll-Support-a-protection-scheme-which-reset-the-GS-v3.patch> Didn't Alexandre already fix this with <http://source.winehq.org/git/wine.git/?a=commit;h=64ddb263d25dfceeeae8457a0…>? -Ken

1 0

Fwd: Re: Fix catalyst brain damage to speed up Falcon BMS 2x
by Stanisław Halik 16 Feb '13

16 Feb '13

Ben allowed me to forward this email. Kudos to him for all the knowledge his brainbox contains! -------- Original Message -------- Subject: Re: Fix catalyst brain damage to speed up Falcon BMS 2x Date: Sat, 16 Feb 2013 11:20:35 -0500 From: Ben Supnik <bsupnik(a)xsquawkbox.net> To: Stanislaw Halik <sthalik(a)misaki.pl> Hi Guys, I'm afraid I don't know enough about the _specific_ situation you guys are seeing. I can tell you guys a few things from my GL work: 1. The ATI OpenGL Linux team is pretty accessible; do you guys have anyone in the fglrx beta program? 2. What we found was that for stream-draw buffers that need to be orphaned, mapped, unmapped and drawn, there was a fixed overhead in the ATI drivers compared to NV; this 'performance gap' is cross-platform - both NV and ATI use the same GL stack (more or less) for Windows and Linux, and we saw the slow-down on both. 3. We originally were using map buffer (not map buffer range) with a NULL glBufferData to "orphan" the buffer (the equivalent of d3d map-discard). I think I tried MBR and it didn't fix it - both were expensive because the fundamental memory mapping operation was slow. 4. The slowness was in milliseconds, e.g. "this hits our fps by 20% or 30%" - but it wasn't "this is 3x slower because it stalled the GPU." So if you're seeing truly face-meltingly bad performance, like a total pipeline stall, you have a different bug. 5. As a general statement, the original glMapBuffer is subject to a lot of heuristic behavior in the drivers; app developers are very fast and loose with how they use it, so the driver vendors tend to try to make it do the fastest, most useful, least crash-y thing because the apps use it like monkeys on type-writers. By comparison, MBR came out later and has much more specific semantics for particular optimizations, as a result, the MBR implementation will often do exactly what you say, _even_ if it's slower. Getting even one flag wrong in MBR can cause it to hit a face-meltingly slow path. We worked around the perf cost of mapping a buffer on ATI hw by using pinned memory (but we do have a Linux-only bug where we get corrupt geometry with pinned memory - it works on Windows); I have some todo items to investigate the problem more thoroughly now. Cheers Ben On 2/16/13 5:47 AM, Stanislaw Halik wrote: > On 2013-02-16 09:04, Stefan Dösinger wrote: >> What you really want to do is figure out why GL_ARB_map_buffer_range >> is slow on fglrx, and make sure that the problem is really fglrx >> specific. I fixed a number of dynamic buffer performance problems in >> the past months, but there are still problems if we're falling back >> to draw_strided_slow for some reason, like fixed function material >> tracking. > > Thanks for reviewing this. > > Going to ask Ben Supnik from Laminar Research (X-Plane developer) and > BCC him, since he has apparently run into the same issue. There's much > info of fglrx woes (not really Linux specific, either) on > http://developer.x-plane.com/ > > He said publicly to be in contact with AMD themselves, and been friendly > to OSS by releasing an X-Plane Linux version, as well as overall cool > fellow. > > Ben, Please help! > >> Other than being wrong conceptually, you're disabling dynamic buffers >> the wrong way: The "proper" way would be to add a quirk to the >> quirk_table in directx.c that removes ARB_map_buffer_range from the >> list of supported extensions if the driver vendor is AMD. > > Like this? Patch attached. > > I've run into hard GPU hangs with fglrx 13.2, no VT switch either. This > helps: > > [Software\\Wine\\Direct3D] > "DirectDrawRenderer"="gdi" > "Multisampling"="disabled" > "OffscreenRenderingMode"="fbo" > "UseGLSL"="enabled" > > Lack of GLSL disables HDR apparently. > > Without GDI, there's some nasty display corruption on FBOs. > > Also Catalyst likes to hang display when switching from 3D to 2D and VT > switch helps. > > But with all this busywork, performance is near-native. Catalyst at > least supports indirect addressing (whatever that means) and doesn't > choke on > 128 temps... FYI Mesa bug submitted: > > https://bugs.freedesktop.org/show_bug.cgi?id=55420 > > -sh > -- Scenery Home Page: http://scenery.x-plane.com/ Scenery blog: http://www.x-plane.com/blog/ Plugin SDK: http://www.xsquawkbox.net/xpsdk/ X-Plane Wiki: http://wiki.x-plane.com/ Scenery mailing list: x-plane-scenery(a)yahoogroups.com Developer mailing list: x-plane-dev(a)yahoogroups.com

1 0

Re: Fix catalyst brain damage to speed up Falcon BMS 2x
by Stefan Dösinger 16 Feb '13

16 Feb '13

Am 16.02.2013 um 08:39 schrieb Stanisław Halik <sthalik(a)misaki.pl>: > + dynamic_buffer_ok = gl_info->supported[APPLE_FLUSH_BUFFER_RANGE] || > + (!gl_info->[GL_AMDX_debug_output] && gl_info->supported[ARB_MAP_BUFFER_RANGE]); What you really want to do is figure out why GL_ARB_map_buffer_range is slow on fglrx, and make sure that the problem is really fglrx specific. I fixed a number of dynamic buffer performance problems in the past months, but there are still problems if we're falling back to draw_strided_slow for some reason, like fixed function material tracking. If you absolutely have to do some fglrx hacks for performant dynamic buffers, look into GL_AMD_pinned_memory. I'd prefer not to use this extension though. Other than being wrong conceptually, you're disabling dynamic buffers the wrong way: The "proper" way would be to add a quirk to the quirk_table in directx.c that removes ARB_map_buffer_range from the list of supported extensions if the driver vendor is AMD.

4 8

Re: Fix catalyst brain damage to speed up Falcon BMS 2x
by Stanislaw Halik 16 Feb '13

16 Feb '13

Updated patch following review.

1 0

Re: [website] Simplified Chinese translation for release 1.5.24
by Jactry Zeng 16 Feb '13

16 Feb '13

Oops...I made a mistake, this is the patch. So sorry :-(

1 0

mmsystem 16bit questions
by Joerg-Cyril.Hoehle＠t-systems.com 15 Feb '13

15 Feb '13

Hi, I noticed some dubious differences among winmm16 functions. I believe there should be none, but perhaps I'm missing some 16bit idiosyncrasies? Please compare waveOutPrepareHeader16, Unprepare16 and waveOutWrite16: http://source.winehq.org/source/dlls/mmsystem.dll16/mmsystem.c#L1287 UINT16 WINAPI waveOutUnprepareHeader16(HWAVEOUT16 hWaveOut, /* [in] */ SEGPTR lpsegWaveOutHdr, /* [???] */ UINT16 uSize) /* [in] */ { [...] return MMSYSTDRV_Message(..., WODM_UNPREPARE, lpsegWaveOutHdr, uSize); UINT16 WINAPI waveOutWrite16(HWAVEOUT16 hWaveOut, /* [in] */ LPWAVEHDR lpsegWaveOutHdr, /* [???] NOTE: SEGPTR */ UINT16 uSize) /* [in] */ { [...] return MMSYSTDRV_Message(..., WODM_WRITE, (DWORD_PTR)lpsegWaveOutHdr, uSize); The mmsystem.spec knows no difference: pascal waveOutPrepareHeader(word segptr word) waveOutPrepareHeader16 pascal waveOutUnprepareHeader(word segptr word) waveOutUnprepareHeader16 pascal waveOutWrite(word segptr word) waveOutWrite16 I believe: - SEGPTR should be used everywhere the .spec says; - (DWORD_PTR) cast then becomes obsolete. Shall I submit a patch? Furthermore, how to perform a 16bit NULL pointer check? waveInUnPrepareHeader16 uses: LPWAVEHDR lpWaveOutHdr = MapSL(lpsegWaveOutHdr); if (lpWaveInHdr == NULL) return MMSYSERR_INVALPARAM; waveInAddBuffer16 directly uses: if (lpsegWaveInHdr == NULL) return MMSYSERR_INVALPARAM; Last but not least, some but not all of the functions check for a NULL pointer. Why only some? When the check is missing, it looks like MMSYSTDRV_Message will crash when calling: map = drvtype->mapmsg16to32W(msg, &param1, &param2); http://source.winehq.org/source/dlls/mmsystem.dll16/message16.c?v=wine-1.5.… I don't know whether the check should occur or if we should let the app crash (all I know is that native MCI_PLAY crashes on w2k when given MCI_FROM/TO or callback flags with a NULL MCI_PARMS pointer, whereas Wine returns INVALPARAM, whereas a NULL pointer is ok in native when not setting any flags that require access via the pointer). If the check shall be done, then I believe that it should occur centrally in MMSYSTDRV_Message, not within the wave/midi* functions, such that if an app calls e.g. waveInMessage16(WIDM_UNPREPARE) that dispatches to MMSYSTDRV_Message, the check is not bypassed. BTW, MMSYSTDRV_WaveOut_UnMap16To32W forgets to deallocate the wh32 when waveOutPrepare fails. Same with wave in and MIDI. Thank you for 16bit enlightening, Jörg Höhle

2 1

Re: [PATCH 2/5] vbscript: Added IRegExp2_QueryInterface tests
by Marvin 15 Feb '13

15 Feb '13

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=24416 Your paranoid android. === WINEBUILD (build) === Make failed

2 1

MSI signing
by Marc-André Lureau 15 Feb '13

15 Feb '13

Hello! PE signing is already publicly documented by Microsoft, and we have an open-source tool (osslsigncode) capable of signing PE and Cabinet. A signed MSI dump easily reveals that the signature is under the '\005DigitalSignature' stream, and it's a regular pkcs#7 signature, whose content info type is 1.3.6.1.4.1.311.2.1.30 (SPC_SIPINFO_OBJID whatever that might be). To ease debugging, I added asn1parse of signature to msitools with "msidump --signature". You can also compare two MSI signatures with msidiff (http://git.gnome.org/browse/msitools) I found this blog http://qistoph.blogspot.com.es/2012/01/manual-verify-pkcs7-signed-data-with… very helpful, so my testing setup currently is: - self signed pkcs12 certificate - minimal "starting point" MSI crafted with msibuild mini.msi -s '', modified with hex editor for 0 length streams - signtool from 7.0A sdk, used simply "sign /f cert.pfx /v /p passphrase mini.msi" >From there, I get a "canonical" message digest sha1 value of 98C090F33284468D64F30EE7B8C00894135D54C8. Only the stream content seems to be used to compute it. All modifications I can think of to the MSI do not seem to alter that sha1, none of: OLECF internal data, stream name, number of (empty) streams, total file length, position of the streams, or whatever I could tweak in the OLECF data seems to change the resulting sha1. So it very much looks like an initial "seed", somehow? Based on my limited knowledge of sha1 computation, I have tried initializing the sha1 by modifying its initial value manually (the h values) to match with the "canonical" sha1, and added 1 byte to a stream. I was hoping to get the same result, but that didn't work that way. It could be that my modified sha1 is wrong, or that they apply further transformation on the stream content before computing the checksum, although this would be surprising since they don't do that for PE or cabinet file. Any suggestion or help is welcome! -- Marc-André Lureau

1 1

64bit integer overflow in 3 days
by Joerg-Cyril.Hoehle＠t-systems.com 15 Feb '13

15 Feb '13

Hi, You believe that 64bit integers give you plenty of room and time to avoid trouble like Y2k and Y2033, don't you? The trouble is, these large data types make using other large numbers more attractive than they used to be, e.g. counting time in 100 nanosecond units, representing one second as 10000000. Combining large numbers will cause an overflow sooner than you think. Specifically, the pattern X * numerator / denominator is troublesome. Let's analyse winealsa:AudioClock_GetPosition: QueryPerformanceCounter(&stamp); QueryPerformanceFrequency(&frequency); *qpctime = (stamp.QuadPart * (INT64)10000000) / frequency.QuadPart; QueryPerformanceCounter (QPC) returns time in an unspecified unit (frequency) and needs to convert that into 100 nanosecond units. Graphically, with g - garbage bits, s - shifted, X - valid bits: 6 5 4 3 2 1 4321098765432109876543210987654321098765432109876543210987654321 <<< shift left, multiply divide, shift right >>> Assuming QPC returns 64 valid bits: XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX QPC Multiply with (INT64)10000000 (log2 ~ 23): XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXsssssssssssssssssssssss *10000000 Divide by frequency (happens to be 10000000 too): gggggggggggggggggggggggXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX /10000000 (100ns) You start to see how the number of valid bits already shrunk a lot. Now you don't care about nanoseconds, you want seconds. Divide again: ggggggggggggggggggggggggggggggggggggggggggggggXXXXXXXXXXXXXXXXXX /10000000 (seconds) Ouch, only 18bits left to express seconds! How many days is that? (/ (ash 1 18) 3600.0 24) -> ~3.03 So after 3 days in Wine, mmdevapi's GetPosition clock has wrapped. 3 days is not much compared to over 50000 years that 64 bit integers allow to express with 100 nanosecond units, isn't it? I don't know if GetPosition is related to bug #26968 where a user reports that foobar stops playing audio after 4-7 days, but you get a clue. Cross-check: 0x40000 is 1^18 seconds ~ 3 days 0x2625A000000 same in 100 nanosecond units 0x16BCC41E900000000 the * (INT64)10000000 part in winealsa ^1 bit unsigned overflow (more with signed integers) 0x6BCC41E900000000 clipped 0xB4DAD65435 after /10000000, a value already reached after ~21 hours There are even less bits, because LARGE_INTEGER is signed. There are several instances of that X * p / q pattern in Wine. Examples: ntdll/threadpool.c:queue_current_time: return now.QuadPart * 1000 / freq.QuadPart; ntdll/kernel_main.c:GetTickCount64: return counter.QuadPart * 1000 / frequency.QuadPart; IOW, GetTickCount64 does not offer 64 valid bits. ntdll/time.c:monotonic_counter: return mach_absolute_time() * timebase.numer / timebase.denom / 100; This means that on MacOS, QPC may overflow even earlier because it does not offer 64 valid bits. I've prepared a few patches to fix some of these. Generally, I don't know what to recommend. Use double floating point intermediates and loose 11 bits for the exponent but at least get the magnitude right? Regards, Jörg Höhle

2 1

Re: d3dx9: Specify arguments in trace to ease debugging
by Alexandre Julliard 15 Feb '13

15 Feb '13

Nozomi Kodama <nozomi.kodama(a)yahoo.com> writes: > Hello. > > I took advantage of this minor patch to clean up and make uniform the > coding style of this file. Please don't do that. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: vbscript: Added support for a few more constants
by Marvin 15 Feb '13

15 Feb '13

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=24408 Your paranoid android. === 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 === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: d3dx9: Specify arguments in trace to ease debugging
by Rico Schüller 15 Feb '13

15 Feb '13

On 15.02.2013 08:20, Nozomi Kodama wrote: > @@ -721,9 +739,10 @@ D3DXMATRIX* WINAPI D3DXMatrixTransformation(D3DXMATRIX *pout, const D3DXVECTOR3 > > @@ -797,69 +821,70 @@ D3DXMATRIX* WINAPI D3DXMatrixTransformation2D(D3DXMATRIX *pout, const D3DXVECTOR Those two functions need a speed improvement. Normally we change only the code (and it's style) if we touch that area. I'm not a fan of changing that in one go in a huge patch, as it might introduce new mistakes, see below. I'm not sure if I caught all issues. > + rot.w = cos(rotation/2.0f); rot.w = cos(rotation / 2.0f); > > - TRACE("iface %p\n", iface); > + TRACE("iface %p, %pm\n", iface, pm); Typo. > + TRACE("iface %p, x %f, y %f, z %f\n", iface, x, y, z); > > + TRACE("(pout %p, pvpoint %p, pvnormal %p)\n", pout, pvpoint, pvnormal); Well, I would use the same style for both... I'd prefer the first one. Cheers Rico

1 0

Re: [PATCH 2/2] wininet: Set available bytes in InternetQueryDataAvailable even if it ends up in async call
by Marvin 14 Feb '13

14 Feb '13

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=24401 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

2 1

Where to put Chinese translation of the Wine user guide?
by Jactry Zeng 14 Feb '13

14 Feb '13

Hi folks, A Chinese Wine user Liu Hongcong had translated [1] the Wine user guide [2] to Chinese three years ago, the translation looks mostly good to me. It is a pity to leave his translation outside of the official wine website. With his consent, I want to put his translation to winehq. But I can't find where to add it into in website's source [3]. Or I should add it into wiki? Thanks for any comment! [1] http://blog.csdn.net/hongmy525/article/details/2238600 [2] http://www.winehq.org/docs/wineusr-guide/index [3] http://source.winehq.org/git/website.git/

2 2

Re: [3/4] gdi32: IntersectClipRect should update actual clipping region for a EMF DC.
by Marvin 14 Feb '13

14 Feb '13

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=24396 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [2/4] gdi32: OffsetClipRgn should update actual clipping region for a EMF DC.
by Marvin 14 Feb '13

14 Feb '13

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=24395 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [1/4] gdi32: Add more EMF clipping tests.
by Marvin 14 Feb '13

14 Feb '13

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=24394 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [PATCH] dsound: copy program parameters in SetFormat
by Andrew Eikum 13 Feb '13

13 Feb '13

On Mon, Feb 11, 2013 at 02:06:42PM +0100, Maarten Lankhorst wrote: > Fixes bug #32312, apparently some programs depend on the format it specifies being returned. > > Just set wBitsPerSample, nSamplesPerSec and nChannels to make those applications happy. > The patch seems fine, but can you add some tests for this so we can nail down exactly what we should be taking from the SetFormat format? E.g. should we just do "device->primary_fmt = CopyFormat(passed_fmt)" in all cases, or can you find a case where the GetFormat() result is changed? FYI, your patch applies with an offset and the Wine patch tracker didn't like this: > --- a/dlls/dsound/primary.c > +++ a/dlls/dsound/primary.c but git-am imported the raw mail file just fine. Andrew

1 0

Re: winmm: More compatible waveIn/Out[Un]Prepare WHDR_* flag handling.
by Andrew Eikum 13 Feb '13

13 Feb '13

These two patches look fine to me. Thanks! On Mon, Feb 11, 2013 at 01:28:09PM +0100, Joerg-Cyril.Hoehle(a)t-systems.com wrote: > Hi, > > This completes my winmm Prepare Header changes on the wave side. > > XyzPrepare: > + header->dwFlags &= ~(WHDR_DONE|WHDR_INQUEUE); /* flags cleared since w2k */ > > I choose to implement the behavior since w2k. Hopefully this will not break some w9x app. > > > WINMM_UnprepareHeader: > - header->dwFlags |= WHDR_DONE; > Never ever observed. Neither w95 nor xp or w7. > Perhaps MS would have been wise to do so, as simply looking at a header you cannot > tell whether it's been processed or not (returned with error code). > I choose to implement compatible behaviour. > > > - if(!(lpWaveInHdr->dwFlags & WHDR_PREPARED)) > - return MMSYSERR_NOERROR; > if(lpWaveInHdr->dwFlags & WHDR_INQUEUE) > return WAVERR_STILLPLAYING; > + if(!(lpWaveInHdr->dwFlags & WHDR_PREPARED)) > + return MMSYSERR_NOERROR; > Note the subtle difference in behaviour between wave and midi headers... > > > >IMHO calling named winmm:xyz functions should be equivalent to dispatching MM* messages > > I forgot to add that I'm still surprised that Andrew Eikum could reimplement > waveIn/OutMessage *without* using MMDRV_Message by simply returning MMSYSERR_NOTSUPPORTED. > > Either all apps use the named winmm:waveIn/OutXyz functions instead of calling waveOutMessage() > or nobody bothered to complain that some apps (perhaps w3.1 ones?) lack sound. > > That area obviously needs some testing... > > Regards, > Jörg Höhle

1 0

Re: kernel32: Indicate that HeapValidate called by GlobalHandle is not the sign of heap corruption.
by Alexandre Julliard 13 Feb '13

13 Feb '13

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > This helps to distinguish this one from real signs of heap corruption, > and saves quite a bit of effort analysing +heap or warn+heap logs. Avoiding HeapValidate would probably be a better approach. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: advapi32: Report world-access ownership by default for registry objects (try 2).
by Alexandre Julliard 13 Feb '13

13 Feb '13

"Erich E. Hoover" <ehoover(a)mymail.mines.edu> writes: > The attached patch fixes an issue where the Opera installer expects > registry keys to return ownership information (Bug #32904), concluding > the cleanup of [Get|Set]NamedSecurityInfo. This patch addresses the > issue by ensuring that NULL owner and group information is never > returned for registry keys and instead returns the World SID (S-1-1-0) > when this information is unavailable, which is sufficient to keep the > Opera installer from encountering an error. The included test shows > that this behavior is essentially the case for system keys on Windows, > though the Administrators SID is returned instead. Since > GetNamedSecurityInfo historically returned the World SID, and several > other routines use it as well) I've opted to use the World SID for > now. We should do the same as Windows unless there's a very good reason to be different. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] netstat: Clarify labels in UDP statistics.
by Frédéric Delanoy 13 Feb '13

13 Feb '13

On Sat, Feb 9, 2013 at 1:03 PM, Lauri Kenttä <lauri.kentta(a)gmail.com> wrote: > Change labels in UDP statistics so that they better reflect the actual meanings as described in MSDN [1]. > > [1] http://msdn.microsoft.com/en-us/library/windows/desktop/aa366929%28v=vs.85%… > --- > - IDS_UDP_NO_PORTS, "No Ports" > - IDS_UDP_RECV_ERRORS, "Receive Errors" > + IDS_UDP_NO_PORTS, "Invalid Port Errors" > + IDS_UDP_RECV_ERRORS, "Invalid Datagrams" > IDS_UDP_DGRAMS_SENT, "Datagrams Sent" "Invalid Datagrams" is too vague. It could mean datagrams sent or received. If you want to disambiguate/explain, use message contexts as explained in http://wiki.winehq.org/Translating Frédéric

2 3

Re: msi/tests: Move a couple of tests from install.c to msi.c.
by Marvin 13 Feb '13

13 Feb '13

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=24381 Your paranoid android. === WNT4WSSP6 (32 bit install) === install.c:5673: Test failed: Per-user icon file isn't where it's expected (C:\WINNT\Profiles\Default User\Application Data\Microsoft\Installer\{7DF88A49-996F-4EC8-A022-BF956F9B2CBB}\testicon)

1 0

Re: gdi32: Make sure that actual clipping region is updated for a EMF DC.
by Alexandre Julliard 13 Feb '13

13 Feb '13

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > This patch not only fixes a todo_wine in the tests, but also makes print > preview and printing work in a pretty large application. You would most likely have to do that for all the clipping functions then. -- Alexandre Julliard julliard(a)winehq.org

2 3

Re: d3dx9: Implement D3DXSHEvalConeLight
by Matteo Bruni 12 Feb '13

12 Feb '13

Hi, in addition to what I already told you in #winehackers (that is, please rename the test struct field red_in to red_out and red_out to red_expected, or something like that), you should take the chance to do some more cleanup. That means e.g. use "const" instead of "CONST" as Rico already suggested and fix the indentation of the last comment you're adding. There are a few other things I don't really like but it's about code style and my personal preferences, so you are free to ignore them. I'd write the test structure declaration as: struct { float *red_out, *green_out, *blue_out; const float *red_expected, *green_expected, *blue_expected; float radius, red_offset, green_offset, blue_offset; } test[] = { {rout, gout, bout, table, &table[72], &table[144], 0.5f, 1.01f, 1.02f, 1.03f}, ... (I'm sure Gmail will wrap and make this piece of code unreadable, but well...) In general I'd also avoid using caps in variable names (e.g. r_intensity instead of Rintensity).

1 0

Re: [PATCH 2/4] urlmon: Fixed QueryInfo tests during BINDSTATUS_PROXYDETECTING notification
by Marvin 12 Feb '13

12 Feb '13

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=24361 Your paranoid android. === WVISTAADM (32 bit url) === url.c:1868: Test failed: binding failed: 800c0005, expected 00000000 url.c:1879: Test failed: res = 00002ee7, expected 00000000 url.c:3035: Test failed: IMoniker_BindToStorage failed: 800c0005 url.c:3036: Test failed: unk == NULL url.c:1868: Test failed: binding failed: 800c0005, expected 00000000 url.c:1879: Test failed: res = 00002ee7, expected 00000000 url.c:3035: Test failed: IMoniker_BindToStorage failed: 800c0005 url.c:3036: Test failed: unk == NULL url.c:1658: Test failed: unexpected call Obj_OnProgress_FINDINGRESOURCE url.c:1868: Test failed: binding failed: 800c0005, expected 00000000 url.c:1879: Test failed: res = 00002ee7, expected 00000000 url.c:1662: Test failed: unexpected call OnProgress_FINDINGRESOURCE url.c:1868: Test failed: binding failed: 800c0005, expected 00000000 url.c:1879: Test failed: res = 00002ee7, expected 00000000 url.c:3035: Test failed: IMoniker_BindToStorage failed: 800c0005 url.c:3036: Test failed: unk == NULL url.c:1868: Test failed: binding failed: 800c0005, expected 800c0019 url.c:1879: Test failed: res = 00002efd, expected 800c0019 url.c:3024: Test failed: Got 800c0005 url.c:3119: Test failed: expected QueryInterface_IHttpSecurity url.c:3120: Test failed: expected QueryService_IHttpSecurity url.c:3121: Test failed: expected OnSecurityProblem url.c:1662: Test failed: unexpected call OnProgress_FINDINGRESOURCE url.c:1868: Test failed: binding failed: 800c0005, expected 800c0019 url.c:1879: Test failed: res = 00002efd, expected 800c0019 url.c:3024: Test failed: Got 800c0005 url.c:3119: Test failed: expected QueryInterface_IHttpSecurity url.c:3120: Test failed: expected QueryService_IHttpSecurity url.c:3121: Test failed: expected OnSecurityProblem url.c:1662: Test failed: unexpected call OnProgress_FINDINGRESOURCE url.c:1868: Test failed: binding failed: 800c0005, expected 00000000 url.c:1879: Test failed: res = 00002efd, expected 00000000 url.c:3035: Test failed: IMoniker_BindToStorage failed: 800c0005 url.c:3036: Test failed: unk == NULL url.c:3119: Test failed: expected QueryInterface_IHttpSecurity url.c:3120: Test failed: expected QueryService_IHttpSecurity url.c:3121: Test failed: expected OnSecurityProblem url.c:3139: Test failed: expected OnResponse url.c:3143: Test failed: expected OnProgress_MIMETYPEAVAILABLE url.c:3144: Test failed: expected OnProgress_BEGINDOWNLOADDATA url.c:3145: Test failed: expected OnProgress_ENDDOWNLOADDATA url.c:3153: Test failed: expected OnDataAvailable url.c:1662: Test failed: unexpected call OnProgress_FINDINGRESOURCE url.c:1868: Test failed: binding failed: 800c0005, expected 00000000 url.c:1879: Test failed: res = 00002efd, expected 00000000 url.c:3027: Test failed: IMoniker_BindToStorage failed: 800c0005 url.c:3028: Test failed: unk == NULL url.c:3030: Test failed: Expected security problem to be ignored. url.c:3119: Test failed: expected QueryInterface_IHttpSecurity url.c:3120: Test failed: expected QueryService_IHttpSecurity url.c:3121: Test failed: expected OnSecurityProblem url.c:1662: Test failed: unexpected call OnProgress_FINDINGRESOURCE url.c:1868: Test failed: binding failed: 800c0005, expected 00000000 url.c:1879: Test failed: res = 00002ee7, expected 00000000 url.c:3133: Test failed: expected OnProgress_SENDINGREQUEST url.c:3139: Test failed: expected OnResponse url.c:3143: Test failed: expected OnProgress_MIMETYPEAVAILABLE url.c:3144: Test failed: expected OnProgress_BEGINDOWNLOADDATA url.c:3145: Test failed: expected OnProgress_ENDDOWNLOADDATA url.c:3157: Test failed: expected OnDataAvailable url.c:1662: Test failed: unexpected call OnProgress_FINDINGRESOURCE url.c:1868: Test failed: binding failed: 800c0005, expected 00000000 url.c:1879: Test failed: res = 00002ee7, expected 00000000 url.c:3133: Test failed: expected OnProgress_SENDINGREQUEST url.c:3139: Test failed: expected OnResponse url.c:3143: Test failed: expected OnProgress_MIMETYPEAVAILABLE url.c:3144: Test failed: expected OnProgress_BEGINDOWNLOADDATA url.c:3145: Test failed: expected OnProgress_ENDDOWNLOADDATA url.c:3157: Test failed: expected OnDataAvailable url.c:1658: Test failed: unexpected call Obj_OnProgress_FINDINGRESOURCE url.c:1868: Test failed: binding failed: 800c0005, expected 00000000 url.c:1879: Test failed: res = 00002ee7, expected 00000000 url.c:3353: Test failed: expected Obj_OnProgress_SENDINGREQUEST url.c:3357: Test failed: expected OnResponse url.c:3359: Test failed: expected Obj_OnProgress_MIMETYPEAVAILABLE url.c:3360: Test failed: expected Obj_OnProgress_BEGINDOWNLOADDATA url.c:3366: Test failed: expected Obj_OnProgress_CLASSIDAVAILABLE url.c:3367: Test failed: expected Obj_OnProgress_BEGINSYNCOPERATION url.c:3368: Test failed: expected Obj_OnProgress_ENDSYNCOPERATION url.c:3369: Test failed: expected OnObjectAvailable url.c:1662: Test failed: unexpected call OnProgress_FINDINGRESOURCE url.c:1868: Test failed: binding failed: 800c0005, expected 00000000 url.c:1879: Test failed: res = 00002ee7, expected 00000000 url.c:3133: Test failed: expected OnProgress_SENDINGREQUEST url.c:3139: Test failed: expected OnResponse url.c:3143: Test failed: expected OnProgress_MIMETYPEAVAILABLE url.c:3144: Test failed: expected OnProgress_BEGINDOWNLOADDATA url.c:3145: Test failed: expected OnProgress_ENDDOWNLOADDATA url.c:3157: Test failed: expected OnDataAvailable url.c:1658: Test failed: unexpected call Obj_OnProgress_FINDINGRESOURCE Timeout

2 1

winealsa: Have the MIDI recorder wait in poll(), not snd_seq_event_input().
by Joerg-Cyril.Hoehle＠t-systems.com 12 Feb '13

12 Feb '13

Johannes Kroll asked: >Did this patch get through? I don't see it on >http://source.winehq.org/patches/ ... It did. You looked too late in patches/. Committed patches are deleted sooner than rejected ones that are in turn deleted sooner than new/pending patches. I'm still interested in your testing my other MIDI patches. I fact I have a queue of half a dozen patches ready for submission, except that I'd like somebody to test them with real MIDI HW. There's another patch yet to be written: midiOutLongMessage CAN be used for coalesced non-sysex messages. This affects winealsa. Wineoss needs no change, because it simply dumps the bytes, one by one, regardless of SysEx or standard message. ALSA needs different API functions to be used. I've yet to check MacOSX CoreMIDI. Regards, Jörg Höhle

2 1

Re: winspool.drv: There is no need to speculate about wineps.drv default DEVMODE.
by Marvin 12 Feb '13

12 Feb '13

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=24350 Your paranoid android. === WINEBUILD (build) === Can't determine base name

1 0

Re: make_xftmpl: Explicitly use signed char when appropriate
by Alexandre Julliard 11 Feb '13

11 Feb '13

André Hentschel <nerv(a)dawncrow.de> writes: > For platforms where char is expected to be unsigned (e.g. it fixes a warning on ARM) There's no reason to make it signed. Please fix the bug correctly instead. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: windowscodecsext: Implement WICCreateColorTransform_Proxy and IWICColorTransform_Initialize_Proxy.
by Vincent Povirk 11 Feb '13

11 Feb '13

OK, I'm about to say something weird, and I'll understand if everyone else decides to ignore it. I don't think WICCreateColorTransform_Proxy, or any of the Proxy functions, should initialize or make use of COM. Instead, I think it should work by calling WICCreateImagingFactory_Proxy. I suspect that part of the purpose of the _Proxy functions is to make it possible to use WIC without needing COM at all, which is something we may want in the future if we want to use it from user32. (Another consequence of this is that we should be able to create instances of builtin decoders/encoders without COM, which we currently can't do. But I'd have to write a test for that, and currently there's been no demonstrated need for it.)

1 0

Re: winealsa: Have the MIDI recorder wait in poll(), not snd_seq_event_input().
by Johannes Kroll 11 Feb '13

11 Feb '13

On Thu, 24 Jan 2013 10:33:58 +0100 <Joerg-Cyril.Hoehle(a)t-systems.com> wrote: > Compare with wineoss which uses poll <= 0 already... > > Johannes Kroll mentioned that MIDIIn takes a long time to end. > Indeed, our testsuite ran 3 times into the "kill after 5s" timeout due to this bug. > Now the tests execute much faster. > > Regards, > Jörg höhle Did this patch get through? I don't see it on http://source.winehq.org/patches/ ...

1 0

Re: windowscodecsext: Implement WICCreateColorTransform_Proxy and IWICColorTransform_Initialize_Proxy.
by Marvin 11 Feb '13

11 Feb '13

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=24341 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: winspool.drv: Fallback to cupsGetPPD if cupsGetPPD3 is missing or failed.
by Huw Davies 11 Feb '13

11 Feb '13

On Mon, Feb 11, 2013 at 02:03:26PM +0800, Dmitry Timoshkov wrote: > - if (pcupsGetPPD3) return pcupsGetPPD3( http, name, modtime, buffer, bufsize ); > + if (pcupsGetPPD3) > + { > + http_status_t status; > + > + TRACE( "Calling cupsGetPPD3\n" ); > + status = pcupsGetPPD3( http, name, modtime, buffer, bufsize ); > + if (status == HTTP_OK) return HTTP_OK; > + TRACE("cupsGetPPD3 failed (status %d)\n", status); > + } > Under what conditions does cupsGetPPD succeed if cupsGetPPD3 fails? Huw.

2 2

Re: [2/2] xmllite: Basic test for chunk value read vs full value read
by Marvin 11 Feb '13

11 Feb '13

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=24335 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Debugging and steam
by Gediminas Jakutis 11 Feb '13

11 Feb '13

Good day! Does anyone has any experience on dealing with Steam when debugging a game which depends on Steam? I am trying to squash a bug which makes a game crash. I started from relay logs, but have a bit of a problem - calls from Steam itself produce a few megabytes worth of relay logs every second. Those get mixed together with calls originating from the game. Also, it appears the game takes a couple of seconds after that "bad" call before crashing. When it is mixed together with Steam's own spam, it is a nightmare finding where it went wrong. I tried every idea I could muster on how to get a "pure" log of the game. Not a single one turned out to be effective. I am all out of ideas now. So if anyone had to deal with this and found a satisfactory solution, please enlighten me. Thank You in advance! -- Gediminas Jakutis LDK Varčiai www.varciai.lt

2 1

Re: comctl32/listview: fix LVM_SETICONSPACING on 64bit machines
by Marvin 09 Feb '13

09 Feb '13

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=24327 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: ole32/tests: Add test for IDispatch interface in IOleObject_QueryInterface.
by Alistair Leslie-Hughes 09 Feb '13

09 Feb '13

On 9/02/2013 2:08 AM, Roman Dadkov wrote: > This test is necessary to show that the function > IOleObject_QueryInterface does not have enough processing IID_IDispatch, > but it works for Windose. Therefore, the creation of some objects in a > variety of applications leads to crash. > + hr = IOleObject_QueryInterface(ole_obj, &IID_IDispatch, (void**)&dispatch); > + todo_wine > + ok(hr == S_OK, "got %08x\n", hr); > The IDispatch Interface should be released. Best Regards Alistair Leslie-Hughes

1 0

Re: ole32/tests: Add test for IDispatch interface in IOleObject_QueryInterface.
by Dmitry Timoshkov 09 Feb '13

09 Feb '13

Roman Dadkov <romand(a)etersoft.ru> wrote: > + const CLSID non_existent_class = {0xa5f1772f, 0x3772, 0x490f, {0x9e, 0xc6, 0x77, 0x13, 0xe8, 0xb3, 0x4b, 0x5d}}; Probably it would be a good idea to try make CLSID really unique and bad enough that it's unlikely that somebody uses it for real. Something like 0xdeadbeef,0xde,0xad,0xbe,0xef,... -- Dmitry.

1 0

Re: ole32/tests: Add test for IDispatch interface in IOleObject_QueryInterface.
by Marvin 09 Feb '13

09 Feb '13

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=24324 Your paranoid android. === WNT4WSSP6 (32 bit defaulthandler) === defaulthandler.c:115: Test failed: got 80004002 === W2KPROSP4 (32 bit defaulthandler) === defaulthandler.c:115: Test failed: got 80004002 === WXPPROSP3 (32 bit defaulthandler) === defaulthandler.c:115: Test failed: got 80004002 === W2K3R2SESP2 (32 bit defaulthandler) === defaulthandler.c:115: Test failed: got 80004002 === WVISTAADM (32 bit defaulthandler) === defaulthandler.c:115: Test failed: got 80004002 === W2K8SE (32 bit defaulthandler) === defaulthandler.c:115: Test failed: got 80004002 === W7PRO (32 bit defaulthandler) === defaulthandler.c:115: Test failed: got 80004002 === W7PROX64 (32 bit defaulthandler) === defaulthandler.c:115: Test failed: got 80004002 === TEST64_W7SP1 (32 bit defaulthandler) === defaulthandler.c:115: Test failed: got 80004002 === W7PROX64 (64 bit defaulthandler) === defaulthandler.c:115: Test failed: got 80004002 === TEST64_W7SP1 (64 bit defaulthandler) === defaulthandler.c:115: Test failed: got 80004002

1 0

Re: [PATCH 1/2] shell32: Add a basic implementation for SHGetStockIconInfo
by Dmitry Timoshkov 08 Feb '13

08 Feb '13

Detlef Riekenberg <wine.dev(a)web.de> wrote: > +/**************************************************************************** > + * helper for SHGetStockIconInfo > + */ > +typedef struct stockiconentry_t { > + SHSTOCKICONID id; > + DWORD iconid; > +} stockiconentry; Why bother with a typedef? > +static stockiconentry stockicontable[] = { > + {SIID_DOCNOASSOC, IDI_SHELL_DOCUMENT}, > + {SIID_DOCASSOC, IDI_SHELL_DOCUMENT}, > + {SIID_FOLDER, IDI_SHELL_FOLDER}, > + {SIID_DRIVERNET, IDI_SHELL_NETDRIVE}, > + {SIID_DRIVERCD, IDI_SHELL_CDROM}, > + {SIID_DRIVERRAM, IDI_SHELL_RAMDISK}, > + {SIID_DESKTOPPC, IDI_SHELL_MY_COMPUTER}, > + {SIID_PRINTER, IDI_SHELL_PRINTER}, > + {SIID_SETTINGS, IDI_SHELL_CONTROL_PANEL}, > + {SIID_RECYCLERFULL, IDI_SHELL_FULL_RECYCLE_BIN}, > + {SIID_DELETE, IDI_SHELL_CONFIRM_DELETE}, > +}; Please don't forget to add 'const'. > +static int cmp_stockiconentry(const void *entry1, const void *entry2) > +{ > + stockiconentry *p1 = (stockiconentry *) entry1; > + stockiconentry *p2 = (stockiconentry *) entry2; > + > + return p1->id - p2->id; > +} Don't cast away 'const'. > +HRESULT WINAPI SHGetStockIconInfo(SHSTOCKICONID id, UINT flags, SHSTOCKICONINFO *sii) > +{ > + stockiconentry *entry; > + > + TRACE("(%d, 0x%x, %p)\n", id, flags, sii); > + if ((id < 0) | (id >= (SIID_MAX_ICONS - 1)) || !sii || (sii->cbSize != sizeof(SHSTOCKICONINFO))) > + return E_INVALIDARG; > + > + /* find the requested icon */ > + entry = bsearch(&id, stockicontable, sizeof(stockicontable)/sizeof(stockicontable[0]), > + sizeof(stockiconentry), cmp_stockiconentry); > + > + if (!entry) { > + FIXME("using fallback for id %d\n", id); > + entry = stockicontable; > + } > + > + if (flags) > + FIXME("flags 0x%x not implemented\n", flags); > + > + sii->hIcon = NULL; > + sii->iSysImageIndex = -1; > + sii->iIcon = - entry->iconid; > + > + GetModuleFileNameW(shell32_hInstance, sii->szPath, MAX_PATH); > + > + return S_OK; > +} GetModuleFileNameW can fail, so filling 'sii' may be a bit premature. -- Dmitry.

3 2

Re: [PATCH] winebuild: Use $CCAS to assemble if found
by Charles Davis 08 Feb '13

08 Feb '13

Hi, On Feb 3, 2013, at 7:33 PM, Alexandre Rostovtsev wrote: > Commit c14bdaf1 made winebuild use Clang to assemble if found. > > However, just because a user has some version of Clang installed, it > does not mean that she wants to use Clang to assemble Wine. For example, > a user who has both Clang and GAS installed may want to use GAS to avoid > textrels (see https://bugs.gentoo.org/show_bug.cgi?id=455308). Long term, that should really get fixed in LLVM/Clang. I'll file a PR (if one wasn't already). > > This patch allows the user to override which assembler gets used by > exporting CCAS at Wine configure time; the name CCAS was chosen for > compatibility with automake's standard AM_PROG_AS macro. I wrote the patch that prompted this one. And I'll freely admit, this is why I wanted to limit that patch to Mac OS--to minimize fallout on other platforms (like, obviously, Gentoo :). > --- > configure | 106 ++++++++++++++++++++++++++++++++++++++++++++ > configure.ac | 4 ++ > tools/winebuild/Makefile.in | 5 ++- > tools/winebuild/utils.c | 12 +++++ > 4 files changed, 126 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index e3253ee..d0b7777 100755 > --- a/configure > +++ b/configure > @@ -732,6 +732,8 @@ FLEX > TOOLSDIR > WOW64_DISABLE > TARGETFLAGS > +ac_ct_CCAS > +CCAS > CPPBIN > ac_ct_CXX > CXXFLAGS > @@ -861,6 +863,7 @@ CPPFLAGS > CXX > CXXFLAGS > CCC > +CCAS > CPP > XMKMF' > > @@ -1549,6 +1552,7 @@ Some influential environment variables: > you have headers in a nonstandard directory <include dir> > CXX C++ compiler command > CXXFLAGS C++ compiler flags > + CCAS Assembler command > CPP C preprocessor > XMKMF Path to xmkmf, Makefile generator for X Window System > > @@ -4075,6 +4079,108 @@ cat >>confdefs.h <<_ACEOF > _ACEOF > > > + > +if test -n "$ac_tool_prefix"; then > + for ac_prog in clang gas as > + do > + # Extract the first word of "$ac_tool_prefix$ac_prog", so it can be a program name with args. > +set dummy $ac_tool_prefix$ac_prog; ac_word=$2 > +{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for $ac_word" >&5 > +$as_echo_n "checking for $ac_word... " >&6; } > +if ${ac_cv_prog_CCAS+:} false; then : > + $as_echo_n "(cached) " >&6 > +else > + if test -n "$CCAS"; then > + ac_cv_prog_CCAS="$CCAS" # Let the user override the test. > +else > +as_save_IFS=$IFS; IFS=$PATH_SEPARATOR > +for as_dir in $PATH > +do > + IFS=$as_save_IFS > + test -z "$as_dir" && as_dir=. > + for ac_exec_ext in '' $ac_executable_extensions; do > + if as_fn_executable_p "$as_dir/$ac_word$ac_exec_ext"; then > + ac_cv_prog_CCAS="$ac_tool_prefix$ac_prog" > + $as_echo "$as_me:${as_lineno-$LINENO}: found $as_dir/$ac_word$ac_exec_ext" >&5 > + break 2 > + fi > +done > + done > +IFS=$as_save_IFS > + > +fi > +fi > +CCAS=$ac_cv_prog_CCAS > +if test -n "$CCAS"; then > + { $as_echo "$as_me:${as_lineno-$LINENO}: result: $CCAS" >&5 > +$as_echo "$CCAS" >&6; } > +else > + { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5 > +$as_echo "no" >&6; } > +fi > + > + > + test -n "$CCAS" && break > + done > +fi > +if test -z "$CCAS"; then > + ac_ct_CCAS=$CCAS > + for ac_prog in clang gas as > +do > + # Extract the first word of "$ac_prog", so it can be a program name with args. > +set dummy $ac_prog; ac_word=$2 > +{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for $ac_word" >&5 > +$as_echo_n "checking for $ac_word... " >&6; } > +if ${ac_cv_prog_ac_ct_CCAS+:} false; then : > + $as_echo_n "(cached) " >&6 > +else > + if test -n "$ac_ct_CCAS"; then > + ac_cv_prog_ac_ct_CCAS="$ac_ct_CCAS" # Let the user override the test. > +else > +as_save_IFS=$IFS; IFS=$PATH_SEPARATOR > +for as_dir in $PATH > +do > + IFS=$as_save_IFS > + test -z "$as_dir" && as_dir=. > + for ac_exec_ext in '' $ac_executable_extensions; do > + if as_fn_executable_p "$as_dir/$ac_word$ac_exec_ext"; then > + ac_cv_prog_ac_ct_CCAS="$ac_prog" > + $as_echo "$as_me:${as_lineno-$LINENO}: found $as_dir/$ac_word$ac_exec_ext" >&5 > + break 2 > + fi > +done > + done > +IFS=$as_save_IFS > + > +fi > +fi > +ac_ct_CCAS=$ac_cv_prog_ac_ct_CCAS > +if test -n "$ac_ct_CCAS"; then > + { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_ct_CCAS" >&5 > +$as_echo "$ac_ct_CCAS" >&6; } > +else > + { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5 > +$as_echo "no" >&6; } > +fi > + > + > + test -n "$ac_ct_CCAS" && break > +done > + > + if test "x$ac_ct_CCAS" = x; then > + CCAS=""$CC"" > + else > + case $cross_compiling:$ac_tool_warned in > +yes:) > +{ $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: using cross tools not prefixed with host triplet" >&5 > +$as_echo "$as_me: WARNING: using cross tools not prefixed with host triplet" >&2;} > +ac_tool_warned=yes ;; > +esac > + CCAS=$ac_ct_CCAS > + fi > +fi > + > + > case $host in > *-darwin*) > if test "x$enable_win64" = "xyes" > diff --git a/configure.ac b/configure.ac > index c93cd08..760f325 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -126,6 +126,10 @@ dnl We can't use AC_PROG_CPP for winegcc, it uses by default $(CC) -E > AC_CHECK_TOOL(CPPBIN,cpp,cpp) > AC_DEFINE_UNQUOTED(EXEEXT,["$ac_exeext"],[Define to the file extension for executables.]) > > +dnl Analogous to AM_PROG_AS > +AC_ARG_VAR([CCAS],[Assembler command]) > +AC_CHECK_TOOLS(CCAS,[clang gas as],["$CC"]) If the user doesn't set CCAS--which she doesn't the majority of the time--configure will pick up the first of {clang, gas, as} in the PATH. Are you sure that's what you want? One of the earlier versions of the "Use Clang to assemble" patch attempted to detect Clang in the PATH, and use it if it was there. I was told that winebuild should detect the binary to use at runtime instead. Perhaps we should just not define CCAS if the user didn't specify it. Then below, you can wrap the block you added in an #ifdef. AJ may have more to say about this. > + > case $host in > *-darwin*) > if test "x$enable_win64" = "xyes" > diff --git a/tools/winebuild/Makefile.in b/tools/winebuild/Makefile.in > index 2017129..3fe47a3 100644 > --- a/tools/winebuild/Makefile.in > +++ b/tools/winebuild/Makefile.in > @@ -1,4 +1,7 @@ > -DEFS = -D__WINESRC__ $(EXTRADEFS) > +DEFS = \ > + -DCCAS="\"@CCAS@\"" \ > + -D__WINESRC__ \ > + $(EXTRADEFS) > > PROGRAMS = winebuild$(EXEEXT) > MANPAGE = winebuild.man > diff --git a/tools/winebuild/utils.c b/tools/winebuild/utils.c > index 262ff3a..1c5f918 100644 > --- a/tools/winebuild/utils.c > +++ b/tools/winebuild/utils.c > @@ -352,6 +352,18 @@ struct strarray *get_as_command(void) > static int as_is_clang = 0; > struct strarray *args = strarray_init(); > > + if (!as_command && strlen( CCAS )) > + { > + struct stat st; > + > + if (!stat( CCAS, &st )) > + as_command = CCAS; > + else > + as_command = find_tool( CCAS, NULL ); > + > + if (as_command && strstr( as_command, "clang" )) as_is_clang = 1; You may recall that one of the earlier versions (later than the one that detected Clang in configure) used strstr(3) to detect Clang; I was then told not to use it. I suspect this is because some systems' strstr(3) exhibits quadratic-time behavior. I don't know if you can avoid that in this case, though. Maybe you can just grab the basename and strip off the CTARGET prefix; then you should just be able to do a strcmp(3). Or am I missing something? More generally, calling the variable "CCAS" implies that any old C compiler driver that knows how to assemble can be specified. If the user specifies a compiler that isn't Clang--GCC, for example--she'll get the wrong behavior. In particular, the 'as_is_clang' variable is used below to pass the right switches and options to Clang. Presumably we'd want to do that for the GCC case, too. > + } > + > if (!as_command) > { > as_command = find_tool( "clang", NULL ); > -- > 1.8.1.2 > > > I'm sorry for any inconvenience I might have caused, and I hope we can get this fixed soon. Chip

3 3

[PATCH v2 1/8] locks: make flock_lock_file take is_conflict callback parm
by Pavel Shilovsky 07 Feb '13

07 Feb '13

This parm demetermines how to check if locks have conflicts. This let us use flock_lock_file funtions further to add O_DENY* flags support through flocks. Signed-off-by: Pavel Shilovsky <piastry(a)etersoft.ru> --- fs/locks.c | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/fs/locks.c b/fs/locks.c index a94e331..9edfec4 100644 --- a/fs/locks.c +++ b/fs/locks.c @@ -697,14 +697,20 @@ static int posix_locks_deadlock(struct file_lock *caller_fl, return 0; } -/* Try to create a FLOCK lock on filp. We always insert new FLOCK locks +/* + * Try to create a FLOCK lock on filp. We always insert new FLOCK locks * after any leases, but before any posix locks. * * Note that if called with an FL_EXISTS argument, the caller may determine * whether or not a lock was successfully freed by testing the return * value for -ENOENT. + * + * Take @is_conflict callback that determines how to check if locks have + * conflicts or not. */ -static int flock_lock_file(struct file *filp, struct file_lock *request) +static int +flock_lock_file(struct file *filp, struct file_lock *request, + int (*is_conflict)(struct file_lock *, struct file_lock *)) { struct file_lock *new_fl = NULL; struct file_lock **before; @@ -760,7 +766,7 @@ find_conflict: break; if (IS_LEASE(fl)) continue; - if (!flock_locks_conflict(request, fl)) + if (!is_conflict(request, fl)) continue; error = -EAGAIN; if (!(request->fl_flags & FL_SLEEP)) @@ -1589,7 +1595,7 @@ int flock_lock_file_wait(struct file *filp, struct file_lock *fl) int error; might_sleep(); for (;;) { - error = flock_lock_file(filp, fl); + error = flock_lock_file(filp, fl, flock_locks_conflict); if (error != FILE_LOCK_DEFERRED) break; error = wait_event_interruptible(fl->fl_wait, !fl->fl_next); -- 1.8.1.1

2 20

Re: [PATCH] gdi32/tests: Skip justification tests where Times New Roman is not installed.
by Alexandre Julliard 07 Feb '13

07 Feb '13

Akihiro Sagawa <sagawa.aki(a)gmail.com> writes: > --- > dlls/gdi32/tests/font.c | 6 ++++++ > 1 file changed, 6 insertions(+) There's no reason to require Times New Roman in that test, it could fall back to some other font. -- Alexandre Julliard julliard(a)winehq.org

1 0

[PATCH v2 0/8] Add O_DENY* support for VFS and CIFS/NFS
by Pavel Shilovsky 07 Feb '13

07 Feb '13

This patchset adds support of O_DENY* flags for Linux fs layer. These flags can be used by any application that needs share reservations to organize a file access. VFS already has some sort of this capability - now it's done through flock/LOCK_MAND mechanis, but that approach is non-atomic. This patchset build new capabilities on top of the existing one but doesn't bring any changes into the flock call semantic. These flags can be used by NFS (built-in-kernel) and CIFS (Samba) servers and Wine applications through VFS (for local filesystems) or CIFS/NFS modules. This will help when e.g. Samba and NFS server share the same directory for Windows and Linux users or Wine applications use Samba/NFS share to access the same data from different clients. According to the previous discussions the most problematic question is how to prevent situations like DoS attacks where e.g /lib/liba.so file can be open with DENYREAD, or smth like this. That's why one extra flag O_DENYMAND is added. It indicates to underlying layer that an application want to use O_DENY* flags semantic. It allows us not affect native Linux applications (that don't use O_DENYMAND flag) - so, these flags (and the semantic of open syscall that they bring) are used only for those applications that really want it proccessed that way. So, we have four new flags: O_DENYREAD - to prevent other opens with read access, O_DENYWRITE - to prevent other opens with write access, O_DENYDELETE - to prevent delete operations (this flag is not implemented in VFS and NFS part and only suitable for CIFS module), O_DENYMAND - to switch on/off three flags above. The #1 patch bring infrastructure to let us add this capability easier. #2 patch adds flags to fcntl while #3 patch implement VFS part. Patches #4, #5, #6 are related to CIFS-specific changes. #7 and #8 describe NFS and NFSD parts. Also, I created the preliminary patch for Samba that replaces the existing use of flock/LOCK_MAND mechanism with O_DENY* flags: http://git.etersoft.ru/people/piastry/packages/?p=samba.git;a=commitdiff;h=… Pavel Shilovsky (8): locks: make flock_lock_file take is_conflict callback parm fcntl: Introduce new O_DENY* open flags vfs: Add O_DENYREAD/WRITE flags support for open syscall CIFS: Add O_DENY* open flags support CIFS: Use NT_CREATE_ANDX command for forcemand mounts CIFS: Translate SHARING_VIOLATION to -ETXTBSY error code for SMB2 NFSv4: Add O_DENY* open flags support NFSD: Pass share reservations flags to VFS fs/cifs/cifsacl.c | 10 ++-- fs/cifs/cifsglob.h | 12 +++- fs/cifs/cifsproto.h | 9 +-- fs/cifs/cifssmb.c | 47 ++++++++------- fs/cifs/dir.c | 14 +++-- fs/cifs/file.c | 18 ++++-- fs/cifs/inode.c | 11 ++-- fs/cifs/link.c | 10 ++-- fs/cifs/readdir.c | 2 +- fs/cifs/smb1ops.c | 15 ++--- fs/cifs/smb2file.c | 10 ++-- fs/cifs/smb2inode.c | 4 +- fs/cifs/smb2maperror.c | 2 +- fs/cifs/smb2ops.c | 10 ++-- fs/cifs/smb2pdu.c | 6 +- fs/cifs/smb2proto.h | 14 +++-- fs/fcntl.c | 5 +- fs/locks.c | 121 ++++++++++++++++++++++++++++++++++++--- fs/namei.c | 10 +++- fs/nfs/nfs4xdr.c | 24 ++++++-- fs/nfsd/nfs4state.c | 46 ++++++++++++++- include/linux/fs.h | 6 ++ include/uapi/asm-generic/fcntl.h | 14 +++++ 23 files changed, 324 insertions(+), 96 deletions(-) -- 1.8.1.1

3 6

Re: iphlpapi: Let C look like C.
by Austin English 07 Feb '13

07 Feb '13

On Feb 6, 2013 11:13 PM, "Michael Stefaniuc" <mstefani(a)redhat.de> wrote: > > --- > dlls/iphlpapi/ipstats.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/dlls/iphlpapi/ipstats.c b/dlls/iphlpapi/ipstats.c > index f8191ba..966d24e 100644 > --- a/dlls/iphlpapi/ipstats.c > +++ b/dlls/iphlpapi/ipstats.c > @@ -693,7 +693,7 @@ DWORD WINAPI GetIcmpStatisticsEx(PMIB_ICMP_EX stats, DWORD family) > } > > ret = GetIcmpStatistics(&ipv4stats); > - if SUCCEEDED(ret) > + if (SUCCEEDED(ret)) > { > stats->icmpInStats.dwMsgs = ipv4stats.stats.icmpInStats.dwMsgs; > stats->icmpInStats.dwErrors = ipv4stats.stats.icmpInStats.dwErrors; > -- > 1.7.6.5 Minor nitpick, "Make C look like C" would be a more proper title, imho.

3 3

Re: comctl32/tests: drop broken test
by Marvin 06 Feb '13

06 Feb '13

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=24289 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [1/4] windowscodecs: Implement IWICColorContext::InitializeFromFilename.
by Ludger Sprenker 06 Feb '13

06 Feb '13

On Wed, 06 Feb 2013 13:51:32 +0100, Hans Leidekker wrote: > + handle = CreateFileW(filename, GENERIC_READ, 0, NULL, > OPEN_EXISTING, 0, NULL); > + if (handle == INVALID_HANDLE_VALUE) return > HRESULT_FROM_WIN32(GetLastError()); Hello Hans, I think, you are hiding a fixme for the default color profiles shipped with windows. (like "c:\windows\system32\spool\drivers\color\sRGB Color Space Profile.icm") I don't know the correct way to support these files. Maybe wine could search in "/usr/share/color/icc" for compatible profiles on runtime or take these files from other open source projects and include them in the wine installation. Regards, Ludger

2 1

Re: [4/4] windowscodecs: Implement IWICComponentFactory::CreateDecoderFromFileHandle.
by Vincent Povirk 06 Feb '13

06 Feb '13

> + hr = map_file(file, &map, &mem, &size); > + if (FAILED(hr)) return hr; > + ... > + > + hr = IWICStreamImpl_InitializeFromMemory(stream, mem, size.u.LowPart); > + if (FAILED(hr)) goto error; This makes me sad. Is this really simpler than implementing Write and Seek for a file handle?

2 1

kernel32: Use the monotonic counter in GetTickCount64.
by Joerg-Cyril.Hoehle＠t-systems.com 06 Feb '13

06 Feb '13

Hi, I believe we should use this opportunity and review the usage of timers in Wine. E.g. I have a patch to have CreateTimerqueue use the monotonic timer too that I'll submit after some more testing. Mmdevapi's audio currently depends on the TimerQueue which IMHO erroneously depends on gettimeofday() instead of clocks not affected by DST and NTP. As a result, strange things should happen to Wine audio each time the clock is adjusted... What do you think? BTW, MSDN is wonderfully inconsistent - witness "system time": "System time is the current date and time of day. [...] The system bases system time on coordinated universal time (UTC)." http://msdn.microsoft.com/en-us/library/windows/desktop/ms724961%28v=vs.85%… "The system time is the time, in milliseconds, since the Microsoft Windows operating system was started." http://msdn.microsoft.com/en-us/library/windows/desktop/dd743637%28v=vs.85%… The latter is within the context of windows multimedia. I believe different teams at MS wrote documents without cross-checking their use of vocabulary. As a result, expressions don't mean the same things. Regards, Jörg Höhle

1 0

Re: kernel32/tests : added tests for copyfileex try7
by Alexandre Julliard 06 Feb '13

06 Feb '13

Patrick Rudolph <siro(a)das-labor.org> writes: > + if( !GetTempPathA( MAX_PATH,tmppath ) ) > + { > + win_skip("GetTempPathA failed\n"); > + return; > + } > + > + /* create a dummy file for copy tests */ > + if( !GetTempFileNameA( tmppath,"IN",0,source_name ) ) > + { > + win_skip("GetTempFileNameA failed\n"); > + return; > + } > + > + /* create a dummy file for copy tests */ > + if( !GetTempFileNameA( tmppath,"OUT",0,dest_name ) ) > + { > + win_skip("GetTempFileNameA failed\n"); > + return; > + } > + > + source_handle = CreateFileA( source_name, GENERIC_WRITE, 0, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, 0 ); > + if( !source_handle ) > + { > + win_skip("CreateFileA failed (%d)\n", GetLastError()); > + return; > + } > + > +#define COPYFILEEXFILESIZE 10000000 > + > + /* set filesize to COPYFILEEXFILESIZE */ > + TotalFileSize.QuadPart = COPYFILEEXFILESIZE; > + > + if( !pSetFilePointerEx(source_handle,TotalFileSize,NULL,FILE_BEGIN) ) > + { > + win_skip("SetFilePointerEx failed (%d)\n", GetLastError()); > + return; > + } > + > + if( !pSetEndOfFile(source_handle) ) > + { > + win_skip("SetEndOfFile failed (%d)\n", GetLastError()); > + return; > + } All these calls are not supposed to fail on Windows either. win_skip is not appropriate. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] Support a protection scheme which reset the GS selector
by Alexandre Julliard 06 Feb '13

06 Feb '13

Alessandro Pignotti <a.pignotti(a)sssup.it> writes: > Hi everyone, > > I've found two different games: > > -) Of orcs and men > -) The testament of sherlock holmes > > which are using a protection scheme which reset the GS segment selector, > possibly to confuse virtual machines. Since on linux the GS selector is > used internally by glibc the game would crash as soon as a native glibc > method is called by wine. The proposed patch tries to workaround this > problem by restoring the GS to a known working value in wine's SEGFAULS > handler and try the instruction again. This should be done similarly to the other checks in raise_segv_exception. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [2/5] windowscodecs: Basic tests for tiff encoder options (try2)
by Vincent Povirk 05 Feb '13

05 Feb '13

> @Vincent: I thought about adding the properties one at a time but it seemed > a bit like overkill for the tiff encoder, which has only two options. But I > will defenitly do this for the jpeg encoder, which has much more options. Well, actually, on further consideration I can understand why you did it this way, since adding a property is very easy and doesn't really require implementing that property. Though it'd be nice to get some notification when a property is used that we don't support.

1 0

Re: [2/5] windowscodecs: Basic tests for tiff encoder options
by Vincent Povirk 05 Feb '13

05 Feb '13

+ { wszTiffCompressionMethod, VT_UI1, VT_UI1, {.iVal=WICTiffCompressionDontCare} }, This .field=value initialization syntax is not acceptable for Wine (not in C89). I don't know another way to initialize unions. + if (clsid_encoder == &CLSID_WICTiffEncoder) + todo_wine test_specific_encoder_properties(options, testdata_tiff_props, all_props, cProperties2); It might be better to mark the individual properties with a todo, so they can be added one at a time.

1 0

Compiling Wine on Mac OS X 10.8.2?
by Jeremiah Flerchinger 05 Feb '13

05 Feb '13

Is anyone compiling Wine from the git repository for OS X version 10.8.2? At first I thought I broke my toolchain and could no longer compile Wine at all, but this is not the case. I can still compile old releases, but can no longer compile from git. I'm just curious if anyone else has tried...

5 8

Re: comctl32/listview: do not touch icon spacing if set explicitly
by Marvin 04 Feb '13

04 Feb '13

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=24258 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: comctl32/listview: always use large icon size when calculating icon spacing
by Marvin 04 Feb '13

04 Feb '13

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=24257 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: comctl32/listview: correct icon spacing calculation when set to 0 or -1
by Marvin 04 Feb '13

04 Feb '13

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=24256 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [2/5] windowscodecs: Implement IPropertyBag2::CountProperties
by Marvin 04 Feb '13

04 Feb '13

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=24252 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

2 1

Re: [3/5] windowscodecs: Add initializations for tiff encoder options
by Marvin 04 Feb '13

04 Feb '13

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=24254 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: Keynote made me remember Mac Icons
by Ken Thomases 03 Feb '13

03 Feb '13

Hi, On Feb 3, 2013, at 3:56 AM, Austin English wrote: > On Sun, Feb 3, 2013 at 9:45 AM, Jeremiah Flerchinger > <jeremiah.flerchinger(a)gmail.com> wrote: >> The keynote made me realize I have some Mac icons for Wine. Would anyone be >> interested in them? It's definitely more functional than just using the >> default terminal icon. I also had a winecfg icon somewhere but I can't >> remember where that one is right now. >> >> Let me know and tell me where to send them. > > Ken Thomases (cc'ed) is the person on winemac.drv, though wine-devel > or attaching them to bugzilla would likely be best to preserve a copy > of it. I may be missing some context. What icons is this referring to? Do you mean just the various icons for Wine and its programs (e.g. Notepad) but in ICNS format? I'm not sure yet how useful that will be. Although I haven't gotten this far in my Mac driver patch submissions, the Mac driver will eventually attempt to extract the icon for the current .exe and use that for the application icon of the running process. Not all executables have an icon, though, so in some cases it will still default to the terminal-looking icon. It probably would be better to use Wine's wine-glass icon in that case. However, there's a question of how to deploy the icon. I'm not sure a .icns file is useful. I don't know where such a .icns file would be kept and how it would be located relative to the Wine installation (or build directory, if running from there). So, it may just be easiest to embed a Windows ICO resource into the Mac driver .so file and extract it at runtime just like the code will extract the icon from the .exe file. So, thanks for the offer and I'll keep it in mind, but at this point I'm not sure what I would do with Mac icons for Wine. -Ken

2 1

Support for game DRM which overwrite the GS segment selector
by Alessandro Pignotti 03 Feb '13

03 Feb '13

Hi everyone, I'm trying to get a specific game which employs a seemingly custom protection scheme to work. The DRM does various bad things as usual, but a very bad one is manipulating to GS segment selector and setting it to a NULL segment. The GS segment is used by libc though in various ways (stack protection and syscall support, and probably others). I managed to get the activation procedure to go further and further by enclosing each offending syscall using the following 2 macros. #define SAFE_GS_START \ do { \ wine_set_gs(ntdll_get_thread_data()->gs); \ do #define SAFE_GS_END \ while(0); \ } while(0) Still, this method is very cumbersome since system calls happens in many places even outside of ntdll. Fixing the GS is also needed to support sigsetjmp which is used by wine's exception handling. I'd like to ask for feedback about what would be a sane way of supporting this application. A possible solution would be to modify wine's segfault handler to check if the instruction has a GS prefix (0x65 IIRC) and try to execute the instruction again after fixing the GS. Please keep me in CC since I'm not subscribed to the ML. Regards, Alessandro Pignotti

2 3

Re: fusion: return correct error for null public token
by Nikolay Sivov 03 Feb '13

03 Feb '13

> + ok(hr == FUSION_E_PRIVATE_ASM_DISALLOWED, "Expected S_OK, got %08x\n", hr); You could just strip excepted value message part to avoid typos like that. On Fri, Feb 1, 2013 at 8:10 AM, Alistair Leslie-Hughes < leslie_alistair(a)hotmail.com> wrote: > Hi, > The other fusion patch is superseded by this one. > > Changelog: > fusion: return correct error for null public token > > > Best Regards > Alistair Leslie-Hughes > > > > > >

1 0

Re: [PATCH] cryptui: use add_oid_to_usage correctly (Coverity)
by Juan Lang 02 Feb '13

02 Feb '13

Hi Marcus, - add_oid_to_usage(usage, ptr); + usage = add_oid_to_usage(usage, ptr); This looks fine, but would you mind making the same change on line 337? Thanks, --Juan

2 3

Re: kernel32/tests : added tests for copyfileex try7
by Marvin 01 Feb '13

01 Feb '13

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=24239 Your paranoid android. === WNT4WSSP6 (32 bit path) === path: unhandled exception c0000005 at 00000000

1 0

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-devel February 2013