August 2008 - Wine-Devel - list.winehq.org

Re: kernel32: Tiny improvement to the GetVolumePathNameW stub (try 4)
by James Hawkins Aug. 21, 2008

Aug. 21, 2008

On Mon, Aug 18, 2008 at 4:18 AM, Louis. Lenders <xerox_xerox2000(a)yahoo.co.uk> wrote: > (the return TRUE statement got lost from last patch, thanks Michael for > noticing) > > > Hi, this fixes Adobe Lightroom 2.0 start up bug, mentioned in > http://bugs.winehq.org/show_bug.cgi?id=8224#c4 > > as long wine doesn't handle with volume mount points, it's probably safe to > satisfy most apps. > > (A simple google for > "fixme:volume:GetVolumePathNameW" > revealed only about 3 apps , and "fixme:volume:GetVolumePathNameA" revealed > zero hits, so for now the use of this api seems not really wide-spread, and > no need to touch GetVolumePathNameA i guess) > + /* As long as we don't handle volumemountpoints, and filename is something like "X:\\blabla\\bla", it's probably safe to return "X:\" for + now to keep some applications happy */ + Please get rid of this comment. + if(strlenW(filename)>=3 && filename[1] == ':' && filename[2] == '\\' && buflen >3) + { + lstrcpynW(volumepathname,filename,4); + return TRUE; + } You've handled *a* successful case, but what about the error case? Also, you're copying 4 bytes of filename into volumepathname. I don't think you understand what lstrcpyn does. Imagine this case: volumepathname = "aaaaaaaa" buflen = 8 filename = "C:\\file" After the call to lstrcpyn: volumepathname = "C:\\faaaa" -- James Hawkins

3 3

Re: kernel32: Tiny improvement to the GetVolumePathNameW stub (try 4)
by Louis. Lenders Aug. 21, 2008

Aug. 21, 2008

----- Original Message ---- From: James Hawkins <truiken(a)gmail.com> To: Louis. Lenders <xerox_xerox2000(a)yahoo.co.uk> Cc: wine-devel(a)winehq.org Sent: Thursday, 21 August, 2008 3:55:07 PM Subject: Re: kernel32: Tiny improvement to the GetVolumePathNameW stub (try 4) >That whole paragraphs reads "hack now, fix later", which is definitely >not acceptable. Well, I would call it a "semi-stub", there's more of them scattered all over the wine-source. But if opinions are it's too hackish, that's fine with me, I was only trying to help a user to get an app running > > + { > + lstrcpynW(volumepathname,filename,4); > + return TRUE; > + } > > >>Also, you're copying 4 bytes of filename into volumepathname. I don't >>think you understand what lstrcpyn does. Imagine this case: > > volumepathname = "aaaaaaaa" > buflen = > 8 > filename = "C:\\file" > > After the call to lstrcpyn: > > volumepathname = "C:\\faaaa" > > > > MSDN says volumepathname was a pointer to an "out" - string, so how would it > become someting > like "aaaaaa", or am i overseeing something? > Yes you are overseeing something. I think it would be a good idea if you read up a bit more on C in your spare time. I get the feeling you don't quite have a grasp on the intricacies of the language, particularly pointers and strings. What I meant is, why would an app set volumepathname to "aaaaaa", as it's the pointer where it expects to find the volumepoint, so on windows this would be filled by kernel32.dll, not by the app. Maybe you mean this could be the case after a second call to GetVolumePathName? Anyway, this is not the place to discuss a C-language course so I'll reread the docs. Thanks for the comments anyway. Send instant messages to your online friends http://uk.messenger.yahoo.com

2 1

re: [6/6][PATCH]Updates WSAIoctl in ws2_32 to return AcceptEx and GetAcceptExSockaddrs
by Dan Kegel Aug. 21, 2008

Aug. 21, 2008

slindeneau at gmail.com wrote: > This patch will allow the patch [1/6] to successfully complete on wine > and windows. Prior to this patch the conformance tests submitted in > [1/6] of this series will fail on wine. It's exciting to see such progress on AcceptEx! But the wine project likes to have all tests passing after each patch in a series. If patch 1/6 adds tests that fail on Wine, they need to be marked with todo_wine. If patch 6/6 fixes the problem in wine that caused those tests to fail, it should also remove the todo_wine from the test. Following this rule will help make sure your patches don't turn red on http://kegel.com/wine/patchwatcher/results/

2 1

Re:kernel32: Tiny improvement to the GetVolumePathNameW stub (try 4)
by Louis. Lenders Aug. 21, 2008

Aug. 21, 2008

>You've handled *a* successful case, but what about the error case? As i said already in the mail with the patch, it might be *a* case, but this is the case that most apps will follow as far as i can see. Futhermore, when apps are really crashing into the other cases, it can be added later on i.m.o. For now they will just fall back to the current "implementation", returning "not implmented" then, so there will be no differnce with current behaviour. + { + lstrcpynW(volumepathname,filename,4); + return TRUE; + } >Also, you're copying 4 bytes of filename into volumepathname. I don't >think you understand what lstrcpyn does. Imagine this case: volumepathname = "aaaaaaaa" buflen = 8 filename = "C:\\file" After the call to lstrcpyn: volumepathname = "C:\\faaaa" MSDN says volumepathname was a pointer to an "out" - string, so how would it become someting like "aaaaaa", or am i overseeing something? Send instant messages to your online friends http://uk.messenger.yahoo.com

2 1

Re:shobjidl.idl: add Taskbarlist interface defenitions
by Louis. Lenders Aug. 21, 2008

Aug. 21, 2008

>The "I'll send the rest later" is a warning light. You should further develop this and send it all later so >we know this is actually going somewhere. The whole patch, that lets Governor of Poker start i have attached to the bug http://bugs.winehq.org/attachment.cgi?id=15297 a few weeks ago. As i said before , it's enough to stub out interface calls like in that patch to let the game start I discovered later I put the headers in the wrong (seperate ) header file, so i thought first to send in the definitions in the correct header. I thought later on send the rest in small pieces thanks again for the review Send instant messages to your online friends http://uk.messenger.yahoo.com

1 0

Re:activeds: add stub for ADsGetLastError
by Louis. Lenders Aug. 21, 2008

Aug. 21, 2008

Thanks for reviewing the patch James @@ -6,7 +6,7 @@ 8 stub ADsBuildVarArrayInt 9 stdcall ADsOpenObject(wstr wstr wstr long ptr ptr) 12 stub ADsSetLastError -13 stub ADsGetLastError +13 stdcall ADsGetLastError(ptr wstr long wstr long) >Both of those wstr parameters are out pointers so they should be ptr, not wstr. Didn't know that, so that has to be changed for sure /***************************************************** + * ADsGetLastError [ACTIVEDS.13] + */ +HRESULT WINAPI ADsGetLastError(LPDWORD pError, LPWSTR buffer, DWORD buflen, LPWSTR bufname, DWORD bufnamelen) +{ + FIXME("(%p,%p,%d,%p,%d)stub\n", pError, buffer, buflen, bufname, bufnamelen); + return E_POINTER; +} >Returning E_POINTER seems pretty wrong. In the documentation on http://msdn.microsoft.com/en-us/library/aa772183(VS.85).aspx it is listed as one of the possible return values. Of course one could also return E_NOTIMPL or E_FAIL etc, but for a stub this seemed like a good choice to me as well, the application (Dameware) doesn't seem to bother what is returned, it starts up just fine Send instant messages to your online friends http://uk.messenger.yahoo.com

2 1

Re: shobjidl.idl: add Taskbarlist interface defenitions
by James Hawkins Aug. 21, 2008

Aug. 21, 2008

On Wed, Aug 20, 2008 at 3:24 AM, Louis. Lenders <xerox_xerox2000(a)yahoo.co.uk> wrote: > Hi , this interface is needed to get governor if poker running > (http://bugs.winehq.org/show_bug.cgi?id=14139) these first 3 patches is > only preliminary stuff, the rest of the stuff i'll send later on. > The "I'll send the rest later" is a warning light. You should further develop this and send it all later so we know this is actually going somewhere. -- James Hawkins

1 0

Re: [2/6][PATCH]Fixes an Async bug.
by Alexandre Julliard Aug. 21, 2008

Aug. 21, 2008

"Scott Lindeneau" <slindeneau(a)gmail.com> writes: > Fixes a bug in the async implementation. When checking for waiting > elements on a queue you need to check to see if ANY element is waiting, > not just the first element. When waking elements up you should ALERT an > element that is not already alerted. All other status messages should be > set. (This compensates for poll events that trigger other poll events > before garbage collection occurs and removes alerted elements from the > queues). This is wrong, it will lead to busy waiting. We very much want the first async to have been handled before we wake up for the next one. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: activeds: add stub for ADsGetLastError
by James Hawkins Aug. 21, 2008

Aug. 21, 2008

On Tue, Aug 19, 2008 at 4:14 PM, Louis. Lenders <xerox_xerox2000(a)yahoo.co.uk> wrote: > Fixes a crash in Dameware utilities > @@ -6,7 +6,7 @@ 8 stub ADsBuildVarArrayInt 9 stdcall ADsOpenObject(wstr wstr wstr long ptr ptr) 12 stub ADsSetLastError -13 stub ADsGetLastError +13 stdcall ADsGetLastError(ptr wstr long wstr long) Both of those wstr parameters are out pointers so they should be ptr, not wstr. /***************************************************** + * ADsGetLastError [ACTIVEDS.13] + */ +HRESULT WINAPI ADsGetLastError(LPDWORD pError, LPWSTR buffer, DWORD buflen, LPWSTR bufname, DWORD bufnamelen) +{ + FIXME("(%p,%p,%d,%p,%d)stub\n", pError, buffer, buflen, bufname, bufnamelen); + return E_POINTER; +} Returning E_POINTER seems pretty wrong. -- James Hawkins

1 0

patches
by Louis. Lenders Aug. 21, 2008

Aug. 21, 2008

Hi, apparently none of my patches is making it into git, nor did i recieve any response from Alexandre about what's wrong with them. I guess i'm kind of "blacklisted" because of my dxdiag patch from a few months ago, where i took some code from a directX tutorial ( i thought tutorials were free code, well they aren't apparently). Anyway, there are i think some patches that contain useful things to fix some bugs, so if anyone would like to pick them up, feel free (espesially the fix for the google lively download) (@Alexandre: this is not a request from me to explain why the patches have been rejected. I don't want to be pissed off again by you like in your last mail; I may be a crappy coder, but i've been helping wine-users for a few years now, so i think i deserved a little more respect , than was reflected by that mail) Send instant messages to your online friends http://uk.messenger.yahoo.com

1 0

Re: [1/6][PATCH]AcceptEx Conformance Test
by James Hawkins Aug. 21, 2008

Aug. 21, 2008

On Thu, Aug 21, 2008 at 12:59 PM, Scott Lindeneau <slindeneau(a)gmail.com> wrote: > Simple overlapped server based on AcceptEx added to the test list > using simple clients. > Tests must always 'pass'. That is, the individual ok tests must all pass, or if they fail you must wrap the failing tests in todo_wine blocks. Since your test was sent before the fixes and there are no todo_wine's, I'm assuming some of these tests fail. -- James Hawkins

1 0

Re: Spore patch to fix UI and Part descriptions
by Marcus Meissner Aug. 21, 2008

Aug. 21, 2008

> > diff --git a/dlls/d3d9/device.c b/dlls/d3d9/device.c > index ddeaca3..dfbf818 100644 > --- a/dlls/d3d9/device.c > +++ b/dlls/d3d9/device.c > @@ -746,7 +746,8 @@ static HRESULT WINAPI IDirect3DDevice9Impl_GetDepthStencilSurface(LPDIRECT3DDE > } > } else { > WARN("Call to IWineD3DDevice_GetDepthStencilSurface failed\n"); > - *ppZStencilSurface = NULL; > + //*ppZStencilSurface = NULL; > + return D3DERR_INVALIDCALL; > } > LeaveCriticalSection(&d3d9_cs); You might cause a locking problem here, returning with d3d9_cs held. Ciao, Marcus

1 0

LinuxJournal poll: What do you use to run Windows applications on your Linux desktop?
by Dan Kegel Aug. 21, 2008

Aug. 21, 2008

Don't all go out and vote - we don't want to skew the results - but check out http://www.linuxjournal.com/node/1007221/results What do you use to run Windows applications on your Linux desktop? August 19th, 2008 by LJ Staff Cedega 1% (8 votes) Crossover 4% (38 votes) VirtualBox 22% (195 votes) VMWare 14% (129 votes) Wine 37% (333 votes) Other 2% (21 votes) I don't need or want to run Windows apps 19% (167 votes) Wine comes out on top, but not yet majority. - Dan

1 0

WSAIoctl question.
by Scott Lindeneau Aug. 21, 2008

Aug. 21, 2008

I noticed a patch that was submitted by John Griffiths on the 13th of July that forwards functions by mwsock.c using the following code: (commit: 2da98052d90591474c65bed853ca75e1da714823) +static void get_fn(SOCKET s, GUID* guid, FARPROC* fn) +{ + FARPROC func; + DWORD len; + if (!WSAIoctl(s, SIO_GET_EXTENSION_FUNCTION_POINTER, guid, sizeof(*guid), + &func, sizeof(func), &len, NULL, NULL)) + *fn = func; +} But wine's WSAIoctl does not support the SIO_GET_EXTENSION_FUNCTION_POINTER. I was wondering how this works and what its for? Is it for using native libraries?

1 0

Re: [PATCH] mstask: Task PersistFile AddRef, QueryInterface, and Release.
by Alexandre Julliard Aug. 21, 2008

Aug. 21, 2008

Roy Shea <royshea(a)gmail.com> writes: > + TaskImpl *This = (TaskImpl*) > + ((char*)(iface) - offsetof(TaskImpl, persistVtbl)); Please define an inline function for this like many other interfaces do. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: dpnet: Add stubbed DirectPlay8Peer interface (try2)
by Alexandre Julliard Aug. 21, 2008

Aug. 21, 2008

Alexander Nicolaysen Sørnes <alex(a)thehandofagony.com> writes: > HRESULT DPNET_CreateDirectPlay8Peer(LPCLASSFACTORY iface, LPUNKNOWN punkOuter, REFIID riid, LPVOID *ppobj) { > - WARN("(%p, %s, %p): stub.\n", punkOuter, debugstr_guid(riid), ppobj); > - return CLASS_E_CLASSNOTAVAILABLE; > + IDirectPlay8PeerImpl* Client; > + > + Client = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(IDirectPlay8PeerImpl)); > + > + if(Client == NULL) > + { > + *ppobj = NULL; > + WARN("Not enough memory\n"); > + return E_OUTOFMEMORY; > + } > + > + Client->lpVtbl = &DirectPlay8Peer_Vtbl; > + return IDirectPlay8PeerImpl_QueryInterface((PDIRECTPLAY8PEER)Client, riid, ppobj); You are leaking the object on error. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: user32: fix error setting in FindWindow
by Dmitry Timoshkov Aug. 21, 2008

Aug. 21, 2008

"Louis. Lenders" <xerox_xerox2000(a)yahoo.co.uk> wrote: > @@ -1613,6 +1613,7 @@ HWND WINAPI FindWindowExW( HWND parent, HWND child, LPCWSTR className, LPCWSTR t > done: > HeapFree( GetProcessHeap(), 0, list ); > HeapFree( GetProcessHeap(), 0, buffer ); > + if(!retvalue) SetLastError(ERROR_FILE_NOT_FOUND); > return retvalue; > } The prevailing indentation style in this file is to have a space after 'if', i.e. use 'if (!retvalue)'. > HWND WINAPI FindWindowA( LPCSTR className, LPCSTR title ) > { > HWND ret = FindWindowExA( 0, 0, className, title ); > - if (!ret) SetLastError (ERROR_CANNOT_FIND_WND_CLASS); > return ret; > } Changing the whole block to just return FindWindowExA( 0, 0, className, title ); would be more logical. -- Dmitry.

1 0

user32: Fix hook_proc DLL loading using LoadLibraryExW instead of LoadLibraryW
by Hongbo Ni Aug. 21, 2008

Aug. 21, 2008

> >This patch fixes Bug 14784. > >As requested by Dmitry Timoshkov, I have created a standalone test case >for DLL injection via SetWindowHookEx. It has been attached in Bug 14784 >with MSVC source code and compiled exe and dll. > >With LoadLibraryExW(module, NULL, LOAD_WITH_ALTERED_SEARCH_PATH), the >Wine will search any imported DLL first in the same directory as module, >then in the standard search path. > >LoadLibraryW(module) searches imported DLL(s) in the directory of current >process, which may not be the same process where the module is first >loaded. That is why that LoadLibraryW(module) fails to hook to another >process if the DLL containing the hook_proc links to another DLL(s) in >the same directory. > >--- > dlls/user32/hook.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > >diff --git a/dlls/user32/hook.c b/dlls/user32/hook.c >index 4354e05..638a302 100644 >--- a/dlls/user32/hook.c >+++ b/dlls/user32/hook.c >@@ -322,7 +322,7 @@ void *get_hook_proc( void *proc, const WCHAR *module ) > { > TRACE( "loading %s\n", debugstr_w(module) ); > /* FIXME: the library will never be freed */ >- if (!(mod = LoadLibraryW(module))) return NULL; >+ if (!(mod = LoadLibraryExW(module, NULL, LOAD_WITH_ALTERED_SEARCH_PATH))) return NULL; > } > return (char *)mod + (ULONG_PTR)proc; > } >-- Is there any problems with this patch? Hongbo Ni <hongbo(a)njstar.com> +----------------------------------------------------------------+ * NJStar Software Corp., P.O. Box 40, Epping NSW 2121, Australia. * Tel: +61-2-9869-1088 WebSite: http://www.njstar.com * Fax: +61-2-9869-1099 Email: http://njstar.com/contact/ * eFax: +1-253-595-4788 +----------------------------------------------------------------+

1 0

Running a conformance test on windows before submitting to wine-patches.
by Scott Lindeneau Aug. 21, 2008

Aug. 21, 2008

I would like to submit the following AcceptEx conformance test to wine-patches, but I don't have a windows box (and i haven't got qemu setup(correctly) yet). Would someone run this and let me know how/if it failes (or succeeds... it should succeed). Thanks - Scott

3 2

Re: [PATCH] comctl32: Set listview focus on WM_LBUTTONUP instead of WM_LBUTTONDOWN.
by Lei Zhang Aug. 21, 2008

Aug. 21, 2008

On Mon, Aug 18, 2008 at 4:08 PM, Lei Zhang <thestig(a)google.com> wrote: > Hi, > > This should fix bug 14908. > Any problems with this patch?

1 0

Re: Alexander Nicolaysen Sørnes : regedit: Convert printing&help to unicode.
by Eric Pouech Aug. 20, 2008

Aug. 20, 2008

Alexandre Julliard a écrit : > Module: wine > Branch: master > Commit: 5cb8bccf462790338677dac3bfb394df8f4da020 > URL: http://source.winehq.org/git/wine.git/?a=commit;h=5cb8bccf462790338677dac3b… > > Author: Alexander Nicolaysen Sørnes <alex(a)thehandofagony.com> > Date: Sun Aug 10 12:43:06 2008 +0200 > > regedit: Convert printing&help to unicode. > > --- > > programs/regedit/framewnd.c | 25 +++++++++++++++++-------- > 1 files changed, 17 insertions(+), 8 deletions(-) > > @@ -671,8 +671,11 @@ static BOOL _CmdWndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam) > case ID_REGISTRY_DISCONNECTNETWORKREGISTRY: > break; > case ID_REGISTRY_PRINT: > - PrintRegistryHive(hWnd, _T("")); > + { > + const WCHAR empty = 0; > + PrintRegistryHive(hWnd, empty); > break; > + } > this looks very wrong to me -- Eric Pouech "The problem with designing something completely foolproof is to underestimate the ingenuity of a complete idiot." (Douglas Adams)

3 4

AcceptEx patch series notes
by Dan Kegel Aug. 20, 2008

Aug. 20, 2008

Hi Scott. A few notes about your patch series. There should be no forward references in the patches; you can't have patch 1 depend on patch 2. Also, don't send two versions of tests, one for Wine and one for Windows; the same code has to work in both places. You can't even use an ifdef, since the windows .exe has to pass on wine. You can use a runtime test, perhaps, to skip some tests on Wine, but that's about it.

1 0

Re: Running a conformance test on windows before submitting to wine-patches.
by Scott Lindeneau Aug. 20, 2008

Aug. 20, 2008

Sorry about not knowing how to cross compile before. Here is the patch(gmail doesn't let me attach exe's... I will look into this). Please try this. Tell me if I am doing something wrong (again). -Scott

3 5

re: d3dx9: Implementation of D3DXGetDeclLength
by dank＠kegel.com Aug. 20, 2008

Aug. 20, 2008

http://kegel.com/wine/patchwatcher/results/ says you forgot to include the new file include/d3dx9mesh.h this time; you included it in http://www.winehq.org/pipermail/wine-patches/2008-August/059261.html but some how dropped it when you resent?

1 0

Re: Running a conformance test on windows before submitting to wine-patches.
by Scott Lindeneau Aug. 20, 2008

Aug. 20, 2008

It did. I wrote the test using a different set of patches which let it compile with wine. I will look up how to cross compile it and do that next time. Sorry for the inconvenience. On Thu, Aug 21, 2008 at 2:36 AM, Austin English <austinenglish(a)gmail.com> wrote: > On Wed, Aug 20, 2008 at 12:34 PM, Scott Lindeneau <slindeneau(a)gmail.com> wrote: >> I don't think so? I've never written a conformance test before. I did >> implement AcceptEx in wine, but it shouldn't need that patch to >> compile for windows should it? Or maybe I need to load it into memory >> instead of just calling it first... Terribly sorry. I'm making myself >> a nuisance. I will try writing it using the windows WSAIoctl call to >> load it into memory first. MSDN does say I need that. >> >> -Scott >> >> >> On Thu, Aug 21, 2008 at 2:29 AM, Austin English <austinenglish(a)gmail.com> wrote: >>> On Wed, Aug 20, 2008 at 12:23 PM, Scott Lindeneau <slindeneau(a)gmail.com> wrote: >>>> I would like to submit the following AcceptEx conformance test to >>>> wine-patches, but I don't have a windows box (and i haven't got qemu >>>> setup(correctly) yet). Would someone run this and let me know how/if >>>> it failes (or succeeds... it should succeed). >>>> >>>> Thanks >>>> - Scott >>>> >>>> >>>> >>>> >>> >>> Does it depend on another patch: >>> >>> austin(a)SERVER1:~/wine-git/dlls/ws2_32/tests$ make crosstest >>> i586-mingw32msvc-gcc protocol.cross.o sock.cross.o >>> testlist.cross.o -o ws2_32_crosstest.exe -L../../../dlls >>> -L../../../dlls/ws2_32 -L../../../dlls/kernel32 -lws2_32 -lkernel32 >>> sock.cross.o: In function >>> `overlapped_server':/home/austin/wine-git/dlls/ws2_32/tests/sock.c:633: >>> undefined reference to `_AcceptEx(a)32' >>> :/home/austin/wine-git/dlls/ws2_32/tests/sock.c:677: undefined >>> reference to `_GetAcceptExSockaddrs(a)32' >>> collect2: ld returned 1 exit status >>> make: *** [ws2_32_crosstest.exe] Error 1 >>> >> > > Please bottom post on wine-devel. > > In the future, sending a compiled exe is helpful, saves time. Does it > even past make test for you? > > > austin(a)SERVER1:~/wine-git/dlls/ws2_32/tests$ make test > ccache gcc -c -I. -I. -I../../../include -I../../../include > -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing > -Wdeclaration-after-statement -Wwrite-strings -Wpointer-arith -g -O2 > -o protocol.o protocol.c > ccache gcc -c -I. -I. -I../../../include -I../../../include > -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing > -Wdeclaration-after-statement -Wwrite-strings -Wpointer-arith -g -O2 > -o sock.o sock.c > sock.c: In function âoverlapped_serverâ: > sock.c:670: warning: pointer targets in passing argument 3 of > âGetOverlappedResultâ differ in signedness > ccache gcc -c -I. -I. -I../../../include -I../../../include > -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing > -Wdeclaration-after-statement -Wwrite-strings -Wpointer-arith -g -O2 > -o testlist.o testlist.c > ../../../tools/winegcc/winegcc -B../../../tools/winebuild -mconsole > protocol.o sock.o testlist.o -o ws2_32_test.exe.so > ../../../libs/port/libwine_port.a -lws2_32 -lkernel32 > sock.o: In function `overlapped_server': > /home/austin/wine-git/dlls/ws2_32/tests/sock.c:633: undefined > reference to `AcceptEx' > /home/austin/wine-git/dlls/ws2_32/tests/sock.c:677: undefined > reference to `GetAcceptExSockaddrs' > collect2: ld returned 1 exit status > winegcc: ccache failed > make: *** [ws2_32_test.exe.so] Error 2 >

1 0