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

Re: WineD3D: Properly initialize a variable
by H. Verbeet Aug. 27, 2006

Aug. 27, 2006

On 26/08/06, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > In IWineD3DSurfaceImpl_SaveSnapshot tmpTexture might be uninitialized, which > might lead to accidentally binding the 0 texture. How does initialising it with 0 help then?

2 1

Wine CVS crashing in Direct3D code with XF86-DRI error
by Chris Rankin Aug. 26, 2006

Aug. 26, 2006

Hi, I am trying to run the World of Warcraft 1.11.0 (trial) client using Wine 0.9.20 and XOrg 7.1 with a Radeon 9200 video card. I don't have a guest access code yet, and so I have only reached Warcraft's login screen. However, this screen seems to be working fine... until I press the "Quit" button. When I do that, the client crashes. Here is the default console output: $ /opt/wine/World\ of\ Warcraft/WoW.exe libGL warning: 3D driver claims to not support visual 0x4b libGL warning: 3D driver claims to not support visual 0x4b Mesa: CPU vendor: GenuineIntel Mesa: CPU name: Intel(R) Xeon(TM) CPU 2.66GHz Mesa: MMX cpu detected. Mesa: SSE cpu detected. Mesa: Not testing OS support for SSE, leaving enabled. fixme:advapi:SetSecurityInfo stub fixme:powrprof:DllMain (0x7cad0000, 1, (nil)) not fully implemented fixme:ntdll:NtPowerInformation Unimplemented NtPowerInformation action: 11 fixme:powrprof:DllMain (0x7cad0000, 0, (nil)) not fully implemented fixme:win:EnumDisplayDevicesW ((null),0,0x34eeec,0x00000000), stub! fixme:win:EnumDisplayDevicesW ((null),0,0x34f458,0x00000000), stub! fixme:win:EnumDisplayDevicesW ((null),0,0x34f6f8,0x00000000), stub! fixme:win:EnumDisplayDevicesW ((null),0,0x34f6f8,0x00000000), stub! fixme:d3d:IWineD3DImpl_GetDeviceCaps Caps support for directx9 is nonexistent at the moment! fixme:system:SystemParametersInfoW Unimplemented action: 113 (SPI_SETMOUSESPEED)fixme:d3d:IWineD3DDeviceImpl_GetAvailableTextureMem (0x18ac48) : stub, simulating 253MB for now, returning 253MB left fixme:d3d:IWineD3DDeviceImpl_SetRenderState WINED3DRS_POINTSIZE_MIN not supported on this opengl fixme:d3d:IWineD3DDeviceImpl_SetRenderState WINED3DRS_POINTSIZE_MAX not supported on this opengl fixme:d3d:IWineD3DDeviceImpl_CreateQuery (0x18ac48) Unhandled query type 9 fixme:d3d:IWineD3DDeviceImpl_SetRenderState WINED3DRS_POINTSIZE_MAX not supported on this opengl fixme:d3d:IWineD3DDeviceImpl_SetRenderState WINED3DRS_POINTSIZE_MIN not supported on this opengl fixme:d3d:IWineD3DDeviceImpl_CreateQuery (0x18ac48) Unhandled query type 8 fixme:win:EnumDisplayDevicesW ((null),0,0x34f168,0x00000000), stub! fixme:system:SystemParametersInfoW Unimplemented action: 112 (SPI_GETMOUSESPEED)fixme:system:SystemParametersInfoW Unimplemented action: 113 (SPI_SETMOUSESPEED)fixme:sync:CreateIoCompletionPort (0xffffffff, (nil), 00000000, 00000000): stub.fixme:wininet:InternetSetOptionW Option INTERNET_OPTION_CONNECT_TIMEOUT (5000): STUB fixme:wininet:InternetSetOptionW INTERNET_OPTION_SEND/RECEIVE_TIMEOUT not supported on protocol 4 fixme:wininet:InternetSetOptionW Option INTERNET_OPTION_CONTEXT_VALUE; STUB fixme:imm:ImmAssociateContextEx (0x10024, (nil), 16): stub X Error of failed request: BadValue (integer parameter out of range for operation) Major opcode of failed request: 145 (XFree86-DRI) Minor opcode of failed request: 9 () Value in failed request: 0x2c00058 Serial number of failed request: 485 Current serial number in output stream: 485 This crash is more serious than it looks because WoW only updates its configuration file when it exits, and (of course) this crash happens before it can do that. I have some WINEDEBUG=d3d output, but it's 300K compressed with bzip2 and so I cannot post it here. Can anyone suggest any way I might debug this, please? I *think* that the operation is XF86GetDrawableInfo(), but am not sure. And since this crash only happens on exit, could Wine be requesting "drawable info" for a resource that it has deleted? Thanks for any help, Cheers, Chris ___________________________________________________________ Now you can scan emails quickly with a reading pane. Get the new Yahoo! Mail. http://uk.docs.yahoo.com/nowyoucan.html

2 1

Re: Add support for tooltips for system tray icons
by Oleg Krylov Aug. 26, 2006

Aug. 26, 2006

Hello, On 8/25/06, James Liggett <jrliggett(a)cox.net> wrote: > Add support for tooltips for system tray icons. > > This patch partially based on the original systray implementation by Kai Morich <kai.morich(a)bigfoot.de>. I've been working on the simmilar patch and I want make several comments on the patch > + > + /* create icon tooltip */ > + icon->tooltip = CreateWindowEx(WS_EX_TOPMOST, TOOLTIPS_CLASS, NULL, > + WS_POPUP | TTS_NOPREFIX | TTS_ALWAYSTIP, > + CW_USEDEFAULT, CW_USEDEFAULT, > + CW_USEDEFAULT, CW_USEDEFAULT, > + icon->window, NULL, NULL, NULL); > Why the TTS_NOPREFIX flag is needed? It doesn't seem to be used on windows. Next, we need to set tooltip max width, primarily to enable multiline tooltips, as several applications relay on this. The code can be, just after tooltip creation: SendMessageW (icon->tooltip, TTM_SETMAXTIPWIDTH, 0, 400); On windows width value seem to be dependend on system DPI, but I haven't found the exact formula yet. 400 is value at the default 96 dpi, so it may be a good starting point. Then my idea is to create tool just after tooltip creation, and then in modify icon just change tooltip text if requested. Not sure if it is a better idea, but it fits better into the existing code. Something like this: after tooltip creation creation: TTTOOLINFOW ti; ZeroMemory (&ti, sizeof(TTTOOLINFOW)); ti.cbSize = sizeof(TTTOOLINFOW); ti.uFlags = TTF_SUBCLASS; ti.hwnd = icon->window; ti.hinst = 0; ti.uId = 0; ti.lpszText = NULL; GetClientRect (icon->window, &ti.rect); SendMessageW (icon->tooltip, TTM_ADDTOOLW, 0, (LPARAM)&ti); and in modify icon: if (nid->uFlags & NIF_TIP) { TTTOOLINFOW ti; ZeroMemory (&ti, sizeof(TTTOOLINFOW)); ti.cbSize = sizeof(TTTOOLINFOW); ti.hwnd = icon->window; ti.hinst = 0; ti.uId = 0; ti.lpszText = nid->szTip; SendMessageW (icon->tooltip, TTM_UPDATETIPTEXTW, 0, (LPARAM)&ti); }

2 3

Re: WineD3D: Unbind unused textures
by Stefan Dösinger Aug. 26, 2006

Aug. 26, 2006

Am Samstag 26 August 2006 21:56 schrieb Stefan Dösinger: > Apparently disabling texturing alone doesn't work, so bind texture 0 to all > unused texture units. This fixes the bad texture / reflections on many > objects in half-life 2, dxlevel 80, and makes the flashlight work more > reliably. The flashlight still doesn't work on bump mapped objects. Do not apply this patch, it doesn't fix the real issue.

1 0

Re: ole32: Add support for main-threaded apartments
by Alexandre Julliard Aug. 26, 2006

Aug. 26, 2006

Robert Shearman <rob(a)codeweavers.com> writes: > A main-threaded apartment is the first single-threaded apartment in > the process. It was designed to be used for legacy applications that > don't want to think about threading at all, even if they are always > called from the same context, as is the case for regular STAs. That one crashes the tests: ../../../tools/runtest -q -P wine -M ole32.dll -T ../../.. -p ole32_test.exe.so compobj.c && touch compobj.ok wine: Unhandled page fault on read access to 0x00000017 at address 0x7bc27699 (thread 000e), starting debugger... WineDbg starting on pid 0xd Unhandled exception: page fault on read access to 0x00000017 in 32-bit code (0x7bc27699). fixme:dbghelp:elf_load_debug_info_from_map Alpha-support for Dwarf2 information for ntdll<elf> Register dump: CS:0073 SS:007b DS:007b ES:007b FS:003b GS:0033 EIP:7bc27699 ESP:0033f970 EBP:0033fa28 EFLAGS:00010286( - 00 -RISP1) EAX:ffffffff EBX:7bc76420 ECX:00000000 EDX:0015e284 ESI:0033f9f8 EDI:0015e294 Stack dump: 0x0033f970: 00000040 0033fa5c 0000002e 0033f9cc 0x0033f980: 00000040 0033fa6c 0000002c 000004b8 0x0033f990: 00000002 7b8a9280 7bc53b4b 00000000 0x0033f9a0: 7bc7dca0 0033f9fc 7bc53b4b 6056b6dc 0x0033f9b0: 0033f9cc 0033fa0c 0033fa48 60526ff8 0x0033f9c0: 0033f9cc 03c00001 00000000 00000000 fixme:ntdll:RtlNtStatusToDosErrorNoTeb no mapping for c0000119 Backtrace: =>1 0x7bc27699 RtlpWaitForCriticalSection+0xb9(crit=0x15e284) [/home/julliard/wine/wine/dlls/ntdll/critsection.c:398] in ntdll (0x7bc27699) 2 0x7bc27c37 RtlEnterCriticalSection+0x67(crit=0x15e284) [/home/julliard/wine/wine/dlls/ntdll/critsection.c:508] in ntdll (0x7bc27c37) 3 0x60420ee5 start_apartment_remote_unknown+0x45 [/home/julliard/wine/wine/dlls/ole32/stubmanager.c:749] in ole32 (0x60420ee5) 4 0x6040d6a1 RPC_StartRemoting+0x31(apt=0x15e260) [/home/julliard/wine/wine/dlls/ole32/rpc.c:706] in ole32 (0x6040d6a1) 5 0x603fd2cb StdMarshalImpl_MarshalInterface+0xab(iface=0x15e388, pStm=0x15e360, riid=0x603124f0, pv=0x6031a48c, dwDestContext=0x3, pvDestContext=0x0, mshlflags=0x0) [/home/julliard/wine/wine/dlls/ole32/marshal.c:1081] in ole32 (0x603fd2cb) 6 0x603fef58 CoMarshalInterface+0x528(pStream=0x15e360, riid=0x603124f0, pUnk=0x6031a48c, dwDestContext=0x3, pvDestContext=0x0, mshlFlags=0x0) [/home/julliard/wine/wine/dlls/ole32/marshal.c:1623] in ole32 (0x603fef58) 7 0x602ff177 func_compobj+0xcb7 [/home/julliard/wine/wine/dlls/ole32/tests/compobj.c:411] in ole32_test (0x602ff177) 8 0x603111d0 run_test+0x120(name=0x11039e) [/home/julliard/wine/wine/dlls/ole32/tests/../../../include/wine/test.h:365] in ole32_test (0x603111d0) 9 0x603117d1 main+0x141(argv=0x6031189b) [/home/julliard/wine/wine/dlls/ole32/tests/../../../include/wine/test.h:412] in ole32_test (0x603117d1) 10 0x6031189b __wine_spec_exe_entry+0x5b(peb=0x7ffdf000) [/home/julliard/wine/wine/dlls/winecrt0/exe_entry.c:36] in ole32_test (0x6031189b) 11 0x7b86f84b start_process+0xeb(arg=0x0) [/home/julliard/wine/wine/dlls/kernel/process.c:822] in kernel32 (0x7b86f84b) 12 0x6001f777 wine_switch_to_stack+0x17 in libwine.so.1 (0x6001f777) 0x7bc27699 RtlpWaitForCriticalSection+0xb9 [/home/julliard/wine/wine/dlls/ntdll/critsection.c:398] in ntdll: movl 0x18(%eax),%eax 398 if (crit->DebugInfo) name = (char *)crit->DebugInfo->Spare[0]; -- Alexandre Julliard julliard(a)winehq.org

1 1

WineDBG / dbghelp symbol problem
by Roderick Colenbrander Aug. 26, 2006

Aug. 26, 2006

Hi Eric, One of my wine development machines is running a 64bit version of Debian. In case of debian all 32bit libraries and programs are installed in a 32bit 'chroot' /emul/ia32-linux. Wine is installed there aswell and further the 32bit libs from the chroot environment are in the library path. Wine itself works fine but when some program crashes winedbg doesn't produce very usefull information. For instance a 'bt' using 'winedbg notepad' should give something like: 0x603984f7 start_process+0xe7 [/usr/src/wine/dlls/kernel/process.c:822] in kernel32: movl %edi,0x0(%esp) 822 ExitProcess( entry( peb ) ); On this system I get something like: Wine-dbg>bt Backtrace: =>1 0x603984f7 (0x603984f7) 2 0x60022657 (0x60022657) Further I notice lots of lines like this: fixme:dbghelp:SymLoadModule Should have successfully loaded debug information for image c:\windows\system32\kernel32.dll fixme:dbghelp:SymLoadModule Should have successfully loaded debug information for image c:\windows\system32\ntdll.dll I think that the cause of the problem lies in here as I don't see these lines on a standard 32bit system. Further if I install wine into a standard /usr/local prefix, winedbg works ok aswell. It shows the correct information and the SymLoadModule lines don't fail either.Most likely there's some path issue in dbghelp. I have tried to look at the code but it is a big maze to me. Perhaps you have an idea where the problem can be. Regards, Roderick Colenbrander -- Der GMX SmartSurfer hilft bis zu 70% Ihrer Onlinekosten zu sparen! Ideal für Modem und ISDN: http://www.gmx.net/de/go/smartsurfer

3 4

Re: Alexandre Julliard : crypt32/tests: Avoid sizeof in traces.
by Dimi Paun Aug. 26, 2006

Aug. 26, 2006

On Fri, August 25, 2006 2:34 pm, Alexandre Julliard wrote: > Author: Alexandre Julliard <julliard(a)winehq.org> > Date: Fri Aug 25 18:37:38 2006 +0200 > > crypt32/tests: Avoid sizeof in traces. This is non-obvious -- why is that? -- Dimi Paun <dimi(a)lattica.com> Lattica, Inc.

4 5

Bugzilla statistics: whole lotta fixing going on!
by Dan Kegel Aug. 25, 2006

Aug. 25, 2006

3 3

Re: Add support for tooltips for system tray icons
by Frank Richter Aug. 25, 2006

Aug. 25, 2006

On 25.08.2006 07:44, James Liggett wrote: > + icon->tooltip = CreateWindowEx(WS_EX_TOPMOST, TOOLTIPS_CLASS, NULL, > + WS_POPUP | TTS_NOPREFIX | TTS_ALWAYSTIP, > + CW_USEDEFAULT, CW_USEDEFAULT, > + CW_USEDEFAULT, CW_USEDEFAULT, > + icon->window, NULL, NULL, NULL); Hm, wouldn't it be more economic to try to use one tooltip for all icons? -f.r.

2 3

Wine CVS - r1.88 of dlls/opengl32/wgl.c
by Chris Rankin Aug. 25, 2006

Aug. 25, 2006

Hi, I have just updated my wine installation from CVS, and my OpenGL game is now crashing: X Error of failed request: GLXBadPixmap Major opcode of failed request: 129 (GLX) Minor opcode of failed request: 17 (X_GLXVendorPrivateWithReply) Serial number of failed request: 441 Current serial number in output stream: 441 Reverting wgl.c to r1.87 fixes the problem. I have a Radeon 9250 card with XOrg 7.1 and DRI enabled, but am not using the proprietary ATI driver. Cheers, Chris ___________________________________________________________ To help you stay safe and secure online, we've developed the all new Yahoo! Security Centre. http://uk.security.yahoo.com

2 4

FEAR Combat
by Jeremy Newman Aug. 25, 2006

Aug. 25, 2006

Tried FEAR Combat on Wine. Looks like it needs a d3dx9_27.dll which we don't provide. Anyone else have any luck with this one? I'm not sure what the status is of all the required dx 9.0c dlls are.

9 15

Write-strings warning: the endgame
by Andrew Talbot Aug. 25, 2006

Aug. 25, 2006

Currently, there are just eight write-strings violations remaining in the codebase: six in mshtml/nsembed.c (lines 271, 353, 356, 367, 432 and 1237), one in mshtml/nsservice.c (line 454) and one in widl/parser.c (line 848). So I would advocate that the -Wwrite-strings switch could now be turned on. I have just submitted two speculative patches to attempt to fix these remaining warnings, but, if they are wrong, I would be grateful if those of you with mshtml expertise or yacc expertise should kindly step in and finish the respective job properly. Thanks, -- Andy.

3 3

Re: Write-strings warning fix
by Dan Hipschman Aug. 25, 2006

Aug. 25, 2006

On Fri, Aug 25, 2006 at 11:08:03AM +0100, Andrew Talbot wrote: > What confused me - especially with my ignorance of yacc - is that I couldn't > find a corresponding xmalloc() that assigns the char * variables that the > above free() frees. make_var() gains storage for a var_t type, but the > free() in get_type() releases the storage of a char *. I couldn't quite > follow what was going on, there. Is it because the string is the first > element in the structure, so the pointer to the string also points to the > start of the var_t type? The strings are allocated in the lexer (parser.l): yylval.str = xstrdup(kw); The patch that was actually committed (using xstrdup) is most consistent.

2 1

Re: getwinegit.sh v0.4 released!
by Michael [Plouj] Ploujnikov Aug. 25, 2006

Aug. 25, 2006

In light of the recent discussion about moving to GIT I would like to let send this email to everyone on wine lists. Since Christian's last update to his getwinegit script I have tried to improve it to be more general and work in my special setup. I sent him this same email but I haven't heard a reply yet. ---------- Forwarded message ---------- From: Michael [Plouj] Ploujnikov <ploujj(a)gmail.com> Date: Jul 26, 2006 10:34 AM Subject: Re: getwinegit.sh v0.4 released! To: Christian Lachner <gladiac.lists(a)gmail.com> I have made some fixes to the script and an example config file: - Removed the check for root user. The script shouldn't require root access, not to compile wine, not even to install it. I think the best way is to just check that the user running the script has write permissions to install, log, and git checkout folders. - changed the CONFIGURE variable to be an array to allow for quoted words passed to ./configure. From my configuration that would be something like this: ./configure --x-libraries=/emul/linux/x86/usr/lib/ LDFLAGS="-L/emul/linux/x86/usr/lib -L/emul/linux/x86/lib" (because I need to compile 32bit wine on a 64bit machine with emulation libraries) - fixed one typo that I noticed - added a definition of MAKE to my config file. Without it DEPEND=$MAKE" depend"; and others don't execute make. The same should also be added to the nocolors.config file: MAKE="make"; Of course, you can ignore the path changes in the config file, unless you want an example of what a non-root user would do. The changes are attached as unified diffs. For some reason google made me reply to you rather than wine-devel and I guess it's up to you to post the changes there anyways ... after a thorough review! Enjoy.

1 0

Re: LOSTWAGES: Replace references to CVS with Git
by Robert Shearman Aug. 25, 2006

Aug. 25, 2006

Mike McCormack wrote: >Index: templates/en/development.template >=================================================================== >RCS file: /home/wine/lostwages/templates/en/development.template,v >retrieving revision 1.6 >diff -u -p -r1.6 development.template >--- templates/en/development.template 28 Oct 2005 16:05:10 -0000 1.6 >+++ templates/en/development.template 18 Aug 2006 01:02:06 -0000 >@@ -6,7 +6,7 @@ > WineHQ hosts a number of services aiding the Wine development process. > > <ul> >- <li><a href="{$root}/site/cvs">Wine CVS tree</a></li> >+ <li><a href="{$root}/site/git">Wine Git tree</a></li> > <li><a href="{$root}/site/contributing">Contributing to Wine</a></li> > <li><a href="http://bugs.winehq.org/">Bug tracking</a></li> > <li><a href="http://appdb.winehq.org/">Application Database</a></li> > I think we should leave the reference to the CVS tree page (but still add a link to the Git tree). Some people find it easier to work with CVS than Git. -- Rob Shearman

12 22

Re: LOSTWAGES: Fix the UI status page html and links
by Dimi Paun Aug. 25, 2006

Aug. 25, 2006

On Fri, 2006-08-25 at 19:30 +0900, Mike McCormack wrote: > > Maybe we should move this to the Wiki too? Maybe. But it changes seldom enough, and I think it will be ugly to get all the formatting right in the Wiki format. -- Dimi Paun <dimi(a)lattica.com> Lattica, Inc.

1 0

Re: msi: Store properties in a hash table, not the database.
by Dan Kegel Aug. 25, 2006

Aug. 25, 2006

On 8/24/06, Mike McCormack <mike(a)codeweavers.com> wrote: > Use string insensitive comparisons for property names. > --- > dlls/msi/package.c | 253 package.c: In function 'MSI_CreatePackage': package.c:426: error: 'PROPERTY_HASH_SIZE' undeclared (first use in this function) package.c:427: error: 'struct tagMSIPACKAGE' has no member named 'props' Did you forget to include your change to msipriv.h?

1 0

Finding a regression
by Doug Laidlaw Aug. 25, 2006

Aug. 25, 2006

I may be on the wrong list. A program I use has shown a backward step in graphics between Wine 0.9.15 and 0.9.16, and I am trying to find the change responsible. I currently have Wine set as at 2006-06-21 16:21:20 CDT, it is identifying as 0.9.16, and the fault is present. The Web site says to get the CVS history from the mailing list archives, but the archive is only up to mid-2005. What is the best way to step backwards? Doug. -- When we want to read of the deeds that are done for love, whither do we turn? To the murder column. - G.B. Shaw.

4 9

Re: msi 5: Add initial implementation of the DirectoryListUp event
by Mike McCormack Aug. 24, 2006

Aug. 24, 2006

James Hawkins wrote: > +UINT msi_dialog_directorylist_up( msi_dialog *dialog ) > +{ > + msi_control *control; > + BOOL indirect; > + LPWSTR prop, path, ptr; > + > + control = msi_dialog_find_control_by_class( dialog, WC_LISTVIEWW ); > + indirect = control->attributes & msidbControlAttributesIndirect; > + prop = msi_dialog_dup_property( dialog, control->property, indirect ); I don't think we should be looking up controls by their Window class. Either look up "DirectoryList" by it's MSI control type, or have the DirectoryList code subscribe to the event somehow. > +UINT msi_dialog_directorylist_up( msi_dialog *dialog ); Please put declarations for functions in a different file in msipriv.h Mike

1 0

Re: msi 4: Add initial implementation of the DirectoryCombo control
by Mike McCormack Aug. 24, 2006

Aug. 24, 2006

James Hawkins wrote: > + indirect = msi_dup_property( dialog->package, control->property ); > + prop = indirect; > + } > + > + val = msi_dup_property( dialog->package, prop ); > + lstrcpynW( path, val, MAX_PATH ); > + > + PathStripPathW( path ); > + PathRemoveBackslashW( path ); Calling msi_dup_property allocates new strings, so you need to remember to free those strings too. Since it's allocated, why not use PathStripPathW and PathRemoveBackslashW on the allocated string, rather than trying to copy it first? Mike

1 0

Re: msi: Inform the user with an error message when the MSI file path is invalid
by Alexandre Julliard Aug. 24, 2006

Aug. 24, 2006

"James Hawkins" <truiken(a)gmail.com> writes: > This fixes bug 4404 and matches native behavior. As an aside, is it a > legal problem if our message matches native exactly? I personally > can't see why it would be a problem (interface vs. implementation, > this is User Interface), but I can always change it if it's a problem. It's better not to use the exact same messages, since it's not required for compatibility. Particularly in this case, since you know that the error is that the file does not exist, there's no reason to tell people to "contact the application vendor"... -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: msi: Fix strings with lengths that are exact multiples of 2^16.
by Alexandre Julliard Aug. 24, 2006

Aug. 24, 2006

Mike McCormack <mike(a)codeweavers.com> writes: > if ( datasize != offset ) > - ERR("string table load failed! (%08x != %08lx)\n", datasize, offset ); > + { > + ERR("string table load failed! (%08x != %08lx), please report\n", datasize, offset ); > + ExitProcess(1); > + } Surely it could fail more nicely than by killing the whole process... -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: msi: remove limit on number of handles (take 3)
by Alexandre Julliard Aug. 24, 2006

Aug. 24, 2006

"Dan Kegel" <dank(a)kegel.com> writes: > @@ -66,18 +67,29 @@ MSIHANDLE alloc_msihandle( MSIOBJECTHDR > > EnterCriticalSection( &MSI_handle_cs ); > > + if (msihandletable == NULL) > + { > + msihandletable_size = 256; > + msihandletable = msi_alloc_zero( > + msihandletable_size*sizeof(msi_handle_info)); > + } > + > /* find a slot */ > - for(i=0; i<MSIMAXHANDLES; i++) > + for(i=0; i<msihandletable_size; i++) > if( !msihandletable[i].obj ) > break; > - if( (i>=MSIMAXHANDLES) || msihandletable[i].obj ) > - goto out; > + if( i==msihandletable_size ) > + { > + msihandletable_size *= 2; > + msihandletable = msi_realloc_zero(msihandletable, > + msihandletable_size*sizeof(msi_handle_info)); > + } You should check for allocation failures and fail gracefully instead of crashing. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: replace fchmod() with chmod()
by Alexandre Julliard Aug. 24, 2006

Aug. 24, 2006

"Steven Edwards" <winehacker(a)gmail.com> writes: > Detlef sent this in a while back and I think it slipped through the que. > > Changelog: > Detlef Riekenberg wine.dev at web.de > - tools/winedump: replace fchmod() with chmod() The right way to make that code portable is to get rid of the shell script. -- Alexandre Julliard julliard(a)winehq.org

1 0

Regression with 'explorer' starting automatically by 'Process Explorer'
by Paul Vriens Aug. 24, 2006

Aug. 24, 2006

Hi, normally when I start Process Explorer it automatically launches the 'explorer' process. With the latest git 'explorer' crashes. Regression testing showed that this is most likely due to the XEmbed patch: http://source.winehq.org/git/?p=wine.git;a=commit;h=60a97505a63e5102f239d0d… Debug output: 000b:trace:x11drv:X11DRV_CreateBitmap (0x28c) 16x16 24 bpp 000b:trace:x11drv:X11DRV_GetBitmapBits (bmp=0x28c, buffer=0x16c1a4, count=0x300) 0009:trace:x11drv:X11DRV_SetWindowPos hwnd 0x1003a, after (nil), swp 0,0 391x17 flags 00000014 0009:trace:x11drv:SWP_DoWinPosChanging hwnd 0x1003a, after (nil), swp 0,0 391x17 flags 00001814 0009:trace:x11drv:SWP_DoWinPosChanging current (0,0)-(391,17) style 500000c2 new (0,0)-(391,17) 0009:trace:x11drv:X11DRV_set_window_pos win 0x1003a window (0,0)-(391,17) client (0,0)-(391,17) style 500000c2 0009:trace:x11drv:X11DRV_SetWindowPos status flags = 1807 000b:trace:x11drv:X11DRV_CreateBitmap (0x298) 15x15 24 bpp 000b:trace:x11drv:X11DRV_GetBitmapBits (bmp=0x298, buffer=0x16c122, count=0x2b2) 000b:trace:x11drv:X11DRV_set_window_pos win 0x10052 window (0,0)-(26,45) client (0,0)-(26,45) style 04c00000 000b:trace:x11drv:systray_dock_window Docking tray icon 0x10052 000b:trace:x11drv:X11DRV_SetWindowPos hwnd 0x10052, after (nil), swp 1025,769 0x0 flags 00000005 000b:trace:x11drv:SWP_DoWinPosChanging hwnd 0x10052, after (nil), swp 1025,769 0x0 flags 00001805 000b:trace:x11drv:SWP_DoWinPosChanging current (0,0)-(26,45) style 04c00000 new (1025,769)-(1051,814) 000b:trace:x11drv:SWP_DoOwnedPopups (0x10052) hInsertAfter = (nil) 000b:trace:x11drv:X11DRV_set_window_pos win 0x10052 window (1025,769)-(1051,814) client (1025,769)-(1051,814) style 04c00000 000b:trace:x11drv:X11DRV_sync_window_position setting win 3e00002 pos 1028,791,20x20 after 3 changes=f 000b:trace:x11drv:X11DRV_SetWindowPos status flags = 0801 000b:trace:x11drv:X11DRV_CreateBitmap (0x2a8) 32x32 1 bpp 000b:trace:x11drv:X11DRV_SetBitmapBits (bmp=0x2a8, bits=0x15b760, count=0x80) 000b:trace:x11drv:X11DRV_CreateBitmap (0x2a4) 32x32 24 bpp 000b:trace:x11drv:X11DRV_SetBitmapBits (bmp=0x2a4, bits=0x16c3e0, count=0xc00) 000b:trace:x11drv:X11DRV_set_window_pos win 0x10052 window (0,0)-(26,45) client (0,0)-(26,45) style 04c00000 000b:trace:x11drv:X11DRV_sync_window_position setting win 3e00002 pos 3,22,20x20 after 3 changes=3 000b:trace:x11drv:X11DRV_CreateWindow hwnd 0x10052 cs 0,0 26x45 000b:trace:x11drv:X11DRV_set_window_pos win 0x10052 window (0,0)-(26,45) client (3,22)-(23,42) style 04c00000 000b:trace:x11drv:X11DRV_sync_window_position setting win 3e00002 pos 3,22,20x20 after 3 changes=40 000b:trace:x11drv:X11DRV_CreateWindow win 0x10052 window 0,0,26,45 client 3,22,23,42 whole 3,22,23,42 X client 0,0,20,20 xwin 3e00002 000b:trace:x11drv:X11DRV_ShowWindow hwnd=0x10052, cmd=8, wasVisible 0 000b:trace:x11drv:X11DRV_SetWindowPos hwnd 0x10052, after (nil), swp 0,0 0x0 flags 00000053 000b:trace:x11drv:SWP_DoWinPosChanging hwnd 0x10052, after (nil), swp 0,0 0x0 flags 00001853 000b:trace:x11drv:SWP_DoWinPosChanging current (0,0)-(26,45) style 04c00000 new (0,0)-(26,45) 000b:trace:x11drv:SWP_DoOwnedPopups (0x10052) hInsertAfter = (nil) 000b:trace:x11drv:X11DRV_set_window_pos win 0x10052 window (0,0)-(26,45) client (3,22)-(23,42) style 14c00000 000b:trace:x11drv:X11DRV_sync_window_position setting win 3e00002 pos 3,22,20x20 after 3 changes=40 000b:trace:x11drv:X11DRV_set_window_pos mapping win 0x10052 000b:trace:x11drv:systray_dock_window Docking tray icon 0x10052 000b:trace:x11drv:X11DRV_SetWindowPos hwnd 0x10052, after (nil), swp 1025,769 0x0 flags 00000005 000b:err:syslevel:_CheckNotSysLevel Holding lock 0x603ef040 level 2 000b:trace:seh:raise_exception code=80000003 flags=0 addr=0x77a5fc40 000b:trace:seh:raise_exception eax=00000044 ebx=77a83f64 ecx=602daa80 edx=00000044 esi=7ffdc044 edi=00000000 000b:trace:seh:raise_exception ebp=0034e2b8 esp=0034e290 cs=0073 ds=007b es=007b fs=003b gs=0033 flags=00000206 000b:trace:seh:call_stack_handlers calling handler at 0x77a05ec8 code=80000003 flags=0 wine: Unhandled exception 0x80000003 at address 0x77a5fc40 (thread 000b), starting debugger... 000b:trace:seh:start_debugger Starting debugger "winedbg --auto 10 64" WineDbg starting on pid 0xa 000b:trace:seh:call_stack_handlers handler at 0x77a05ec8 returned 1 000d:fixme:ntdll:NtDeviceIoControlFile Unsupported ioctl 8335000c (device=8335 access=0 func=3 method=0) 0009:trace:seh:raise_exception code=c000013a flags=0 addr=0x7bfff400 0009:trace:seh:raise_exception eax=00000003 ebx=00000007 ecx=0033e2ec edx=00000008 esi=0033e2ec edi=7ffdc000 0009:trace:seh:raise_exception ebp=0033e300 esp=0033e2b8 cs=0073 ds=007b es=007b fs=003b gs=0033 flags=00000293 0009:trace:seh:call_stack_handlers calling handler at 0x441d00 code=c000013a flags=0 [paul(a)penguin latest]$ _CheckNotSysLevel () at /wine/wine-git/dlls/kernel/syslevel.c:178 0x77a5fc41 _CheckNotSysLevel+0x41 [/wine/wine-git/dlls/kernel/syslevel.c:178] in kernel32: addl $36,%esp 178 } Modules: Module Address Debug info Name (46 modules) ELF 101000-16f000 Deferred libfreetype.so.6 ELF 175000-190000 Deferred ld-linux.so.2 ELF 192000-1b7000 Deferred libm.so.6 ELF 1b9000-1bd000 Deferred libdl.so.2 ELF 1bf000-1d2000 Deferred libz.so.1 ELF 1d4000-1da000 Deferred libxdmcp.so.6 ELF 1dc000-1df000 Deferred libxau.so.6 ELF 222000-232000 Deferred libxext.so.6 ELF 23a000-25b000 Deferred libexpat.so.0 ELF 25d000-266000 Deferred libxrender.so.1 ELF 2c2000-2c4000 Deferred libnvidia-tls.so.1 ELF 2cb000-2e1000 Deferred libnsl.so.1 ELF 4e8000-4ec000 Deferred libcap.so.1 ELF 4ee000-561000 Deferred libdbus-1.so.2 ELF 74e000-752000 Deferred libxrandr.so.2 ELF 7bd000-7c2000 Deferred libxxf86vm.so.1 ELF 7da000-7df000 Deferred libxfixes.so.3 ELF 927000-931000 Deferred libxcursor.so.1 ELF adf000-c12000 Deferred libc.so.6 ELF c9b000-ca5000 Deferred libhal.so.1 ELF ca8000-da5000 Deferred libx11.so.6 ELF da7000-dbb000 Deferred libpthread.so.0 ELF dbd000-dfa000 Deferred libfontconfig.so.1 ELF 41b4000-49cb000 Deferred libglcore.so.1 ELF 49cd000-4a5c000 Deferred libgl.so.1 ELF 78d5000-78ef000 Deferred libice.so.6 ELF 7915000-791e000 Deferred libsm.so.6 ELF 60000000-60111000 Deferred libwine.so.1 ELF 6025c000-602dc000 Deferred ntdll<elf> \-PE 60270000-602dc000 \ ntdll ELF 602dc000-602f3000 Deferred explorer<elf> \-PE 602e0000-602f3000 \ explorer ELF 602f3000-60429000 Deferred user32<elf> \-PE 60310000-60429000 \ user32 ELF 60429000-604b9000 Deferred gdi32<elf> \-PE 60440000-604b9000 \ gdi32 ELF 604b9000-604ff000 Deferred advapi32<elf> \-PE 604c0000-604ff000 \ advapi32 ELF 6055d000-605e0000 Deferred winex11<elf> \-PE 60570000-605e0000 \ winex11 ELF 60fcc000-60fe8000 Deferred imm32<elf> \-PE 60fd0000-60fe8000 \ imm32 ELF 778e0000-778eb000 Deferred libnss_files.so.2 ELF 779da000-77adf000 Stabs kernel32<elf> \-PE 779f0000-77adf000 \ kernel32 ELF 7bf00000-7bf03000 Deferred <wine-loader> Threads: process tid prio (all id:s are in hex) 0000000a (D) c:\windows\system32\explorer.exe 0000000c 0 0000000b 0 <== Any idea where to start looking? Cheers, Paul. P.S. I write 'most likely due' as I've seen several cases of different results when bisecting.

2 5