September 2013 - Wine-Devel

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

Nov. 22, 2020

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 Sept. 28, 2018

Sept. 28, 2018

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 June 14, 2018

June 14, 2018

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 June 16, 2016

June 16, 2016

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 July 5, 2015

July 5, 2015

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 Aug. 23, 2014

Aug. 23, 2014

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 Aug. 2, 2014

Aug. 2, 2014

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

Re: Raise the minimum version of FreeBSD to 8.0.
by Frédéric Delanoy Oct. 27, 2013

Oct. 27, 2013

On Sun, Mar 3, 2013 at 12:22 AM, Gerald Pfeifer <gerald(a)pfeifer.com> wrote: > We've had this as a requirement in the FreeBSD ports for quite a > while (and the Wiki describes some background). > > Gerald > --- > - FreeBSD 7.0 or later > + FreeBSD 8.0 or later > ... > FreeBSD info: > - Wine will generally not work properly on versions before FreeBSD 7.0. > + Wine will generally not work properly on versions before FreeBSD 8.0. You should also change the version in the various README.xx translations (in 'documentation' directory). Frédéric

3 2

Re: [PATCH 1/2] advapi32: HKCR merge: foundation
by Alexandre Julliard Oct. 25, 2013

Oct. 25, 2013

George Stephanos <gaf.stephanos(a)gmail.com> writes: > @@ -224,13 +248,138 @@ static NTSTATUS open_key( HKEY *retkey, ACCESS_MASK access, OBJECT_ATTRIBUTES *a > > } > > +static LSTATUS create_hkcr_struct( HKEY *hkey, opened_hkcr_t **hkcr ) > +{ > + UINT_PTR handle = nb_hkcr_handles, i; > + LSTATUS ret = ERROR_SUCCESS; > + > + EnterCriticalSection( &hkcr_handles_cs ); > + > + for (i = 0; i < nb_hkcr_handles; i++) > + { > + if (!hkcr_handles[i]) > + { > + handle = i; > + break; > + } > + } You need some sort of free list instead of a linear search. Also you can probably avoid one level of pointers and store objects directly. Returning a pointer to the object outside of the critical section is not a good idea. > +static HKEY resolve_hkcr( HKEY hkey ) > +{ > + HKEY ret; > + UINT_PTR idx = (UINT_PTR)hkey >> 2; > + opened_hkcr_t *hkcr; > + if (idx <= nb_hkcr_handles) > + { > + EnterCriticalSection( &hkcr_handles_cs ); The count needs to be protected by the critical section too. Also please write a better subject line for the commit. -- Alexandre Julliard julliard(a)winehq.org

2 4

Re: [PATCH try3] atl110: Added new DLL.
by Qian Hong Oct. 16, 2013

Oct. 16, 2013

> +WINE_DEFAULT_DEBUG_CHANNEL(atl); Hi Jacek, we already have a debug channel atl100 for atl100.dll, but we currently use atl for both atl.dll and atl80.dll, do you think it is better to use atl for all, or one debug channel per each dll?

2 5

winspool: ERROR_INVALID_NAME in GetDefaultPrinter() ?
by Detlef Riekenberg Oct. 9, 2013

Oct. 9, 2013

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

winemac.drv: add fullscreen mode support for Mac OS X 10.7+
by Kevin Eaves Oct. 8, 2013

Oct. 8, 2013

Attached patch is a proof-of-concept for the fullscreen APIs provided in OS X 10.7+, which allows applications to enter fullscreen in a separate desktop space. The fullscreen button appears if a main window is resizable. Basically this feature is the zoom button on crack. It doesn't actually go fullscreen 'natively' but simply resizes the window fullscreen, and automatically moves it to its own space. As far as Wine is concerned the window resized — just like clicking the zoom button and manually assigning the window to its own space. I doubled the MaxTrackSize in user32/winpos.c, but it only needs +15 for the height.

2 2

Re: [4/5] ntdll: Make asynchronous NtWriteFile on a disk file always return STATUS_PENDING.
by Alexandre Julliard Oct. 1, 2013

Oct. 1, 2013

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > --- > dlls/ntdll/file.c | 3 ++- > dlls/ntdll/tests/file.c | 5 +++-- > 2 files changed, 5 insertions(+), 3 deletions(-) I don't see the point. Do you actually have an app that depends on this? -- Alexandre Julliard julliard(a)winehq.org

2 6

Increasing max_req_size and subsequent errors
by Hugh McMaster Oct. 1, 2013

Oct. 1, 2013

I need to increase the $max_req_size in tools/make_requests. A new request structure for GetConsoleScreenBufferInfoEx is 80 bytes and primarily made up of 16 unsigned long integers. This is fine, but raises the error: wineserver: request.c:755: open_master_socket: Assertion `sizeof(union generic_reply) == sizeof(struct request_max_size)' failed. It looks like this error is connected to struct request_max_size { int pad[16]; }; in include/wine/server_protocol.h. But other than increasing the size of the array, I'm not sure how to fix this appropriately. Any advice will be appreciated.

1 1

Re: wldap32: Use BOOL type where appropriate
by Rico Schüller Sept. 30, 2013

Sept. 30, 2013

On 01.10.2013 00:06, Frédéric Delanoy wrote: > --- > dlls/wldap32/init.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/dlls/wldap32/init.c b/dlls/wldap32/init.c > index b5aed58..54b5b24 100644 > --- a/dlls/wldap32/init.c > +++ b/dlls/wldap32/init.c > @@ -113,13 +113,13 @@ oom: > } > > /* Determine if a URL starts with a known LDAP scheme */ > -static int has_ldap_scheme( char *url ) > +static BOOL has_ldap_scheme( char *url ) > { > if (!strncasecmp( url, "ldap://", 7 ) || > !strncasecmp( url, "ldaps://", 8 ) || > !strncasecmp( url, "ldapi://", 8 ) || > - !strncasecmp( url, "cldap://", 8 )) return 1; > - return 0; > + !strncasecmp( url, "cldap://", 8 )) return TRUE; > + return FALSE; > } > > /* Flatten an array of hostnames into a space separated string of URLs. > You probably may return the "value" of the if statement directly instead of returning TRUE / FALSE. Cheers Rico

1 0

Re: [3/3] ws2_32: Implement WSASendMsg() (try 2)
by Alexandre Julliard Sept. 30, 2013

Sept. 30, 2013

Bruno Jesus <00cpxxx(a)gmail.com> writes: > @@ -2427,6 +2427,46 @@ static void WINAPI WS2_GetAcceptExSockaddrs(PVOID buffer, DWORD data_size, DWORD > } > > /*********************************************************************** > + * WSASendMsg > + */ > +int WINAPI WSASendMsg( SOCKET s, LPWSAMSG msg, DWORD dwFlags, LPDWORD lpNumberOfBytesSent, > + LPWSAOVERLAPPED lpOverlapped, > + LPWSAOVERLAPPED_COMPLETION_ROUTINE lpCompletionRoutine) > +{ > + int ret = SOCKET_ERROR, fd, sock_type; > + socklen_t optlen = sizeof(sock_type); > + > + if (!msg) > + { > + SetLastError(WSAEFAULT); > + return SOCKET_ERROR; > + } > + > + if ( (fd = get_sock_fd( s, 0, NULL )) == -1) > + return SOCKET_ERROR; > + > + if (getsockopt(fd, SOL_SOCKET, SO_TYPE, &sock_type, &optlen) == -1) > + { > + SetLastError((errno == EBADF) ? WSAENOTSOCK : wsaErrno()); > + goto cleanup; > + } > + > + if (sock_type != SOCK_DGRAM && sock_type != SOCK_RAW) > + { > + SetLastError(WSAEINVAL); > + goto cleanup; > + } > + > + ret = WS2_sendto( s, msg->lpBuffers, msg->dwBufferCount, lpNumberOfBytesSent, > + dwFlags, msg->name, msg->namelen, > + lpOverlapped, lpCompletionRoutine ); This is redundant, WS2_sendto will already fetch the file descriptor, any necessary error checking should happen there. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [2/3] kernel32/tests: test CopyFileEx callback and cancellation (resend)
by Nikolay Sivov Sept. 30, 2013

Sept. 30, 2013

On 9/30/2013 00:51, Daniel Jeliński wrote: > +struct progress_list { > + const DWORD progress_retval_init; /* value to return from progress routine */ > + const BOOL cancel_init; /* value to set Cancel flag to */ > + const DWORD progress_retval_end; /* value to return from progress routine */ > + const BOOL cancel_end; /* value to set Cancel flag to */ > + const DWORD progress_count; /* number of times progress is invoked */ > + const BOOL copy_retval; /* expected CopyFileEx result */ > + const DWORD lastError; /* expected CopyFileEx error code */ > +} ; I don't see a point making them 'const'. > +static DWORD WINAPI progress(LARGE_INTEGER TotalFileSize, > + LARGE_INTEGER TotalBytesTransferred, > + LARGE_INTEGER StreamSize, > + LARGE_INTEGER StreamBytesTransferred, > + DWORD dwStreamNumber, > + DWORD dwCallbackReason, > + HANDLE hSourceFile, > + HANDLE hDestinationFile, > + LPVOID lpData) > +{ > + progressInvoked++; Please pass all globals as context data with lpData, and please use 'void*' instead of LPVOID.

3 3

Re: atl90: Add new DLL
by Zhenbo Li Sept. 30, 2013

Sept. 30, 2013

2013/9/4 Zhenbo Li <litimetal(a)gmail.com>: > This patch could fix bug 34431, which is caused by missing atl90.dll > > --- > configure | 1 + > configure.ac | 1 + > dlls/atl90/Makefile.in | 7 ++++ > dlls/atl90/atl90.c | 93 > ++++++++++++++++++++++++++++++++++++++++++++++++++ > dlls/atl90/atl90.spec | 52 ++++++++++++++++++++++++++++ > include/atlbase.h | 1 + > tools/make_specfiles | 1 + > 7 files changed, 156 insertions(+) Excuse me, this is my first time to submit a patch to Wine. And I don't know that if I had made some mistakes. Would someone please review my patch and tell me what to do? Thank you. -- Have a nice day! Zhenbo Li

1 0

Re: [PATCH 1/2] setupapi: Also look in datadir/fakedlls for the re-compiled fakedlls.
by Alexandre Julliard Sept. 30, 2013

Sept. 30, 2013

Huw Davies <huw(a)codeweavers.com> writes: > --- > dlls/setupapi/fakedll.c | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) I don't think that's correct. The datadir is for platform-independent files, but the fakedlls are platform specific. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: mmdevapi/tests: test MMDevPropStore_GetAt
by Marvin Sept. 30, 2013

Sept. 30, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2403 Your paranoid android. === wvista (32 bit propstore) === propstore.c:108: Test failed: DEVPKEY_Device_FriendlyName not found

1 0

Re: kernel32/time: Implement GetSystemTimes for linux.
by Bernard Ladenthin Sept. 29, 2013

Sept. 29, 2013

Am 29.09.2013 20:08, schrieb André Hentschel: > Am 28.09.2013 21:51, schrieb Bernard Ladenthin: >> --- >> dlls/kernel32/time.c | 149 >> ++++++++++++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 149 insertions(+) > > > Hi, > please don't use C++ comments, also the rest of the coding style isn't often seen in Wine... > (e.g. missing space between if/for/... and the bracket, your way of variable definig/initializing, most likely more) > > maybe you also want to change the FIXME into something like "not implemented for this platform" and add a TRACE for the linux way. > > > Hi, thanks for the feedback, I will write a new patch. Regards, Bernard

1 0

Re: add ualapi directory (try 12)
by Bruno Jesus Sept. 29, 2013

Sept. 29, 2013

I still don't understand why the copyright is different for the .c and .h file. In one file you add your first name only and in the other your full name. Regards, Bruno On Sat, Sep 28, 2013 at 4:14 AM, Akira Nakagawa <matyapiro31(a)gmail.com> wrote: > > > > > > -- universe* god::bigbang (void); //and then it all began...

1 0

Re: [3/3] ws2_32: Implement WSASendMsg() (try 2)
by Marvin Sept. 28, 2013

Sept. 28, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2393 Your paranoid android. === wxppro (32 bit sock) === sock.c:519: Test failed: oob_server (1d0): unexpectedly at the OOB mark: 0 === w2008s64 (32 bit sock) === sock.c:519: Test failed: oob_server (5a0): unexpectedly at the OOB mark: 0 === w2008s64 (64 bit sock) === sock.c:509: Test failed: oob_server (b64): unexpectedly at the OOB mark: 0 sock.c:519: Test failed: oob_server (b64): unexpectedly at the OOB mark: 0 === w7pro64 (64 bit sock) === sock.c:509: Test failed: oob_server (650): unexpectedly at the OOB mark: 0 sock.c:519: Test failed: oob_server (650): unexpectedly at the OOB mark: 0 sock.c:288: Test failed: do_synchronous_recv (650): error 10054: sock.c:534: Test failed: oob_server (650): not at the OOB mark: 1

1 0

Re: [1/3] ws2_32/tests: Fix an ok() call comment
by Marvin Sept. 28, 2013

Sept. 28, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2392 Your paranoid android. === wxppro (32 bit sock) === sock.c:519: Test failed: oob_server (748): unexpectedly at the OOB mark: 0 === w2008s64 (32 bit sock) === sock.c:519: Test failed: oob_server (998): unexpectedly at the OOB mark: 0 sock: unhandled exception c0000005 at 00000000 === w7pro64 (32 bit sock) === sock.c:509: Test failed: oob_server (2f0): unexpectedly at the OOB mark: 0 sock.c:519: Test failed: oob_server (2f0): unexpectedly at the OOB mark: 0 sock.c:288: Test failed: do_synchronous_recv (2f0): error 10054: sock.c:534: Test failed: oob_server (2f0): not at the OOB mark: 1 === w2008s64 (64 bit sock) === sock.c:509: Test failed: oob_server (938): unexpectedly at the OOB mark: 0 sock.c:519: Test failed: oob_server (938): unexpectedly at the OOB mark: 0 sock: unhandled exception c0000005 at 000007FEFE229CD9

1 0

Re: [website] templates: More protocol relative urls
by Jactry Zeng Sept. 28, 2013

Sept. 28, 2013

Hi André, This patch seem let some urls in non-english version broken. Such as some urls in this page in Chinese: http://www.winehq.org/site/forums >- [<a href="http://www.winehq.org/mailman/listinfo/wine-announce ">（取消）订阅</a>] >- [<a href="http://www.winehq.org/pipermail/wine-announce/">存档 1</a>] >+ [<a href=//www.winehq.org/mailman/listinfo/wine-announce">（取消）订阅</a>] >+ [<a href=//www.winehq.org/pipermail/wine-announce/">存档 1</a>] You will find there is a quotation mark in the end of some urls in this page. We may need to remove the / in the end? >-<p> Wine 将会永远是<a href="http://www.winehq.org/license"title="Wine licensing">自由软件</a>。 >+<p> Wine 将会永远是<a href=//www.winehq.org/license"title="Wine licensing">自由软件</a>。 And the about page in Chinese, the urls of "自由软件" (Free Software) also been broken. There seem need a space or a enter between the url and 'title'. Thanks for your work! 2013/9/23 André Hentschel <nerv(a)dawncrow.de> > --- > templates/he/about.template | 2 +- > templates/he/forums.template | 46 > +++++++++++++++++++-------------------- > templates/he/help.template | 8 +++---- > templates/he/home.template | 4 ++-- > templates/he/license.template | 6 +++--- > templates/he/resources.template | 6 +++--- > templates/pl/forums.template | 2 +- > templates/pt/about.template | 4 ++-- > templates/pt/forums.template | 48 > ++++++++++++++++++++--------------------- > templates/pt/help.template | 8 +++---- > templates/pt/home.template | 4 ++-- > templates/zh-cn/about.template | 6 +++--- > templates/zh-cn/forums.template | 44 ++++++++++++++++++------------------- > templates/zh-cn/help.template | 8 +++---- > templates/zh-cn/home.template | 4 ++-- > templates/zh-cn/irc.template | 4 ++-- > 16 files changed, 102 insertions(+), 102 deletions(-) > > > > > > -- Regards, Jactry Zeng

1 0

Re: add ualapi directory (try 11)
by Marcus Meissner Sept. 28, 2013

Sept. 28, 2013

On Sat, Sep 28, 2013 at 11:58:33AM +0900, Akira Nakagawa wrote: > > > > From 2ecc71fdb3406d3cf79ebcbd7f6d0c711b68d699 Mon Sep 17 00:00:00 2001 > Message-Id: <2ecc71fdb3406d3cf79ebcbd7f6d0c711b68d699.1379923234.git.matyapiro31(a)gmail…> > From: Akira Nakagawa <matyapiro31(a)gmail.com> > Date: Mon, 23 Sep 2013 17:00:04 +0900 > Subject: [PATCH] add ual directory (try9) > > --- > configure | 2 + Not needed to be submitted > configure.ac | 2 + also not actually needed as these parts are autogenerated. > dlls/ualapi/Makefile.in | 6 +++ > dlls/ualapi/main.c | 86 +++++++++++++++++++++++++++++++++++++++++ > dlls/ualapi/tests/Makefile.in | 8 ++++ > dlls/ualapi/tests/ualapi_test.c | 38 ++++++++++++++++++ You have no working tests yet, so leave these two files out. Add them with the patch with the first tests. > dlls/ualapi/ualapi.spec | 4 ++ > include/Makefile.in | 1 + include/Makefile.in is also autoadjusted. > include/ual.h | 48 +++++++++++++++++++++++ > 9 files changed, 195 insertions(+) rest looks ok, but you really only should submit: dlls/ualapi/Makefile.in dlls/ualapi/main.c dlls/ualapi/ualapi.spec include/ual.h as the rest are autogenerated. Ciao, Marcus

2 1

Re: add ualapi directory (try 12)
by Marvin Sept. 28, 2013

Sept. 28, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2391 Your paranoid android. === build (build) === Missing build status line The build timed out

1 0

Re: [2/2] ws2_32: Implement WSASendMsg()
by Marvin Sept. 28, 2013

Sept. 28, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2388 Your paranoid android. === wxppro (32 bit sock) === sock.c:519: Test failed: oob_server (1c4): unexpectedly at the OOB mark: 0 === w7pro64 (32 bit sock) === sock.c:509: Test failed: oob_server (870): unexpectedly at the OOB mark: 0 sock.c:519: Test failed: oob_server (870): unexpectedly at the OOB mark: 0 sock.c:288: Test failed: do_synchronous_recv (870): error 10054: sock.c:534: Test failed: oob_server (870): not at the OOB mark: 1

1 0

Re: [1/2] ws2_32/tests: Fix an ok() call comment
by Marvin Sept. 28, 2013

Sept. 28, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2387 Your paranoid android. === wxppro (32 bit sock) === sock.c:519: Test failed: oob_server (408): unexpectedly at the OOB mark: 0 === w2008s64 (32 bit sock) === sock.c:509: Test failed: oob_server (bb8): unexpectedly at the OOB mark: 0 sock.c:519: Test failed: oob_server (bb8): unexpectedly at the OOB mark: 0 === w7pro64 (32 bit sock) === sock.c:528: Test failed: oob_server (6f0): not at the OOB mark: 1 === w2008s64 (64 bit sock) === sock.c:519: Test failed: oob_server (940): unexpectedly at the OOB mark: 0

1 0

Re: [2/2] ws2_32: Implement WSASendMsg()
by Bruno Jesus Sept. 28, 2013

Sept. 28, 2013

On Sat, Sep 28, 2013 at 1:50 AM, Bruno Jesus <00cpxxx(a)gmail.com> wrote: > Fixes bug 34005. There is a problem with this patch, I'm working on an updated version. Sorry, Bruno

1 0

Re: add ualapi directory (try 11)
by Marvin Sept. 28, 2013

Sept. 28, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2386 Your paranoid android. === build (build) === Make failed

1 0

Re: (try 6)[5/5] imm32: limit cross thread access to ImmSet* functions
by Marvin Sept. 28, 2013

Sept. 28, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2377 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: (try 6)[4/5] imm32: Restrict crossthread Association and destruction
by Marvin Sept. 28, 2013

Sept. 28, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2376 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: (try 6)[3/5] imm32: use thread data from target HWND
by Marvin Sept. 27, 2013

Sept. 27, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2375 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH 2/2] mshtml: Added IHTMLFormElement::sumit tests.
by Marvin Sept. 27, 2013

Sept. 27, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2374 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH 1/2] appwiz.cpl: Use the same dialog template for Mono and Gecko downloader.
by Vincent Povirk Sept. 27, 2013

Sept. 27, 2013

Isn't this going to make the caption "Wine Gecko Installer" in both cases?

1 0

Re: add ualapi directory (try 10)
by Marvin Sept. 27, 2013

Sept. 27, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2370 Your paranoid android. === build (build) === Make failed

1 0

Re: [PATCH 1/5] wined3d: Send present operations through a command stream.
by Stefan Dösinger Sept. 27, 2013

Sept. 27, 2013

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Am 2013-09-27 00:15, schrieb Henri Verbeet: > - swapchain->swapchain_ops->swapchain_present(swapchain, > src_rect, dst_rect, dirty_region, flags); + > wined3d_cs_emit_present(swapchain->device->cs, swapchain, > src_rect, + dst_rect, dst_window_override, dirty_region, > flags); I think we should place comments like "This function is called through the command stream" for all functions that are called through the CS, similar to what we do for context activation. Also note that swapchain_gl_present contains calls to wined3d_surface_blt and wined3d_device_clear that are going to cause problems with the multithreaded CS. I have patches to remove them, so feel free to ignore this problem for now. -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.20 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJSRWaKAAoJEN0/YqbEcdMw9VgP/i2n/G2CHOA8XShC8CZnRzSq TNzbfIXwAQljQYtf66U98JmbMIoeS3SMPaJ4O1+mMiT062Udi3YddiTk2AVF9bi+ f6X4y9W8Q3jPolcuTxKpZ26/WFkViHtPEUtFaa5oBtksCNGZ0b3A4oK9zbrPBram EXwovsq649RqCoBf76aidtbypdkFKLfmC6P6KxtjB3C9cdKJ58kMD0YbVsua6zNc g8gUgExEQ0no9WEyWPtlN7PRf3tQlLtuUeqrRockqnK48cKbwk3VaEgfq8o6+INs EvgjoZvc2C/+YoEWUT+n7nCtIKo9IiMSG20Ey0sbmak/K/LrvZvsyHl78nQD4ss+ t5jRCS9QZs/VzJc0VakGsZARP9jGBvzapsrAcy6Ittu1d53fpji6ySwKeUrPKim+ wMtWGtu+q0FXtIRtgDYUOofd9b1o1sB3xPqUIpRm7+yBRpXiBvuGqbmbsa/14wBe vkI5b5S/djskw9M5bV1lAOCgGmo+zFIYyebxwvx8H+sqz4HmYql9uPLTOHqWvHtn J7iXcETOlcOrC+zOOT8nieR8YYuFcda51auvK5lnXtCTQvoQse5egV16x2hJkkjx Pujit58jX2/O0WS0hqR5oAfhYtPydqUf2BayXgkLUODiZa0z1UWD2eerlOvGSvcq AUs+vi2QpDjeWmQH37BZ =QcW+ -----END PGP SIGNATURE-----

2 3

Re: [1/3] gdi32: Return fake BBox when requested empty glyph metrics. (try 3)
by Alexandre Julliard Sept. 27, 2013

Sept. 27, 2013

Akihiro Sagawa <sagawa.aki(a)gmail.com> writes: > Fixes bug 18440, 20847. > > Try 3: > - Rebase to recent source. > - Separate ABC metrics change. > - Use conditional operator. It breaks the tests: ../../../tools/runtest -q -P wine -M gdi32.dll -T ../../.. -p gdi32_test.exe.so dib.c && touch dib.ok dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 1 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 1 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 1 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 1 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 1 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 0 dib.c:2827: Test failed: hash mismatch - aa 0 make[1]: *** [dib.ok] Error 19 -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 1/3] riched20: Use codepage in ME_ToUnicode. (try 5)
by Nikolay Sivov Sept. 27, 2013

Sept. 27, 2013

On 9/13/2013 13:27, Jactry Zeng wrote: > Re-based to latest git. > > Does it actually accept any other value besides CP_ACP or 1200? Documentation is unclear here, and makes impression that only two these values are valid.

2 2

Re: ntdll: Make asynchronous WaitCommEvent report correct number of bytes returned.
by Marvin Sept. 27, 2013

Sept. 27, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2356 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [2/3] gdi32: Return fake BBox when requested empty glyph metrics. (try 2)
by Marvin Sept. 27, 2013

Sept. 27, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2351 Your paranoid android. === w7u (32 bit font) === The test timed out

1 0

Re: [3/3] gdi32: Don't modify output glyph metrics unless the function succeeds. (try 2)
by Marvin Sept. 26, 2013

Sept. 26, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2352 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [1/3] gdi32: Return GDI_ERROR when requested empty glyph bitmaps. (try 2)
by Marvin Sept. 26, 2013

Sept. 26, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2350 Your paranoid android. === w7u (32 bit font) === No test summary line found Failure running script in VM: network read timed out

1 0

Re: [PATCH 1/2] gdi32: fix for rotated Arc, ArcTo, Chord and Pie drawing problem (try 2)
by Alexandre Julliard Sept. 26, 2013

Sept. 26, 2013

Daniel Wendt <daniel.wendt(a)linux.com> writes: > @@ -308,6 +308,11 @@ static int get_arc_points( PHYSDEV dev, const RECT *rect, POINT start, POINT end > } > > memmove( points, points + count, (pos - count) * sizeof(POINT) ); > + > + if (GetGraphicsMode( dev->hdc ) == GM_ADVANCED) > + { > + LPtoDP( dev->hdc, points, pos - count ); > + } > return pos - count; > } > > @@ -322,7 +327,17 @@ static BOOL draw_arc( PHYSDEV dev, INT left, INT top, INT right, INT bottom, > BOOL ret = TRUE; > HRGN outline = 0, interior = 0; > > - if (!get_pen_device_rect( pdev, &rect, left, top, right, bottom )) return TRUE; > + if (GetGraphicsMode(dev->hdc ) != GM_ADVANCED) > + { > + if (!get_pen_device_rect( pdev, &rect, left, top, right, bottom )) return TRUE; > + } > + else > + { > + rect.left = left; > + rect.top = top; > + rect.right = right; > + rect.bottom = bottom; > + } I don't think that you want to draw arcs in logical space, that can be very inefficient. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 3/3] msvcrt: Call MSVCRT_strtoi64_l in strtoul implementation (try2)
by Marvin Sept. 26, 2013

Sept. 26, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2349 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: imm32: ImmGetDescriptionA return does not include NULL byte
by Alexandre Julliard Sept. 26, 2013

Sept. 26, 2013

Aric Stewart <aric(a)codeweavers.com> writes: > @@ -1684,7 +1684,7 @@ UINT WINAPI ImmGetDescriptionA( > > HeapFree( GetProcessHeap(), 0, buf ); > > - return len; > + return min( len - 1, uBufLen ); > } This may do the right thing on failure, but in an ugly way. A few overflow tests would be a good idea. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 1/6] riched20: Implement ITextDocument_fnOpen.
by Alexandre Julliard Sept. 26, 2013

Sept. 26, 2013

Jactry <jactry92(a)gmail.com> writes: > + chBuffer = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, size+2); > + if(!chBuffer) > + { > + CloseHandle(hFile); > + return HRESULT_FROM_WIN32(GetLastError()); > + } It's not useful to zero the buffer since you are reading into it. Also GetLastError won't do anything useful here. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [2/3] gdi32: Return fake BBox when requested empty glyph metrics. (try 2)
by Alexandre Julliard Sept. 26, 2013

Sept. 26, 2013

Akihiro Sagawa <sagawa.aki(a)gmail.com> writes: > @@ -6519,11 +6519,17 @@ static DWORD get_glyph_outline(GdiFont *incoming_font, UINT glyph, UINT format, > } > > lpgm->gmBlackBoxX = (right - left) >> 6; > + if (lpgm->gmBlackBoxX == 0) > + lpgm->gmBlackBoxX = 1; > lpgm->gmBlackBoxY = (top - bottom) >> 6; > + if (lpgm->gmBlackBoxY == 0) > + lpgm->gmBlackBoxY = 1; > lpgm->gmptGlyphOrigin.x = origin_x >> 6; > lpgm->gmptGlyphOrigin.y = origin_y >> 6; > abc->abcA = left >> 6; > abc->abcB = (right - left) >> 6; > + if (abc->abcB == 0) > + abc->abcB = 1; > abc->abcC = adv - abc->abcA - abc->abcB; It would be a good idea to add tests for the ABC metrics too. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: (try 5)[1/4] imm32: Move thread data from TLSEntry to an internal list
by Alexandre Julliard Sept. 26, 2013

Sept. 26, 2013

Aric Stewart <aric(a)codeweavers.com> writes: > @@ -530,12 +591,18 @@ static BOOL CALLBACK _ImmAssociateContextExEnumProc(HWND hwnd, LPARAM lParam) > */ > BOOL WINAPI ImmAssociateContextEx(HWND hWnd, HIMC hIMC, DWORD dwFlags) > { > + IMMThreadData* thread_data = NULL; > + > TRACE("(%p, %p, 0x%x):\n", hWnd, hIMC, dwFlags); > > - if (!IMM_GetThreadData()->defaultContext) > - IMM_GetThreadData()->defaultContext = ImmCreateContext(); > + thread_data = IMM_GetInitializedThreadData(); > + if (!thread_data) > + return FALSE; > > - if (!hWnd) return FALSE; > + LeaveCriticalSection(&threaddata_cs); > + > + if (!hWnd) > + return FALSE; > > switch (dwFlags) > { > @@ -543,7 +610,7 @@ BOOL WINAPI ImmAssociateContextEx(HWND hWnd, HIMC hIMC, DWORD dwFlags) > ImmAssociateContext(hWnd,hIMC); > return TRUE; > case IACE_DEFAULT: > - ImmAssociateContext(hWnd,IMM_GetThreadData()->defaultContext); > + ImmAssociateContext(hWnd,thread_data->defaultContext); > return TRUE; > case IACE_IGNORENOCONTEXT: > if (GetPropW(hWnd,szwWineIMCProperty)) You shouldn't be accessing thread data outside of the critical section. -- Alexandre Julliard julliard(a)winehq.org

1 0