[Bug 35963] New: crash in Syberia2

Bug ID: 35963 Summary: crash in Syberia2 Product: Wine Version: 1.7.16 Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: directx-d3d Assignee: wine-bugs@winehq.org Reporter: farmboy0+winehq@googlemail.com

Created attachment 48060 --> http://bugs.winehq.org/attachment.cgi?id=48060 backtrace #1 from wine

I have found 2 places in the game that show this backtrace. One is near the beginning of the game. Once you can control Kate Walker enter the toilet in the train.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

Show replies by date

wine-bugs＠winehq.org

10 Apr 10 Apr

7:45 p.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #1 from farmboy0+winehq@googlemail.com --- Created attachment 48061 --> http://bugs.winehq.org/attachment.cgi?id=48061 backtrace #2 from wine

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

9:38 p.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #2 from Austin English austinenglish@gmail.com --- What graphics card/drivers?

Please attach the full log running the game under wine.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

11 Apr 11 Apr

7:47 p.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #3 from farmboy0+winehq@googlemail.com --- Created attachment 48072 --> https://bugs.winehq.org/attachment.cgi?id=48072 full log with mesa debug output for backtrace #1

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

7:47 p.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #4 from farmboy0+winehq@googlemail.com --- Created attachment 48073 --> https://bugs.winehq.org/attachment.cgi?id=48073 full log with mesa debug output for backtrace #2

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

7:49 p.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #5 from farmboy0+winehq@googlemail.com --- I am using * Mesa git * Radeon SI open source driver git * Kernel 3.14 * X.org server 1.15

My graphics card is an AMD Radeon HD 7750 (Cape Verde)

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

12 Apr 12 Apr

1:51 p.m.

New subject: [Bug 35963] crash in Syberia2

Ken Sharp imwellcushtymelike@gmail.com changed:

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

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

3:18 p.m.

New subject: [Bug 35963] crash in Syberia2

Henri Verbeet hverbeet@gmail.com changed:

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

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

5:28 p.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #6 from farmboy0+winehq@googlemail.com --- Created attachment 48091 --> http://bugs.winehq.org/attachment.cgi?id=48091 patch that fixes the problem

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

13 Apr 13 Apr

9:16 a.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #7 from farmboy0+winehq@googlemail.com --- Created attachment 48102 --> http://bugs.winehq.org/attachment.cgi?id=48102 patch v2

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

6:09 p.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #8 from Henri Verbeet hverbeet@gmail.com --- I suspect this might primarily be a Mesa issue. What are the actual values that get passed to glLightf()? INF/NAN by any chance?

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

14 Apr 14 Apr

7:13 p.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #9 from farmboy0+winehq@googlemail.com --- %f prints the value as -nan so you are correct. but both the directx manual and the opengl spec define the values as >= 0, so the check should be correct. There is a bug in mesa however as the glLight function is supposed to return GL_INVALID_VALUE on negative values of ATTENUATION.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

7:40 p.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #10 from Henri Verbeet hverbeet@gmail.com --- (In reply to farmboy0+winehq from comment #9)

%f prints the value as -nan so you are correct. but both the directx manual and the opengl spec define the values as >= 0, so the check should be correct.

Yes, but I think it should be unnecessary, since GL should take care of it.

There is a bug in mesa however as the glLight function is supposed to return GL_INVALID_VALUE on negative values of ATTENUATION.

Yes. Specifically, at http://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/main/light.c#n257 "if (params[0] < 0.0) {" should probably either read "if (!(params[0] >= 0.0)) {", or explicitly check for NaN. http://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/main/light.c#n173 does use ">= 0.0" in the assertion, at that's probably what causes your issue.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

8:42 p.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #11 from farmboy0+winehq@googlemail.com --- (In reply to Henri Verbeet from comment #10)

(In reply to farmboy0+winehq from comment #9)

...
%f prints the value as -nan so you are correct. but both the directx manual and the opengl spec define the values as >= 0, so the check should be correct.

Yes, but I think it should be unnecessary, since GL should take care of it.

No it shouldnt. the GL manual says it should return an GL_INVALID_VALUE which would still be an error in wine but probably with lesser consequences. The comments in the wine code even state not to accept all the BS windows apps set as attenuation values.

...
There is a bug in mesa however as the glLight function is supposed to return GL_INVALID_VALUE on negative values of ATTENUATION.

Yes. Specifically, at http://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/main/light.c#n257 "if (params[0] < 0.0) {" should probably either read "if (!(params[0] >= 0.0)) {", or explicitly check for NaN. http://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/main/light.c#n173 does use ">= 0.0" in the assertion, at that's probably what causes your issue.

Yes the assertion causes the error. I will file a bug in mesa for that.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

16 Apr 16 Apr

9:17 a.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #12 from Henri Verbeet hverbeet@gmail.com --- (In reply to farmboy0+winehq from comment #11)

(In reply to Henri Verbeet from comment #10)

...
(In reply to farmboy0+winehq from comment #9)

...
%f prints the value as -nan so you are correct. but both the directx manual and the opengl spec define the values as >= 0, so the check should be correct.

Yes, but I think it should be unnecessary, since GL should take care of it.

No it shouldnt. the GL manual says it should return an GL_INVALID_VALUE which would still be an error in wine but probably with lesser consequences.

GL commands generating errors are ignored, with only a few exceptions. I.e., if the GL driver would correctly handle the NaN, the effect should be the same as what your patch does.

It's possible NaN is supposed to behave in a specific way in D3D, but that would need some tests, and I suspect it's just undefined.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

26 Apr 26 Apr

4:27 p.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #13 from farmboy0+winehq@googlemail.com --- After some more research I have come to the conclusion that the mesa implementation is in accordance with the OpenGL 2.1 spec. (http://www.opengl.org/sdk/docs/man2/xhtml/glLight.xml)

The spec says: GL_CONSTANT_ATTENUATION GL_LINEAR_ATTENUATION GL_QUADRATIC_ATTENUATION params is a single integer or floating-point value that specifies one of the three light attenuation factors. Integer and floating-point values are mapped directly.

**Only nonnegative values are accepted.**

etc..

It also says: GL_INVALID_VALUE is generated ... if a negative attenuation factor is specified.

NaN is neither nonnegative nor negative thus it is neither accepted nor is an error generated which is exactly what Mesa does.

This means that it is wine's responsibility to reject NaN values.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

9 Sep 9 Sep

8:37 a.m.

New subject: [Bug 35963] crash in Syberia2

--- Comment #14 from Austin English austinenglish@gmail.com --- This is your friendly reminder that there has been no bug activity for over a year. Is this still an issue in current (1.7.51 or newer) wine?

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

24 Apr 24 Apr

4:15 a.m.

New subject: [Bug 35963] crash in Syberia2

https://www.origin.com/fin/fi-fi/store/free-games

winetest@luukku.com changed:

What |Removed |Added ---------------------------------------------------------------------------- CC| |winetest@luukku.com

--- Comment #15 from winetest@luukku.com --- This game is available for free for limited time from origin.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

18 Feb 18 Feb

4:37 a.m.

New subject: [Bug 35963] crash in Syberia2

joaopa jeremielapuree@yahoo.fr changed:

What |Removed |Added ---------------------------------------------------------------------------- CC| |jeremielapuree@yahoo.fr

--- Comment #16 from joaopa jeremielapuree@yahoo.fr --- Does the bug still occur with wine-4.2?

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

wine-bugs＠winehq.org

10:31 a.m.

New subject: [Bug 35963] crash in Syberia2

Sergey Isakov isakov-sl@bk.ru changed:

What |Removed |Added ---------------------------------------------------------------------------- CC| |isakov-sl@bk.ru

--- Comment #17 from Sergey Isakov isakov-sl@bk.ru --- (In reply to joaopa from comment #16)