November 2010 - Wine-Devel - list.winehq.org

Re: [PATCH 3/5] d3d9/tests: Check for multiple expected messages in test_wndproc().
by Marvin Nov. 4, 2010

Nov. 4, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=6786 Your paranoid android. === WVISTAADM (32 bit device) === device.c:2650: Test failed: Expected foreground window 000301CA, got 000201C8.

1 0

Re: [PATCH 2/5] d3d8/tests: Check for multiple expected messages in test_wndproc().
by Marvin Nov. 4, 2010

Nov. 4, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=6785 Your paranoid android. === WVISTAADM (32 bit device) === device.c:1670: Test failed: Expected foreground window 000201D4, got 000101D2. === W2K8SE (32 bit device) === device.c:1670: Test failed: Expected foreground window 00010126, got 0001011A.

1 0

Re: [PATCH] kernel32: Add a stub for SetProcessDEPPolicy
by Marcus Meissner Nov. 4, 2010

Nov. 4, 2010

On Wed, Nov 03, 2010 at 11:54:33PM +0100, Detlef Riekenberg wrote: > --- > dlls/kernel32/kernel32.spec | 1 + > dlls/kernel32/process.c | 8 ++++++++ > 2 files changed, 9 insertions(+), 0 deletions(-) > > diff --git a/dlls/kernel32/kernel32.spec b/dlls/kernel32/kernel32.spec > index 8381ae8..f32371e 100644 > --- a/dlls/kernel32/kernel32.spec > +++ b/dlls/kernel32/kernel32.spec > @@ -1053,6 +1053,7 @@ > @ stdcall SetNamedPipeHandleState(long ptr ptr ptr) > @ stdcall SetPriorityClass(long long) > @ stdcall SetProcessAffinityMask(long long) > +@ stdcall SetProcessDEPPolicy(long) > @ stdcall SetProcessPriorityBoost(long long) > @ stdcall SetProcessShutdownParameters(long long) > @ stdcall SetProcessWorkingSetSize(long long long) > diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c > index fdd19db..86747d9 100644 > --- a/dlls/kernel32/process.c > +++ b/dlls/kernel32/process.c > @@ -3394,3 +3394,11 @@ DEP_SYSTEM_POLICY_TYPE WINAPI GetSystemDEPPolicy(void) > FIXME("stub\n"); > return OptIn; > } > +/********************************************************************** > + * SetProcessDEPPolicy (KERNEL32.@) > + */ > +BOOL WINAPI SetProcessDEPPolicy(DWORD newDEP) > +{ > + FIXME("(%d): stub\n", newDEP); > + return TRUE; > +} Perhaps return FALSE; and SetLastError(..NOTIMPLEMENTED...) as we did not actually do it. (always better not to lie with security related functions.) Ciao, MArcus

1 0

Re: msvcrt/tests: _itow_s tests (try 3)
by Marvin Nov. 4, 2010

Nov. 4, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=6779 Your paranoid android. === WVISTAADM (32 bit string) === string.c:1446: Test failed: expected invalid_parameter_handler string.c:1455: Test failed: expected invalid_parameter_handler string.c:1464: Test failed: expected invalid_parameter_handler string.c:1473: Test failed: expected invalid_parameter_handler string.c:1485: Test failed: expected invalid_parameter_handler string.c:1497: Test failed: expected invalid_parameter_handler string.c:1509: Test failed: expected invalid_parameter_handler === W2K8SE (32 bit string) === string.c:1446: Test failed: expected invalid_parameter_handler string.c:1455: Test failed: expected invalid_parameter_handler string.c:1464: Test failed: expected invalid_parameter_handler string.c:1473: Test failed: expected invalid_parameter_handler string.c:1485: Test failed: expected invalid_parameter_handler string.c:1497: Test failed: expected invalid_parameter_handler string.c:1509: Test failed: expected invalid_parameter_handler === W7PRO (32 bit string) === string.c:1446: Test failed: expected invalid_parameter_handler string.c:1455: Test failed: expected invalid_parameter_handler string.c:1464: Test failed: expected invalid_parameter_handler string.c:1473: Test failed: expected invalid_parameter_handler string.c:1485: Test failed: expected invalid_parameter_handler string.c:1497: Test failed: expected invalid_parameter_handler string.c:1509: Test failed: expected invalid_parameter_handler === W7PROX64 (32 bit string) === string.c:1446: Test failed: expected invalid_parameter_handler string.c:1455: Test failed: expected invalid_parameter_handler string.c:1464: Test failed: expected invalid_parameter_handler string.c:1473: Test failed: expected invalid_parameter_handler string.c:1485: Test failed: expected invalid_parameter_handler string.c:1497: Test failed: expected invalid_parameter_handler string.c:1509: Test failed: expected invalid_parameter_handler === W7PROX64 (64 bit string) === string.c:1446: Test failed: expected invalid_parameter_handler string.c:1455: Test failed: expected invalid_parameter_handler string.c:1464: Test failed: expected invalid_parameter_handler string.c:1473: Test failed: expected invalid_parameter_handler string.c:1485: Test failed: expected invalid_parameter_handler string.c:1497: Test failed: expected invalid_parameter_handler string.c:1509: Test failed: expected invalid_parameter_handler

1 0

Re: [2/2] msvcrt/tests: qsort_s implementation and tests (try 3)
by Marvin Nov. 4, 2010

Nov. 4, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=6778 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

re: First contact
by Dan Kegel Nov. 4, 2010

Nov. 4, 2010

It's probably your game's copy protection. We can't advise you on circumventing it. The question probably belongs on wine-users, not wine-devel...

1 0

First contact
by Andréas LIVET Nov. 3, 2010

Nov. 3, 2010

Hi all wine developers, I read this mailing list for about one year I guess but I think I never send a message. Because until this Monday, I was a spectator, now I wanna become an "actor", I want to learn how to hack wine :). I don't have a lot of time, 1 hour/week, maybe more, but I just want to fix some bugs, if I find some, and maybe more in the future. I'll take my time. For info, I'm a video game developers for 2 years, so I'm a C++ programmer, I know nothing about C (except what I know from the C++) and even if I'm using exclusively linux (ubuntu) for 3/4 years at home, I'm not comfortable with the POSIX development environment... Not that I'm a fan of Visual C++/Visual Assist but I always use those tools (at school and then at work) and wanted to learn something else. I decided to start with a game (Paraworld, but it could be anything) which can't load a level : when I hit the start game button, I got the loading screen, the music etc. but nothing append. The mouse is still reacting, the music still playing but nothing is loading. There is nothing in the trace too, so I tried with the "WINEDEBUG=+relay", error!! I ended quickly with a log > 300Mo before seeing the main menu! I've seen that there is a patch which allows user to control when debug trace will be display, but I don't know how to find it. I found this page : http://www.mail-archive.com/wine-devel(a)winehq.org/msg05542.html But I can't find the archive (the patch file), is there a place where we could download patches? Maybe my question is very simple, but I'm a beginner so be indulgent, please :). Other question, is there a way with winedbg to "pause" program execution without having set any breakpoint. In my case, I don't know where to put the breakpoint so I wanted to break the program execution and see where it ends. This is possible with VCC, maybe it's not with winedbg? Anyway, thank you for your time, and I want to congratulate all the wine developers for their amazing work, win is getting better and better everyday thanks to you, Andréas --Not sent from an iPhone.

3 7

Re: [PATCH] more stubs for mprapi.dll
by Austin English Nov. 3, 2010

Nov. 3, 2010

On Wed, Nov 3, 2010 at 6:39 PM, <netzimme(a)aim.com> wrote: > [PATCH] more stubs for mprapi.dll The patch is gibberish. Additionally, please include your full real name when submitting patches to wine. -- -Austin

1 0

make crosstest fails for _fseeki64 and _ftelli64
by Eryk Wieliczko Nov. 3, 2010

Nov. 3, 2010

I am working on a test for _fseeki64 and _ftelli64. brutalcomputer(a)brutalcomputer-laptop:~/wine-git/dlls/msvcrt/tests$ vim file.c brutalcomputer(a)brutalcomputer-laptop:~/wine-git/dlls/msvcrt/tests$ make crosstest i586-mingw32msvc-gcc -c -I. -I. -I../../../include -I../../../include -I../../../include/msvcrt -I./.. -DWINE_STRICT_PROTOTYPES -DWINE_CROSSTEST -Wall -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=0 -o file.cross.o file.c file.c: In function ‘create_io_inherit_block’: file.c:942: warning: dereferencing type-punned pointer will break strict-aliasing rules ../../../tools/winegcc/winegcc -b i586-mingw32msvc -B../../../tools/winebuild --sysroot=../../.. --lib-suffix=.cross.a cpp.cross.o data.cross.o dir.cross.o environ.cross.o file.cross.o headers.cross.o heap.cross.o locale.cross.o misc.cross.o printf.cross.o scanf.cross.o signal.cross.o string.cross.o time.cross.o testlist.cross.o -o msvcrt_crosstest.exe file.cross.o: In function `func_file': /home/brutalcomputer/wine-git/dlls/msvcrt/tests/file.c:1474: undefined reference to `__ftelli64' collect2: ld returned 1 exit status winegcc: i586-mingw32msvc-gcc failed make: *** [msvcrt_crosstest.exe] Error 2 brutalcomputer(a)brutalcomputer-laptop:~/wine-git/dlls/msvcrt/tests$ of course make test works perfectly. My guess is that mingw doesn't support _ftelli64 and _fseeki64. What should I do in this situation?

1 0

Re: [6/6] msvcr80/tests: _itow_s tests (try 2)
by Marvin Nov. 3, 2010

Nov. 3, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=6768 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [5/6] msvcr80/tests: qsort_s tests (try 2)
by Marvin Nov. 3, 2010

Nov. 3, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=6767 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [4/6] msvcr80/tests: install test files (try 2)
by Marvin Nov. 3, 2010

Nov. 3, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=6766 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [2/6] msvcr80: qsort_s implementation (try 2)
by Alexandre Julliard Nov. 3, 2010

Nov. 3, 2010

"Vincas Miliūnas" <vincas.miliunas(a)gmail.com> writes: > qsort_s and _itow_s implementation and tests for bug #24925. These should go in msvcrt, and the tests in msvcr90. msvcr80 is a simple forwarding dll. -- Alexandre Julliard julliard(a)winehq.org

1 1

Re: [4/4] msxml3: Fix node_get_text() whitespace handling (resend)
by Alexandre Julliard Nov. 3, 2010

Nov. 3, 2010

Adam Martinson <amartinson(a)codeweavers.com> writes: > @@ -68,6 +68,8 @@ static const WCHAR szBinHex[] = {'b','i','n','.','h','e','x',0}; > > static const IID IID_xmlnode = {0x4f2f4ba2,0xb822,0x11df,{0x8b,0x8a,0x68,0x50,0xdf,0xd7,0x20,0x85}}; > > +BOOL is_preserving_whitespace(xmlNodePtr node); Please put function prototypes in msxml_private.h. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: include: Add IDirect3DRMWinDevice interface (try 2)
by Henri Verbeet Nov. 3, 2010

Nov. 3, 2010

On 2 November 2010 23:28, Vijay Kiran Kamuju <infyquest(a)gmail.com> wrote: > +#include <d3drm.h> > + > +#include <d3drm.h> I'd expect once to be enough.

1 0

Re: xlive: add initial stub dll (1/3)
by Alexandre Julliard Nov. 3, 2010

Nov. 3, 2010

Austin English <austinenglish(a)gmail.com> writes: > This dll is not a core part of windows (at least, not yet), but I > think it should be considered for inclusion in Wine. A bit of > explanation is necessary: > > xlive.dll comes from Games for Windows (1,2), whose installer depends > on .Net 3.5 (can be skipped with the /nodotnet parameter). Native > fails on wine, however, unless a native msasn1.dll is provided, > because xlive.dll is digitally signed (so implementing our own > msasn1.dll won't help). As it currently stands, users can't play any > 'Game for Windows' that doesn't have a Windows license. That's not a good enough reason, particularly considering how ugly the resulting code is. And it seems unlikely that this is ever going to move beyond the nasty hack stage, given the lack of documentation. -- Alexandre Julliard julliard(a)winehq.org

5 4

Re: [PATCH 01/14] [Msvcrt]: now using macro for parameters validation itoa_s (and updated the tests as well)
by Alexandre Julliard Nov. 3, 2010

Nov. 3, 2010

Eric Pouech <eric.pouech(a)orange.fr> writes: > +/* As of today, native and builtin msvcr80, msvcr90, msvcr100 raise an exception on CRT > + * parameter checking on "secure" > + * As Wine forwards most of the APIs of those msvcrNN DLLs to msvcrt, builtin msvcrt does > + * raise an exception on those cases > + * Testing shows that native msvcrt doesn't > + * So we make this test always succeed under Windows > + */ That's not very useful. This should be tested in msvcr90, where we can properly check whether the handler is called. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: msvcrt: Implement _get_tzname.
by Dmitry Timoshkov Nov. 3, 2010

Nov. 3, 2010

Eryk Wieliczko <ewdevel(a)gmail.com> wrote: > -static char tzname_std[64] = ""; > -static char tzname_dst[64] = ""; > +static char tzname_std[64] = "PST"; > +static char tzname_dst[64] = "PDT"; > char *MSVCRT__tzname[2] = { tzname_std, tzname_dst }; ... This is not an implementation, this is a quick and dirty hack. -- Dmitry.

1 0

Re: [6/6] msvcr80/tests: _itow_s tests
by Marvin Nov. 3, 2010

Nov. 3, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=6762 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [5/6] msvcr80/tests: qsort_s tests
by Marvin Nov. 3, 2010

Nov. 3, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=6761 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [4/6] msvcr80/tests: install test files
by Marvin Nov. 3, 2010

Nov. 3, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=6760 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: ddraw: Exclusive mode is the corner case for SetCooperative, not the normal mode.
by Marvin Nov. 3, 2010

Nov. 3, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=6757 Your paranoid android. === W98SE (32 bit ddrawmodes) === ddrawmodes: unhandled exception c0000005 at 004122E8 === W7PROX64 (64 bit ddrawmodes) === Timeout

1 0

Use of uninitialized variable in combine_uri()
by Gerald Pfeifer Nov. 3, 2010

Nov. 3, 2010

Hi Thomas, the following change of yours commit bced2e21dbc548ef9d41e3ff11384d7ad964c929 Author: Thomas Mullaly <thomas.mullaly(a)gmail.com> Date: Sat Oct 9 11:02:17 2010 -0400 urlmon: Implemented base case for CoInternetCombineIUri. introduces a new warning, use of uninitialized variable in the line marked "HERE" below. +static HRESULT combine_uri(Uri *base, Uri *relative, DWORD flags, IUri **result + Uri *ret; + HRESULT hr; + parse_data data; + + /* Base case is when the relative Uri has a scheme name, + * if it does, then 'result' will contain the same data + * as the relative Uri. + */ + if(relative->scheme_start > -1) { + DWORD create_flags = 0; + + memset(&data, 0, sizeof(parse_data)); + + data.uri = SysAllocString(relative->raw_uri); + if(!data.uri) { + IUri_Release(URI(ret)); <================== HERE + *result = NULL; + return E_OUTOFMEMORY; + } >From all I can tell this is a legitimate warning, that is, the code really invokes undefined behavior. Would you mind having a look? Gerald

2 1

re: xlive and msasn1.dll
by Dan Kegel Nov. 2, 2010

Nov. 2, 2010

Austin wrote: > xlive.dll comes from Games for Windows ... [and] > fails on wine, however, unless a native msasn1.dll is provided, > because xlive.dll is digitally signed (so implementing our own > msasn1.dll won't help). That's interesting. I wonder what other modules Microsoft is checking signatures on, and whether they're planning on doing that more often in the future.

2 1

(no subject)
by Matijn Woudt Nov. 2, 2010

Nov. 2, 2010

1 0