[Bug 15494] New: ddraw_test throws an exception when run
http://bugs.winehq.org/show_bug.cgi?id=15494 Summary: ddraw_test throws an exception when run Product: Wine Version: 1.1.5 Platform: PC OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown AssignedTo: wine-bugs(a)winehq.org ReportedBy: celticht32(a)aol.com Running ddraw_test on wine with make test from the directory dlls/ddraw/tests/ throws the attached exception. ATI FireGL V5200 3 Gigs of memory RHEL 5.2 -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 --- Comment #1 from chris ahrendt <celticht32(a)aol.com> 2008-10-01 20:31:07 --- Created an attachment (id=16409) --> (http://bugs.winehq.org/attachment.cgi?id=16409) Exception from running d3d test -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 --- Comment #2 from Lei Zhang <thestig(a)google.com> 2008-10-03 13:34:13 --- Created an attachment (id=16449) --> (http://bugs.winehq.org/attachment.cgi?id=16449) test patch Does this patch help? -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 --- Comment #3 from chris ahrendt <celticht32(a)aol.com> 2008-10-05 20:00:20 --- Yes the exception no longer occurs with the patch... however it does not explain how the variables are not set. Also there are alot more Release calls which are not checked. So not sure that this really fixes the bug or not as it doesn't determine why the variable is null in the first place. Chris -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 --- Comment #4 from chris ahrendt <celticht32(a)aol.com> 2008-10-07 20:09:29 --- Created an attachment (id=16529) --> (http://bugs.winehq.org/attachment.cgi?id=16529) addition to the first test patch I took your patch and expanded on it.. checked the variables before every free where its not currently checked.. this allows the test to run.. but it still makes me wonder why they were not initialized in the first place. But I guess we can close the bug and submit the patch to the git tree.. chris -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 chris ahrendt <celticht32(a)aol.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Difficulty|--- |Minutes Component|-unknown |testcases Resolution| |FIXED --- Comment #5 from chris ahrendt <celticht32(a)aol.com> 2008-10-08 07:00:41 --- additional patch ontop of lei's patch fixes issues... lei's patch fixes the exception my patch extends the checking to whole program. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 --- Comment #6 from Roderick Colenbrander <thunderbird2k(a)gmx.net> 2008-10-08 07:06:00 --- The patches aren't bad but I think they are unneeded as clearly the CreateSurface call seems to fail and for that reason a test should abort and not continue. That's I think the issue. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 James Hawkins <truiken(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |UNCONFIRMED Difficulty|Minutes |--- Resolution|FIXED | --- Comment #7 from James Hawkins <truiken(a)gmail.com> 2008-10-08 12:00:06 --- A bug is not fixed until the patches are committed... -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 --- Comment #8 from chris ahrendt <celticht32(a)aol.com> 2008-10-08 12:02:58 --- So should I open a bug with the CreateSurface call? So I probably should put out an error message when I get a null value as well as just checking for it? Chris -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 --- Comment #9 from chris ahrendt <celticht32(a)aol.com> 2008-10-13 10:00:28 --- The exception is actually caused by the code not checking the return for Create Surface call... am working on fixing the patch to just have that... chris -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 --- Comment #10 from Austin English <austinenglish(a)gmail.com> 2009-04-17 12:21:20 --- Is this still an issue in current (1.1.19 or newer) wine? -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 chris ahrendt <celticht32(a)aol.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |FIXED --- Comment #11 from chris ahrendt <celticht32(a)aol.com> 2009-04-17 19:52:04 --- Fixed with the latest GIT Tree no execptions.. however lots of ERR -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=15494 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #12 from Alexandre Julliard <julliard(a)winehq.org> 2009-04-24 12:17:13 --- Closing bugs fixed in 1.1.20. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
participants (1)
-
wine-bugs＠winehq.org