[Bug 30668] New: IAMMultiMediaStreamImpl_GetState is needed by MCM2 (Motocross Madness 2)

http://bugs.winehq.org/show_bug.cgi?id=30668

Bug #: 30668
           Summary: IAMMultiMediaStreamImpl_GetState is needed by MCM2
                    (Motocross Madness 2)
           Product: Wine
           Version: 1.5.4
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: amstream
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
    Classification: Unclassified

Created attachment 40147
  --> http://bugs.winehq.org/attachment.cgi?id=40147
Crash from wine-1.5.4

MCM2 (Motocross Madness 2) crashes on startup thanks to amstream's stub. Native
amstream works around that crash. Wine-1.5.4.

[Bug 30668] IAMMultiMediaStreamImpl_GetState is needed by MCM2 (Motocross Madness 2)

http://bugs.winehq.org/show_bug.cgi?id=30668

Jörg Höhle  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |DUPLICATE

--- Comment #1 from Jörg Höhle  2012-12-15 12:34:19 CST ---
I think we don't need 2 bugs about MCM2, e.g. one per stubbed amstream
function.  So far, it's too early to tell whether the crash is caused by
GetFormat, GetState or any other stubs.

For the last 3 years, I have been tempted to write a patch like:

dlls/amstream/mediastream.c:
static HRESULT WINAPI
DirectDrawMediaStreamImpl_IDirectDrawMediaStream_GetFormat(IDirectDrawMediaStream
*iface,
        DDSURFACEDESC *current_format, IDirectDrawPalette **palette,
        DDSURFACEDESC *desired_format, DWORD *flags)
{
    FIXME("stub, expect crash soon!\n");
so people know what to expect...

*** This bug has been marked as a duplicate of bug 17836 ***

[Bug 30668] IAMMultiMediaStreamImpl_GetState is needed by MCM2 (Motocross Madness 2)

http://bugs.winehq.org/show_bug.cgi?id=30668

Ken Sharp  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #2 from Ken Sharp  2012-12-22 11:35:10 CST ---
Closing bugs marked Duplicate.

[Bug 30668] IAMMultiMediaStreamImpl_GetState is needed by MCM2 ( Motocross Madness 2)

https://bugs.winehq.org/show_bug.cgi?id=30668

Zebediah Figura  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|amstream                    |quartz

[Bug 30668] Motocross Madness 2 needs IAMMultiMediaStream::GetState() implementation

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=30668

Gijs Vermeulen  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|1                           |0
         Resolution|DUPLICATE                   |---
             Status|CLOSED                      |UNCONFIRMED
            Summary|IAMMultiMediaStreamImpl_Get |Motocross Madness 2 needs
                   |State is needed by MCM2     |IAMMultiMediaStream::GetSta
                   |(Motocross Madness 2)       |te() implementation

--- Comment #3 from Gijs Vermeulen  ---
Since Bug 17836 was duped into Bug 24182, I'm going to reopen this one and make
it about IAMMultiMediaStream::SetState().

[Bug 30668] Motocross Madness 2 needs IAMMultiMediaStream::GetState() implementation

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=30668

joaopa  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #4 from joaopa  ---
Bug is fixed in git: 65d917402f43d6c061668cff811f5fd6729d5750
Can an administrator close this bug as fixed.

Note: the videos suffer of the already reported bug: Bug 50145

[Bug 30668] Motocross Madness 2 needs IAMMultiMediaStream::GetState() implementation

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=30668

Gijs Vermeulen  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
      Fixed by SHA1|                            |23cbd7f591caef0dd74919bac4b
                   |                            |c58dc82011363
         Resolution|---                         |FIXED

--- Comment #5 from Gijs Vermeulen  ---
Marking FIXED.

[Bug 30668] Motocross Madness 2 needs IAMMultiMediaStream::GetState() implementation

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=30668

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #6 from Alexandre Julliard  ---
Closing bugs fixed in 5.22.