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

Wine FAQ edits

by Albert Lee

Hi Vitaliy, I notice you have reverted my edits to the FAQ on the wiki ( http://wiki.winehq.org/FAQ?action=info ). As I was trying to clarify the shell metacharacter information and operating system/platform support, and add new file association instructions, I would like to know why this happened and what changes should be done. -Albert

Jan. 29, 2011

Re: ntoskrnl.exe: unimplemented function ntoskrnl.exe.IoGetDeviceInterfaces

by Nikolay Sivov

On 1/29/2011 16:53, Qian Hong wrote: > 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 Please open bugs for such issues. wine-devel is for development process related discussions, not for user support.

Jan. 29, 2011

Re: shdocvw: Implement IWebBrowser::ExecWB and IWebBrowser::QueryStatusWB.

by Erich Hoover

On Sat, Jan 29, 2011 at 4:18 AM, Greg Geldorp <ggeldorp(a)vmware.com> wrote: > Jackpot. This version ran 10,000 times without a glitch on the dual-core machine. Great! Thank you so much for helping me to track this down. Erich Hoover ehoover(a)mines.edu

Jan. 29, 2011

ntoskrnl.exe: unimplemented function ntoskrnl.exe.IoGetDeviceInterfaces

by Qian Hong

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/

Jan. 29, 2011

RE: shdocvw: Implement IWebBrowser::ExecWB and IWebBrowser::QueryStatusWB.

by Greg Geldorp

From: Erich Hoover > Hmm, interesting - thanks for taking a look into this, I really appreciate > it. I ran a test on a different XP SP3 machine here (this time a dual-core) > and it worked just fine. So, at first I thought this must be an issue to > the combination of multi-core and Windows 7. > However, if the location of the exception has anything to say about it > (crashing after test_Navigate2:test_LocationURL), then my guess would be that > it's because of the "wb" global. I didn't notice this before (or if I did, I > forgot) but test_Navigate2:test_ready_state uses the IWebBrowser2 "wb" global, > which I did not initialize until after test_Navigate2. > If you wouldn't mind giving it a try on your system where the error is > reproducible, I've attached a revised patch where this variable is initialized > properly. Jackpot. This version ran 10,000 times without a glitch on the dual-core machine. Greg.

Jan. 29, 2011

Re: winex11.drv: When copying window bits exclude an update region only if the parent doesn't clip its children.

by Dmitry Timoshkov

Alexandre Julliard <julliard(a)winehq.org> wrote: > > - ExcludeUpdateRgn( hdc_dst, data->hwnd ); > > + if (!parent || !(GetWindowLongW( parent, GWL_STYLE ) & WS_CLIPCHILDREN)) > > + ExcludeUpdateRgn( hdc_dst, data->hwnd ); > > I don't see what the parent clipping has to do with this. Could you > please explain? Additional testing shows that this change is not correct, I'll try to figure out a better one. -- Dmitry.

Jan. 29, 2011

Re: Ubuntu 10.04 x86-64 test failures -- dsound:ds3d and mmdevapi

by Scott Ritchie

On 01/27/2011 11:24 PM, Reece Dunn wrote: > On 28 January 2011 01:44, Scott Ritchie <scott(a)open-vote.org> wrote: >> On 01/27/2011 02:20 PM, Reece Dunn wrote: >>> === Steps to Reproduce === >>> >>> Machine: Ubuntu 10.04 64-bit with NVidia binary drivers. >>> >>> 1/ Build the latest version of wine with no special options -- >>> `./configure && make` >>> 2/ Remove any previously run test result -- `find . -type f | grep >>> .ok | xargs rm` >>> 3/ Ensure running in a clean wine prefix -- `rm -rf ~/.wine` >>> 4/ Run the tests -- `make test` >>> >>> This will download the gecko package and then fail in the dsound:ds3d >>> tests with: >>> >>> ds3d.c:467: Test failed: buffer size changed after SetFormat() >>> - previous size was 88200, current size is 22052 >>> >>> This error seems to be specific to 64-bit versions of the Ubuntu >>> family, looking at the http://test.winehq.org/data/ results. >>> >> >> Is this with the ia32-libs from the Wine PPA? > > $ dpkg -s ia32-libs > Package: ia32-libs > Status: install ok installed > Priority: extra > Section: libs > Installed-Size: 143224 > Maintainer: Ubuntu Developers <ubuntu-devel-discuss(a)lists.ubuntu.com> > Architecture: amd64 > Version: 20090808ubuntu9 > > I am not using wine from the PPA, I am building it from git myself. > > - Reece You don't need the wine package from the PPA You do need the ia32-libs package from the Wine PPA in order to get gettext support. I recommend you do this when you're developing. -Scott

Jan. 28, 2011

Re: shdocvw: Implement IWebBrowser::ExecWB and IWebBrowser::QueryStatusWB.

by Erich Hoover

On Fri, Jan 28, 2011 at 6:45 AM, Greg Geldorp <ggeldorp(a)vmware.com> wrote: > ... > I started by running your test binary 10,000 times on W7PROX64. Not a single > crash, so that looked kind of promising. Just to be sure, I then ran the > binary on a dual-core Win7 x64 machine (the W7PROX64 VM has only one core > assigned to it). This resulted in an immediate crash. Tried a few more times, > it crashed 9 out of 10 times. > > To make sure that these crashes are related to your patch, I ran the test > without your patch on the same dual-core machine. First few tries didn't > produce crashes, I then ran it in a loop 10,000 times. Not a single crash. > > So, it looks like your patch introduces some multi-threading issue. I haven't > investigated further, perhaps I'll have some time over the weekend to dig a > bit deeper. Hmm, interesting - thanks for taking a look into this, I really appreciate it. I ran a test on a different XP SP3 machine here (this time a dual-core) and it worked just fine. So, at first I thought this must be an issue to the combination of multi-core and Windows 7. However, if the location of the exception has anything to say about it (crashing after test_Navigate2:test_LocationURL), then my guess would be that it's because of the "wb" global. I didn't notice this before (or if I did, I forgot) but test_Navigate2:test_ready_state uses the IWebBrowser2 "wb" global, which I did not initialize until after test_Navigate2. If you wouldn't mind giving it a try on your system where the error is reproducible, I've attached a revised patch where this variable is initialized properly. Erich Hoover ehoover(a)mines.edu

Jan. 28, 2011

RE: Correction to crash inside RtlCaptureStackBackTrace() + test case

by Janne Hakonen

> Now, when I ran these tests on Testbot using all base VMs, the tests were run also on W2KPROSP4 and WNT4WSSP6. However, the > function seems to buggy in those. It throws access violation whenever it is not skipping frames. > Is there some way I could prevent/skip these tests to be run on Windows NT4 and 2000? Ah, no matter, I think I've got it. Adding following lines to start allows the tests to be skipped on NT and 2000: __TRY { pRtlCaptureStackBackTrace(0, 62, callers, NULL); } __EXCEPT_ALL { win_skip("RtlCaptureStackBackTrace threw SEH exception, " "most likely running on Windows NT or 2000\n"); return; } __ENDTRY Thanks, Janne

Jan. 28, 2011

Re: Alexander Scott-Johns : ole32: Add support for rendering METAFILEPICT clipboard objects.

by Nikolay Sivov

On 1/28/2011 21:30, Alexandre Julliard wrote: > Module: wine > Branch: master > Commit: 0f4fbd30f6bf3d2ef471d3d75307c7fa8a7df137 > URL: http://source.winehq.org/git/wine.git/?a=commit;h=0f4fbd30f6bf3d2ef471d3d75… > > Author: Alexander Scott-Johns<alexander.scott.johns(a)googlemail.com> > Date: Tue Jan 25 15:09:21 2011 +0000 > > ole32: Add support for rendering METAFILEPICT clipboard objects. > > --- > dlls/ole32/clipboard.c | 97 +++++++++++++++++++++++++++++++++++++++++++++++- > 1 files changed, 96 insertions(+), 1 deletions(-) > > +/*************************************************************************** > + * get_data_from_metafilepict > + */ > +static HRESULT get_data_from_metafilepict(IDataObject *data, FORMATETC *fmt, HGLOBAL *mem) > +{ > + HGLOBAL copy; > + HRESULT hr; > + FORMATETC mem_fmt; > + STGMEDIUM med; > + > + *mem = NULL; > + > + mem_fmt = *fmt; > + mem_fmt.tymed = TYMED_MFPICT; > + > + hr = IDataObject_GetData(data,&mem_fmt,&med); > + if(FAILED(hr)) return hr; > + > + hr = dup_metafilepict(med.u.hMetaFilePict,&copy); > + if(FAILED(hr)) return hr; 'med' should be released here on failure. > + > + *mem = copy; > + > + ReleaseStgMedium(&med); > + > + return hr; > +} > + > /***********************************************************************

Jan. 28, 2011