[Bug 24182] New: Memory Access Violation

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

Summary: Memory Access Violation
           Product: Wine
           Version: 1.3.1
          Platform: x86-64
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: directx-d3d
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=30442)
 --> (http://bugs.winehq.org/attachment.cgi?id=30442)
Console output dumped in a text file

Hi, it's my 1st submission of a bug here, so I don't know exactly if I'm doing
the right procedure. I'm trying to run a game which works fine in Windows with
dx8.

The way I used to get a debug output was too simple:

$ winedbg NFR.exe 2>&1 | tee debug.txt

The result follows attached, and I'm waiting for instructions. I'm able to do
any kind of test once I get directions from someone more experient, since I'm
not a skilled programmer.

Thanks in advance

[Bug 24182] Memory Access Violation

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

--- Comment #1 from Andrew Nguyen  2010-08-28 11:30:06 CDT ---
What game is this, and is there a download link? Your log doesn't show a crash,
and the initial backtrace indicates a harmless DIB section access.

[Bug 24182] Memory Access Violation

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

--- Comment #2 from Bremm  2010-08-28 11:54:14 CDT ---
Created an attachment (id=30448)
 --> (http://bugs.winehq.org/attachment.cgi?id=30448)
Console output with regular run

[Bug 24182] Memory Access Violation

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

--- Comment #3 from Bremm  2010-08-28 11:56:20 CDT ---
As far as I know, there's no download link. I bought it on CD in 2008.

On Wine it does open a little window, then jumps to full screen (1024x768).
Screen keeps itself black and when it goes back to desktop there's a message
"Memory Access Violation" in a little box with a "OK" button. Desktop
resolution sometimes doesn't go back to native size (1920x1080).

Looking in a regular terminal, it shows up the text file attached
(terminal.txt).

[Bug 24182] Need for Russia crashes

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

Jeff Zaroyko  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|directx-d3d                 |-unknown
            Summary|Memory Access Violation     |Need for Russia crashes

[Bug 24182] Need for Russia crashes

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

--- Comment #4 from Bremm  2010-08-28 19:50:12 CDT ---
I just made a short script and a profile for this game, so I can run it
"windowed".

$ cat nfr.sh
#!/bin/sh
export LANG=ru_RU.CP1251
export WINEPREFIX="/home/$USER/.nfr-debug"
export WINEDEBUG=+all
/usr/bin/wine NFR.exe

I'm reading about winegdb procedures to make the debug useful for developers.
:-)

[Bug 24182] Need for Russia crashes

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

--- Comment #5 from Vincent Povirk  2010-09-16 15:36:34 CDT ---
err:ntdll:RtlpWaitForCriticalSection section 0x7bca5884 "loader.c:
loader_section" wait timed out in thread 001d, blocked by 001f, retrying (60
sec)
wine: Critical section 7bca5884 wait failed at address 0x7bc3590b (thread
001d), starting debugger...

A backtrace may not be very useful, since it won't tell us why the other thread
has already grabbed this lock. I would suggest getting a +relay,+tid,+loader
log.

[Bug 24182] Need for Russia crashes

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

--- Comment #6 from Bremm  2011-04-10 05:42:35 CDT ---
Created an attachment (id=34026)
 --> (http://bugs.winehq.org/attachment.cgi?id=34026)
Compressed log file

Log generated with the options below:

WINEDEBUG=+relay,+tid,+loader ./NfR2.exe &> log.txt

[Bug 24182] Need for Russia crashes

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

Bruno Jesus  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |http://www.gamershell.com/d
                   |                            |ownload_38878.shtml
                 CC|                            |[e-mail]

[Bug 24182] Need for Russia crashes

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

Bruno Jesus  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1

--- Comment #7 from Bruno Jesus  2013-10-01 21:54:00 CDT ---
Confirming in wine 1.7.3.

[Bug 24182] Need for Russia crashes

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

--- Comment #8 from Bruno Jesus  2013-10-01 21:54:41 CDT ---
Created attachment 46165
  --> http://bugs.winehq.org/attachment.cgi?id=46165
wine 1.7.3 log

Native quartz, devenum and amstream do not fix the issue.

[Bug 24182] Need for Russia crashes

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

Jarkko K  changed:

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

--- Comment #9 from Jarkko K  ---
I downloaded the demo, not used fileplanet because it said that server is full
or something.

used this
http://finnish.feedburner.es/lv/group/view/kl226282/Need_for_Russia_Greatest...

it had some suspicious bundles, wants to add 3rd group programs.

This is only output I get

fixme:win:EnumDisplayDevicesW ((null),0,0x33f3c8,0x00000000), stub!
fixme:win:EnumDisplayDevicesW ((null),0,0x33f108,0x00000000), stub!
fixme:win:EnumDisplayDevicesW ((null),0,0x33f568,0x00000000), stub!
fixme:win:EnumDisplayDevicesW ((null),0,0x33f578,0x00000000), stub!

but it also had dialog which said that

"Memory access violation."

wine 1.7.18

[Bug 24182] Need for Russia crashes on startup (failure to create ddraw offscreen surface with no dimensions)

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

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|http://www.gamershell.com/d |http://static.lhp.hu/letolt
                   |ownload_38878.shtml         |okozpont.hu/programok/n/nee
                   |                            |d_for_russia_greatest_cars_
                   |                            |from_cccp_demo_eng.zip
                 CC|                            |[e-mail]
          Component|-unknown                    |directx-d3d
            Summary|Need for Russia crashes     |Need for Russia crashes on
                   |                            |startup (failure to create
                   |                            |ddraw offscreen surface
                   |                            |with no dimensions)

--- Comment #10 from Anastasius Focht  ---
Hello folks,

this download works better:
http://static.lhp.hu/letoltokozpont.hu/programok/n/need_for_russia_greatest_...

Relevant part of trace log:

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

$ WINEDEBUG=+tid,+seh,+relay,+d3d,+ddraw wine ./NFR.exe >>log.txt 2>&1
...
0029:trace:ddraw:ddraw7_CreateSurface iface 0x14b010, surface_desc 0x33faf4,
surface 0x33faf0, outer_unknown (nil).
...
0029:trace:ddraw:ddraw_surface_create ddraw 0x14b010, surface_desc 0x33faf4,
surface 0x33fa6c, outer_unknown (nil), version 7.
0029:trace:ddraw:ddraw_surface_create Requesting surface desc:
0029:trace:ddraw:DDRAW_dump_members  - DDSD_CAPS : DDSCAPS_OFFSCREENPLAIN
DDSCAPS_SYSTEMMEMORY 
...
0029:Call
wined3d.wined3d_get_adapter_display_mode(0014b780,00000000,0033f88c,00000000)
ret=7e68267b
0029:trace:d3d:wined3d_get_adapter_display_mode wined3d 0x14b780, adapter_idx
0, display_mode 0x33f88c, rotation (nil).
0029:Call user32.EnumDisplaySettingsExW(0014e63c
L"\\.\DISPLAY1",ffffffff,0033f6b0,00000000) ret=7e5327a0
0029:Call
winex11.drv.EnumDisplaySettingsEx(0014e63c,ffffffff,0033f6b0,00000000)
ret=7ed0877e
0029:Ret  winex11.drv.EnumDisplaySettingsEx() retval=00000001 ret=7ed0877e
0029:Ret  user32.EnumDisplaySettingsExW() retval=00000001 ret=7e5327a0
0029:trace:d3d:wined3d_get_adapter_display_mode Returning 1024x768@60
WINED3DFMT_B8G8R8X8_UNORM 0x1.
0029:Ret  wined3d.wined3d_get_adapter_display_mode() retval=00000000
ret=7e68267b
0029:trace:ddraw:ddrawformat_from_wined3dformat Returning: ( DDPF_RGB , RGB
bits: 32, R 0x00ff0000 G 0x0000ff00 B 0x000000ff)
0029:trace:ddraw:wined3dformat_from_ddrawformat Convert a DirectDraw
Pixelformat to a WineD3D Pixelformat
0029:trace:ddraw:DDRAW_dump_pixelformat ( DDPF_RGB , RGB bits: 32, R 0x00ff0000
G 0x0000ff00 B 0x000000ff)
0029:warn:ddraw:ddraw_surface_create No width / height specified.
0029:Call ntdll.RtlFreeHeap(00110000,00000000,00b35500) ret=7e6828c7
0029:Ret  ntdll.RtlFreeHeap() retval=00000001 ret=7e6828c7
0029:Call wined3d.wined3d_mutex_unlock() ret=7e6547bc
0029:Ret  wined3d.wined3d_mutex_unlock() retval=00000000 ret=7e6547bc
0029:trace:seh:raise_exception code=c0000005 flags=0 addr=0x10052410
ip=10052410 tid=0029
0029:trace:seh:raise_exception  info[0]=00000000
0029:trace:seh:raise_exception  info[1]=00000010
0029:trace:seh:raise_exception  eax=00000000 ebx=004b452c ecx=00000000
edx=7e636b44 esi=004b4508 edi=0033fc28
0029:trace:seh:raise_exception  ebp=0033fc78 esp=0033fbac cs=0023 ds=002b
es=002b fs=0063 gs=006b flags=00210206 
...
0029:Call user32.MessageBoxA(00000000,1010d528 "Memory access
violation",1010d4e0 "Error!",00050000) ret=100012e6 
--- snip ---

The game engine tries to create an offscreen surface with no dimensions and
Wine's DirectDraw doesn't allow this:

--- snip ---
Wine-dbg>bt

Backtrace:

=>0 0x7e6828ca ddraw_surface_create+0xa57(ddraw=0x133108,
surface_desc=0x33faf4, surface=0x33fa6c, outer_unknown=(nil), version=0x7)
[/home/focht/projects/wine/wine.repo/src/dlls/ddraw/surface.c:5689] in ddraw
(0x0033fa38)

1 0x7e6547b4 ddraw7_CreateSurface+0x273(iface=<couldn't compute location>,
surface_desc=<couldn't compute location>, surface=<couldn't compute location>,
outer_unknown=<couldn't compute location>)
[/home/focht/projects/wine/wine.repo/src/dlls/ddraw/ddraw.c:2800] in ddraw
(0x0033fab8)

5682        /* No width or no height? Use the screen size. */
5683        if (!(desc->dwFlags & DDSD_WIDTH) || !(desc->dwFlags &
DDSD_HEIGHT))
5684        {
5685            if (!(desc->ddsCaps.dwCaps & DDSCAPS_PRIMARYSURFACE))
5686            {
5687                WARN("No width / height specified.\n");
5688                HeapFree(GetProcessHeap(), 0, texture);
5689                return DDERR_INVALIDPARAMS;
5690            }
--- snip ---

Source:
http://source.winehq.org/git/wine.git/blob/a227188573dc727dd6d2c53059c1c093a...

If I remove the check for being a primary surface the game starts and runs fine
(main menu shown, played a single race)

Maybe Henri or Stefan can comment on this behaviour.

$ sha1sum need_for_russia_greatest_cars_from_cccp_demo_eng.zip 
3d61347191f506673c8caeb42cd669c1fef0dd44 
need_for_russia_greatest_cars_from_cccp_demo_eng.zip

$ du -sh need_for_russia_greatest_cars_from_cccp_demo_eng.zip 
48M    need_for_russia_greatest_cars_from_cccp_demo_eng.zip

$ wine --version
wine-1.7.18-92-gb01fc1a

Regards

[Bug 24182] Need for Russia crashes on startup (failure to create ddraw offscreen surface with no dimensions)

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

--- Comment #11 from Henri Verbeet  ---
Just removing that check will make SizeTest() in dlls/ddraw/tests/dsurface.c
fail. It's not impossible that this is supposed to work for some other flag
combination than just DDSCAPS_PRIMARYSURFACE, but that would need some more
tests.

[Bug 24182] Need for Russia crashes on startup ( failure to create ddraw offscreen surface with no dimensions)

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

joaopa  changed:

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

--- Comment #12 from joaopa  ---
Can not test the bug because of problem of gstreamer. 
Can an administrator put this working link to the url place ?
https://games.softpedia.com/get/Games-Demo/Need-for-Russia-Greatest-Cars-fro...

[Bug 24182] Need for Russia crashes on startup ( failure to create ddraw offscreen surface with no dimensions)

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

Zebediah Figura  changed:

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

--- Comment #13 from Zebediah Figura  ---
(In reply to joaopa from comment #12)
> Can not test the bug because of problem of gstreamer. 
> Can an administrator put this working link to the url place ?
> https://games.softpedia.com/get/Games-Demo/Need-for-Russia-Greatest-Cars-
> from-CCCP.shtml

If there's a problem with gstreamer, can you please file a bug?

[Bug 24182] Need for Russia crashes on startup ( failure to create ddraw offscreen surface with no dimensions)

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

pattietreutel  changed:

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

[Bug 24182] Need for Russia crashes on startup (failure to create ddraw offscreen surface with no dimensions)

[WineHQ Bugzilla]

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

--- Comment #14 from Gijs Vermeulen  ---
I can still reproduce this with wine-5.10.

The workaround from Comment #10 still works.

[Bug 24182] Need for Russia crashes on startup (failure to create ddraw offscreen surface with no dimensions)

[WineHQ Bugzilla]

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

Gijs Vermeulen  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|http://static.lhp.hu/letolt |https://games.softpedia.com
                   |okozpont.hu/programok/n/nee |/get/Games-Demo/Need-for-Ru
                   |d_for_russia_greatest_cars_ |ssia-Greatest-Cars-from-CCC
                   |from_cccp_demo_eng.zip      |P.shtml

[Bug 24182] Need for Russia crashes on startup (failure to create ddraw offscreen surface with no dimensions)

[WineHQ Bugzilla]

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

--- Comment #15 from joaopa  ---
Created attachment 67408
  --> https://bugs.winehq.org/attachment.cgi?id=67408
console output with buitin amstrea

I am not convinced that it is a bug in ddraw. It looks like it is a bug in
amstream.
> fixme:amstream:ddraw_IDirectDrawMediaStream_GetFormat (012E4A44)->(0031FCCC,00000000,00000000,00000000) stub!

So format of surface is not given...

[Bug 24182] Need for Russia crashes on startup (failure to create ddraw offscreen surface with no dimensions)

[WineHQ Bugzilla]

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

Zebediah Figura  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|directx-d3d                 |quartz

[Bug 24182] Need for Russia crashes on startup (failure to create ddraw offscreen surface with no dimensions)

[WineHQ Bugzilla]

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

--- Comment #16 from joaopa  ---
Created attachment 67409
  --> https://bugs.winehq.org/attachment.cgi?id=67409
console ouput with native amstream

With native amstream, the message box does not occur anymore.
The game still does not work because of gstreamer problem...

[Bug 24182] Multiple games need IDirectDrawMediaStream::Get/SetFormat implementation (Need for Russia, Motocross Madness 2, The Sims Complete Collection)

[WineHQ Bugzilla]

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

Gijs Vermeulen  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Need for Russia crashes on  |Multiple games need
                   |startup (failure to create  |IDirectDrawMediaStream::Get
                   |ddraw offscreen surface     |/SetFormat implementation
                   |with no dimensions)         |(Need for Russia, Motocross
                   |                            |Madness 2, The Sims
                   |                            |Complete Collection)

[Bug 24182] Multiple games need IDirectDrawMediaStream::Get/SetFormat implementation (Need for Russia, Motocross Madness 2, The Sims Complete Collection)

[WineHQ Bugzilla]

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

Gijs Vermeulen  changed:

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

--- Comment #17 from Gijs Vermeulen  ---
*** Bug 17836 has been marked as a duplicate of this bug. ***

[Bug 24182] Multiple games need IDirectDrawMediaStream::Get/SetFormat implementation (Need for Russia, Motocross Madness 2, The Sims Complete Collection)

[WineHQ Bugzilla]

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

Gijs Vermeulen  changed:

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

--- Comment #18 from Gijs Vermeulen  ---
*** Bug 47081 has been marked as a duplicate of this bug. ***

[Bug 24182] Multiple games need IDirectDrawMediaStream::Get/SetFormat implementation (Need for Russia, Motocross Madness 2, The Sims Complete Collection)

[WineHQ Bugzilla]

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

Anya  changed:

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

[Bug 24182] Multiple games need IDirectDrawMediaStream::Get/SetFormat implementation (Need for Russia, Motocross Madness 2, The Sims Complete Collection)

[WineHQ Bugzilla]

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

--- Comment #19 from Alexandr Oleynikov  ---
Created attachment 68125
  --> https://bugs.winehq.org/attachment.cgi?id=68125
Sims 1 +amstream, +quartz

Seems like the patchset that implements AMDirectDrawStream::GetFormat
(https://source.winehq.org/patches/data/192243) isn't enough to get Sims 1
Complete Collection intro working. While with the patchset it no longer just
closes after being launched, it's still stuck on a black screen with a spam of
these lines:

0024:fixme:amstream:ddraw_sample_CompletionStatus (0318B280)->(3,4294967295):
stub
0024:fixme:amstream:ddraw_sample_Update (0318B280)->(3,00000000,00000000,0):
stub

Full log attached to the message.

[Bug 24182] Multiple games need IDirectDrawMediaStream::Get/SetFormat implementation (Need for Russia, Motocross Madness 2, The Sims Complete Collection)

[WineHQ Bugzilla]

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

Gijs Vermeulen  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Fixed by SHA1|                            |3f712c1d43122c120eee23fb980
                   |                            |a631942fc9e78
             Status|NEW                         |RESOLVED

--- Comment #20 from Gijs Vermeulen  ---
This is implemented now and Need for Russia now runs into Bug 29926 (videos can
be skipped, so demo is playable).

Marking FIXED.

[Bug 24182] Multiple games need IDirectDrawMediaStream::Get/SetFormat implementation (Need for Russia, Motocross Madness 2, The Sims Complete Collection)

[WineHQ Bugzilla]

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

Alexandre Julliard  changed:

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

--- Comment #21 from Alexandre Julliard  ---
Closing bugs fixed in 5.17.