WineHQ Bugzilla – Bug 35676 – iMesh 7.x crashes on IWMPCore::get_settings method stub

Bugzilla

Last modified: 2021-02-04 03:17:20 UTC

iMesh 7.x crashes on IWMPCore::get_settings method stub

Bug 35676 - iMesh 7.x crashes on IWMPCore::get_settings method stub


Summary:	iMesh 7.x crashes on IWMPCore::get_settings method stub

Status:	CLOSED FIXED

AppDB:	Show Apps affected by this bug
Product:	Wine
Classification:	Unclassified
Component:	wmp&wmvcore
Version:	1.7.13
Hardware:	x86 Linux

Importance:	P2 normal
Target Milestone:	---
Assigned To:	Mr. Bugs

URL:	https://web.archive.org/web/202102040...
Keywords:	download

Depends on:
Blocks:
	Show dependency tree

Reported:	2014-02-25 13:16 UTC by Anastasius Focht
Modified:	2021-02-04 03:17 UTC (History)
CC List:	1 user (show)

See Also:
Regression SHA1:
Fixed by SHA1:	fea6fedf29ace809b77be9294350229ed7ffcebd
Distribution:	---
Staged patchset:

Attachments
Add an attachment (back traces, logs, proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Anastasius Focht 2014-02-25 13:16:56 UTC

Hello folks,

the Windows Media Player core infrastructure is now in place - thanks to Jacek (bug 15286).

--- snip ---
$ pwd
/home/focht/.wine/drive_c/Program Files/iMesh Applications/iMesh

$ WINEDEBUG=+tid,+seh,+relay,+ole,+variant wine ./iMesh.exe >>log.txt 2>&1
...
0024:fixme:wmp:OleControl_GetControlInfo (0x1fdd80)->(0xeb070c)
0024:Call user32.GetWindowLongW(000103a4,fffffff0) ret=009404fd
0024:Ret  user32.GetWindowLongW() retval=46000000 ret=009404fd
0024:fixme:wmp:WMPPlayer4_get_settings (0x1fdd80)->(0x33e4a4)
0024:trace:seh:raise_exception code=c0000005 flags=0 addr=0x6093b3 ip=006093b3 tid=0024
0024:trace:seh:raise_exception  info[0]=00000000
0024:trace:seh:raise_exception  info[1]=00000000
0024:trace:seh:raise_exception  eax=00000000 ebx=031342dc ecx=00000000 edx=7bcecb88 esi=03134068 edi=001fdd80
0024:trace:seh:raise_exception  ebp=0033e4b4 esp=0033e488 cs=0023 ds=002b es=002b fs=0063 gs=006b flags=00210202
...
---- snip ---

The app tries to directly access the 'ppSettings' out parameter without looking at method HRESULT.

Source: http://source.winehq.org/git/wine.git/blob/b0b6728cf1ce323aa0198473e5210c2ec442a788:/dlls/wmp/player.c#l123

--- snip ---
123 static HRESULT WINAPI WMPPlayer4_get_settings(IWMPPlayer4 *iface, IWMPSettings **ppSettings)
124 {
125     WindowsMediaPlayer *This = impl_from_IWMPPlayer4(iface);
126     FIXME("(%p)->(%p)\n", This, ppSettings);
127     return E_NOTIMPL;
128 }
--- snip ---

MSDN: http://msdn.microsoft.com/en-us/library/windows/desktop/dd563236%28v=vs.85%29.aspx

$ sha1sum iMeshV7.exe 
0e4aa7ae0da88b7f69a7daa1bca36f3f2f6c497b  iMeshV7.exe

$ du -sh iMeshV7.exe 
8.4M	iMeshV7.exe

$ wine --version
wine-1.7.13-53-g37e0a1a

Regards

Comment 1 Jacek Caban 2014-02-26 05:05:37 UTC

Patch sent:

http://source.winehq.org/patches/data/102791

Comment 2 Anastasius Focht 2014-02-26 17:40:17 UTC

Hello folks,

this is fixed by commit http://source.winehq.org/git/wine.git/commitdiff/fea6fedf29ace809b77be9294350229ed7ffcebd

Thanks Jacek

--- snip ---
...
fixme:wmp:WMPSettings_put_enableErrorDialogs (0x1eb520)->(0)
fixme:wmp:WMPSettings_put_volume (0x1eb520)->(100)
fixme:wmp:ConnectionPointContainer_FindConnectionPoint (0x1eb520)->({19a6627b-da9e-47c1-bb23-00b5e668236a} 0x31242e4)
fixme:wmp:ConnectionPointContainer_FindConnectionPoint (0x1eb520)->({6bf52a51-394a-11d3-b153-00c04f79faa6} 0x31242e4)
fixme:wmp:WMPSettings_put_mute (0x1eb520)->(0)
fixme:wmp:WMPSettings_put_volume (0x1eb520)->(0)
...
--- snip ---

Regards

Comment 3 Alexandre Julliard 2014-03-07 14:13:54 UTC

Closing bugs fixed in 1.7.14.