January 2007 - Wine-Devel - list.winehq.org

Re: [lostwages] Update download-deb page with better information, error fixes

by Detlef Riekenberg

On Di, 2007-01-30 at 01:09 -0800, Scott Ritchie wrote: > + > +<p>While there is currently no Wine package explicitly designed for > the 64-bit version > +of Ubuntu, there are several hacks that can be used to install the > 32-bit package > +into the 64-bit distribution and have it function normally. See > +<a href="https://wiki.ubuntu.com/I386WineInAmd64">this page on the > Ubuntu wiki</a> > +for more details.</p> IMHO, a link to our own Wiki should be placed on the download-deb page http://wiki.winehq.org/WineOn64bit#head-56206e8bc74083807ffe06ccb471d3f964c… And as a Note on https://wiki.ubuntu.com/I386WineInAmd64 and https://help.ubuntu.com/community/WineForAMD64 The provided automatic installation script includes the sidenet-script. Please include a BIG WARNING, that this is never supported by winehq and the users need a clean wine, before they ask for help on #winehq or wine-users(a)winehq.org You mention sidenet on that page: "If you followed the manual instructions you need to install the Sidenet script" This is wrong. Please remove that line! Even, when the sidenet-script might help some users to run an app, "you need to install the Sidenet script" is wrong! -- By by ... Detlef

Jan. 30, 2007

Re: [PATCH 7/7] server/kernel32: Marshal security descriptor to server for a new process.

by Mirek

Hi, i cant compile wine with this patch, do you know why? gcc -c -I. -I. -I../../include -I../../include -D__WINESRC__ -DCOM_NO_WINDOWS_H -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wwrite-strings -Wpointer-arith -g -O2 -o regsvr.o regsvr.c make[2]: Leaving directory `/root/.WineCVS/sources/cvswine/wine/dlls/mmdevldr.vxd' gcc -c -I. -I. -I../../include -I../../include -D__WINESRC__ -D_KERNEL32_ -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wwrite-strings -Wpointer-arith -g -O2 -o profile.o profile.c process.c: In function ‘create_process’: process.c:1314: error: ‘struct new_process_request’ has no member named ‘sd_len’ process.c:1318: error: ‘struct new_process_request’ has no member named ‘sd_len’ process.c:1320: error: ‘struct new_process_request’ has no member named ‘sd_len’ process.c:1322: error: ‘struct new_process_request’ has no member named ‘sd_len’ process.c:1323: error: ‘struct new_process_request’ has no member named ‘sd_len’ process.c:1325: error: ‘struct new_process_request’ has no member named ‘sd_len’ process.c:1327: error: ‘struct new_process_request’ has no member named ‘sd_len’ make[2]: *** [process.o] Error 1 make[2]: *** Waiting for unfinished jobs.... make[2]: Entering directory `/root/.WineCVS/sources/cvswine/wine/dlls/monodebg.vxd' gcc -c -I. -I. -I../../include -I../../include -D__WINESRC__ -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wwrite-strings -Wpointer-arith -g -O2 -o monodebg.o monodebg.c ../../tools/winegcc/winegcc -B../../tools/winebuild -shared ./monodebg.vxd.spec monodebg.o -o monodebg.vxd.so -lkernel32 ../../libs/port/libwine_port.a ../../tools/winegcc/winegcc -B../../tools/winebuild -shared ./mlang.spec mlang.o regsvr.o -o mlang.dll.so -loleaut32 -lole32 -luser32 -lgdi32 -ladvapi32 -lkernel32 -lntdll -Wb,-doleaut32 -luuid ../../libs/port/libwine_port.a make[2]: Leaving directory `/root/.WineCVS/sources/cvswine/wine/dlls/kernel32' make[1]: *** [kernel32] Error 2 make[1]: *** Waiting for unfinished jobs.... make[2]: Leaving directory `/root/.WineCVS/sources/cvswine/wine/dlls/mlang' make[2]: Leaving directory `/root/.WineCVS/sources/cvswine/wine/dlls/monodebg.vxd' make[1]: Leaving directory `/root/.WineCVS/sources/cvswine/wine/dlls' make: *** [dlls] Error 2 Mirek Vitaliy Margolen napsal(a): > We are limited to only 5 server data calls. Here it would have been 7. > And possibly 12 in the future. Hence decision to send self-relative > SECURITY_DESCRIPTOR. > --- > dlls/kernel32/process.c | 25 +++++++++++++++++++++++++ > server/process.c | 8 ++++++-- > server/protocol.def | 2 ++ > 3 files changed, 33 insertions(+), 2 deletions(-) > > > ------------------------------------------------------------------------ > > diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c > index 8cdcd72..463071d 100644 > --- a/dlls/kernel32/process.c > +++ b/dlls/kernel32/process.c > @@ -1277,6 +1277,8 @@ static BOOL create_process( HANDLE hFile > > SERVER_START_REQ( new_process ) > { > + PSECURITY_DESCRIPTOR p_sd = NULL; > + > req->inherit_all = inherit; > req->create_flags = flags; > req->socket_fd = socketfd[1]; > @@ -1303,6 +1305,27 @@ static BOOL create_process( HANDLE hFile > if (is_console_handle(req->hstderr)) req->hstderr = console_handle_unmap(req->hstderr); > } > > + /* marshal security descriptor */ > + if (psa && psa->lpSecurityDescriptor) > + { > + SECURITY_DESCRIPTOR_CONTROL control; > + DWORD revision; > + > + req->sd_len = RtlLengthSecurityDescriptor( psa->lpSecurityDescriptor ); > + RtlGetControlSecurityDescriptor( psa->lpSecurityDescriptor, &control, &revision ); > + > + if (control & SE_SELF_RELATIVE) > + wine_server_add_data( req, psa->lpSecurityDescriptor, req->sd_len ); > + else > + if ((p_sd = HeapAlloc( GetProcessHeap(), 0, req->sd_len ))) > + { > + RtlMakeSelfRelativeSD( psa->lpSecurityDescriptor, p_sd, &req->sd_len ); > + wine_server_add_data( req, p_sd, req->sd_len ); > + } > + else req->sd_len = 0; > + } > + else req->sd_len = 0; > + > wine_server_add_data( req, params, params->Size ); > wine_server_add_data( req, params->Environment, (env_end-params->Environment)*sizeof(WCHAR) ); > if ((ret = !wine_server_call_err( req ))) > @@ -1313,6 +1336,8 @@ static BOOL create_process( HANDLE hFile > info->hThread = reply->thandle; > } > process_info = reply->info; > + > + HeapFree( GetProcessHeap(), 0, p_sd ); > } > SERVER_END_REQ; > > diff --git a/server/process.c b/server/process.c > index 1c53f49..21dd8cb 100644 > --- a/server/process.c > +++ b/server/process.c > @@ -820,6 +820,7 @@ DECL_HANDLER(new_process) > struct thread *thread; > struct process *process; > struct process *parent = current->process; > + const SECURITY_DESCRIPTOR *p_sd = get_req_data(); > int socket_fd = thread_get_inflight_fd( current, req->socket_fd ); > > if (socket_fd == -1) > @@ -841,14 +842,14 @@ DECL_HANDLER(new_process) > info->hstderr = req->hstderr; > info->exe_file = NULL; > info->process = NULL; > - info->data_size = get_req_data_size(); > + info->data_size = get_req_data_size() - req->sd_len; > info->data = NULL; > > if (req->exe_file && > !(info->exe_file = get_file_obj( current->process, req->exe_file, FILE_READ_DATA ))) > goto done; > > - if (!(info->data = memdup( get_req_data(), info->data_size ))) goto done; > + if (!(info->data = memdup( (char*)get_req_data() + req->sd_len, info->data_size ))) goto done; > > if (!(thread = create_process( socket_fd, current, req->inherit_all ))) goto done; > process = thread->process; > @@ -900,6 +901,9 @@ DECL_HANDLER(new_process) > reply->phandle = alloc_handle( parent, process, req->process_access, req->process_attr ); > reply->thandle = alloc_handle( parent, thread, req->thread_access, req->thread_attr ); > > + if (req->sd_len) > + set_object_sd( &process->obj, p_sd, OWNER_SECURITY_INFORMATION | GROUP_SECURITY_INFORMATION | > + SACL_SECURITY_INFORMATION | DACL_SECURITY_INFORMATION ); > done: > release_object( info ); > } > diff --git a/server/protocol.def b/server/protocol.def > index f49f331..9ebbcbc 100644 > --- a/server/protocol.def > +++ b/server/protocol.def > @@ -436,6 +436,8 @@ typedef union > unsigned int process_attr; /* attributes for process object */ > unsigned int thread_access; /* access rights for thread object */ > unsigned int thread_attr; /* attributes for thread object */ > + data_size_t sd_len; /* length of process' security descriptor */ > + VARARG(sd,bytes,sd_len); /* process' security descriptor to set */ > VARARG(info,startup_info); /* startup information */ > VARARG(env,unicode_str); /* environment for new process */ > @REPLY > > > > ------------------------------------------------------------------------ > >

Jan. 30, 2007

test.winehq.com status

by Felix Nawothnig

Hi. Anyone knows what happened to test.winehq.com/data? There has been no new crossbuilds for quite some time now... Felix

Jan. 30, 2007

Re: Test case for Bug 50 [Was: Bug 50]

by Pedro Araujo Chaves Jr.

On 1/30/07, Dmitry Timoshkov wrote: > > Then probably DrawText(DT_CALCRECT) could better suit your needs. > Almost on the bull's-eye! Actually, DrawText() returns the *height* of the string... how lucky I am, huh? But a "See Also" link in the MSDN page for DrawText() [1] pointed me to TabbedTextOut() [2], which returns both the height and the width... I'll give it a shot, even though the comment for TEXT_TabbedTextOut() in dlls/user32/text.c [3] doesn't encourage me much... ¬¬ (I shouldn't change *that* behavior of Windows, should I? =P) Thanks for the suggestion! Anyone else? - Pedro. [1] http://msdn.microsoft.com/library/en-us/gdi/fontext_0odw.asp [2] http://msdn.microsoft.com/library/en-us/gdi/fontext_001g.asp [3] It reads: /*********************************************************************** * TEXT_TabbedTextOut * * Helper function for TabbedTextOut() and GetTabbedTextExtent(). * Note: this doesn't work too well for text-alignment modes other * than TA_LEFT|TA_TOP. But we want bug-for-bug compatibility :-) */

Jan. 30, 2007

Detect screen resolution change errors

by Francois Gouget

When I run the dlls/d3d9/device conformance test on Windows in VMware, I get a trace telling me that IDirect3D9_CreateDevice() failed and returned 0x8876086c. This seems to be because Windows has trouble changing the screen resolution when running in VMware (though it's perfectly capable of doing it when I go through the control panel). But when I run the test in Wine I'm getting a test failure because: * IDirect3D9_CreateDevice() did not change the screen resolution * but it returned success so that the screen size checks fail That's because XRandR fails to change the resolution (for whatever reason, it's irrelevant here). Wine detects that but does not report the information to the upper layers. So I have made a couple of patches: * the first one detects when XRandR, XVidMode or some low level routine fails to change the screen resolution, and propagates this so ChangeDisplaySettingsEx() returns an error * the second one modifies IWineD3DDeviceImpl_CreateAdditionalSwapChain() so it detects when ChangeDisplaySettingsEx() propagates that error upstream too Now the dlls/d3d9/device test succeeds on my machine (all while loudly complaining something is wrong): [...] err:xrandr:X11DRV_XRandR_SetCurrentMode Resolution change not successful -- perhaps display has changed? fixme:d3d9:IDirect3DDevice9Impl_CreateAdditionalSwapChain (0x1c4358) call to IWineD3DDevice_CreateAdditionalSwapChain failed fixme:d3d9:IDirect3D9Impl_CreateDevice (0x171448) D3D Initialization failed for WineD3DDevice 0x174ee8 device.c:741:could not create device, IDirect3D9_CreateDevice returned 0x8876086c device.c: 245 tests executed (0 marked as todo, 0 failures), 0 skipped. I have attached the relevant patches. Does this look ok to the DirectX gurus? -- Francois Gouget fgouget(a)codeweavers.com

Jan. 30, 2007

Daylight saving changes

by Francois Gouget

Quite a few countries seem to be changing their daylight saving rules this year. Also Wine stores timezone information in the registry so won't we need to update the data we store in the registry? While looking at wine's timezone data I also found quite a few differences with Windows' data. Hence the question: Where does wine.inf's current data come from? Where can one find authoritative data for this? Does any one have an opinion on what to do about those? * wine.inf's TZI field does not match Windows for these time zones: Won't this impact email tools? Arabic Standard Time Azores Standard Time Caucasus Standard Time Dateline Standard Time E. South America Standard Time GTB Standard Time Iran Standard Time Israel Standard Time New Zealand Standard Time Newfoundland Standard Time Pacific SA Standard Time * wine.inf is missing some time zones: Ekaterinburg Standard Time Greenland Standard Time Mid-Atlantic Standard Time Nepal Standard Time Sri Lanka Standard Time W. Central Africa Standard Time * wine.inf has an extra time zone: Mexico Standard Time 2 * Wine's display name is not in the same format as the Windows one: For instance for Tonga Standard Time: Windows has: "Display"="(GMT+13:00) Nuku'alofa" Wine has : "Display"="Pacific/Tongatapu" -- Francois Gouget fgouget(a)codeweavers.com

Jan. 30, 2007

Bugzilla: Adding a 'patch' keyword

by Francois Gouget

During a query I've stumbled on bug 5012 which happens to have a patch attached to it. Nine months ago it was suggested to the author to send this patch to wine-patches but either he didn't do it or it was not accepted. Now, I don't consider myself qualified to judge the patch and I have no particular interest in it. So I may not send it to wine-patches myself. But it would be nice if there was an easy way to find bugs that have a patch, even, and maybe especially, if it is incorrect. This would make it easier for interested parties to survey what potential fixes are languishing in Bugzilla and push them to wine-devel or wine-patches. And it would also give a starting point for someone interested in taking an existing patch that did not get committed to then update it, clean it up, and fix it to make it acceptable. Does that make sense? And for reference: * Missing NOVST timezone http://bugs.winehq.org/show_bug.cgi?id=5012 (in fact the timezone data seems wrong) -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ Indifference will certainly be the downfall of mankind, but who cares?

Jan. 30, 2007

Re: [msvcrt] stat64 (1/3) add declarations for stat64, fstat64 and wstat64

by Francois Gouget

On Tue, 30 Jan 2007, Damjan Jovanovic wrote: > Changelog: > * Added declarations for stat64, fstat64 and wstat64 --- a/include/msvcrt/sys/stat.h 2006-11-24 18:18:04.000000000 +0200 +++ b/include/msvcrt/sys/stat.h 2007-01-28 15:48:28.000000000 +0200 @@ -104,6 +104,20 @@ +struct __stat64 { + _dev_t st_dev; + _ino_t st_ino; + unsigned short st_mode; + short st_nlink; + short st_uid; + short st_gid; + _dev_t st_rdev; + __int64 st_size; + __time64_t st_atime; + __time64_t st_mtime; + __time64_t st_ctime; +}; This structure is called _stat64 in Visual Studio 2005. Has it changed in more recent versions? If not I would stick to it otherwise Windows code that uses 'struct _stat64' would fail to compile with Winelib. [...] +int _fstat64(const char*,struct __stat64*); +int _stat64(int,struct __stat64*); You have reversed the _fstat64() and _stat64() prototypes. --- a/include/msvcrt/sys/types.h 2006-11-24 18:18:04.000000000 +0200 +++ b/include/msvcrt/sys/types.h 2007-01-28 14:23:09.000000000 +0200 @@ -56,6 +56,11 @@ +#ifndef _TIME64_T_DEFINED +typedef __int64 __time64_t; +#define _TIME64_T_DEFINED +#endif I'd write it as below but that's purely an aesthetic thing (I like how the #ifndef and the #define line up): +#ifndef _TIME64_T_DEFINED +#define _TIME64_T_DEFINED +typedef __int64 __time64_t; +#endif -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ Any sufficiently advanced bug is indistinguishable from a feature. -- from some indian guy

Jan. 30, 2007

Re: [ws2_32] INET_ADDRSTRLEN (1/2) add INET_ADDRSTRLEN and INET_ADDRSTRLEN6

by Francois Gouget

On Tue, 30 Jan 2007, Damjan Jovanovic wrote: > Changelog: > * Added INET_ADDRSTRLEN and INET_ADDRSTRLEN6 defines. > > --- a/include/ws2tcpip.h 2006-11-24 18:18:04.000000000 +0200 > +++ b/include/ws2tcpip.h 2007-01-26 21:14:25.000000000 +0200 > @@ -204,6 +204,9 @@ > # define WS_EAI_SOCKTYPE WSAESOCKTNOSUPPORT > #endif > > +#define INET_ADDRSTRLEN 16 > +#define INET6_ADDRSTRLEN 46 The PSDK says that the values for these are respectively 22 and 65. Also the latest PSDK defines these in ws2ipdef.h which is a new header which is included from ws2tcpip.h. -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ tcA thgirypoC muinelliM latigiD eht detaloiv tsuj evah uoY

Jan. 30, 2007

Re: Make.rules.in: Generate better output

by Francois Gouget

On Tue, 30 Jan 2007, Bang Jun-Young wrote: > It's quite annoying to see tons of ugly "gcc -I. -I. -I../../include > -I../../include" > on the screen during build and in the log file thereafter, so... [...] > +ifeq ($(TOPSRCDIR),$(TOPOBJDIR)) > +ifeq ($(SRCDIR),.) > +INCLUDES = -I. -I$(TOPSRCDIR)/include $(EXTRAINCL) 'ifeq' is a GNU Make extension and thus the policy is to avoid it, especially for purely cosmetic issues. -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ Nouvelle version : les anciens bogues ont été remplacés par de nouveaux.

Jan. 30, 2007