[Bug 28799] New: d3dx9_36/tests/mesh.ok: memcpy source and destination overlap in weld_float16_2() triggered by test_weld_vertices()
http://bugs.winehq.org/show_bug.cgi?id=28799 Bug #: 28799 Summary: d3dx9_36/tests/mesh.ok: memcpy source and destination overlap in weld_float16_2() triggered by test_weld_vertices() Product: Wine Version: 1.3.30 Platform: x86 OS/Version: Linux Status: NEW Severity: normal Priority: P2 Component: directx-d3dx9 AssignedTo: wine-bugs(a)winehq.org ReportedBy: dank(a)kegel.com CC: wine-bugs(a)winehq.org Classification: Unclassified In d3dx9_36/tests, in "make mesh.ok", valgrind complains: Source and destination overlap in memcpy(0x7fd893dc, 0x7fd893dc, 4) at memcpy (mc_replace_strmem.c:635) by weld_float16_2 (mesh.c:5861) by weld_component (mesh.c:5958) by D3DXWeldVertices (mesh.c:6242) by test_weld_vertices (mesh.c:8069) by func_mesh (mesh.c:8193) by run_test (test.h:556) by main (test.h:624) -- 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=28799 --- Comment #1 from Dan Kegel <dank(a)kegel.com> 2011-10-19 11:26:24 CDT --- Also see the same backtrace but with weld_float16_4 and weld_float16_1 (iirc). -- 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=28799 Michael Mc Donnell <michael(a)mcdonnell.dk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |michael(a)mcdonnell.dk --- Comment #2 from Michael Mc Donnell <michael(a)mcdonnell.dk> 2012-03-21 04:49:32 CDT --- Thanks for reporting it Dan. I'll have a look at it. -- 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=28799 --- Comment #3 from Michael Mc Donnell <michael(a)mcdonnell.dk> 2012-03-21 18:12:21 CDT --- Created attachment 39490 --> http://bugs.winehq.org/attachment.cgi?id=39490 Patch for not welding self It looks like the error message is caused by the destination and source being the same. This patch makes sure that a vertex is not welded with itself. -- 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=28799 Bruno Jesus <00cpxxx(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |00cpxxx(a)gmail.com --- Comment #4 from Bruno Jesus <00cpxxx(a)gmail.com> 2012-03-21 18:36:11 CDT --- Isn't it simpler to use memmove instead? Although your solution may be faster. -- 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=28799 --- Comment #5 from Michael Mc Donnell <michael(a)mcdonnell.dk> 2012-03-22 04:27:45 CDT --- (In reply to comment #4)
Isn't it simpler to use memmove instead? Although your solution may be faster.
Using a memmove would just hide the problem. There should not be any overlapping memory regions at all. -- 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=28799 --- Comment #6 from Michael Mc Donnell <michael(a)mcdonnell.dk> 2012-03-29 17:29:36 CDT --- The fix has been committed(f144626e81c3b60bbc9a9c028e4a8ec1f5d96c24). Please re-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=28799 --- Comment #7 from Michael Mc Donnell <michael(a)mcdonnell.dk> 2012-05-31 12:54:55 CDT --- Can someone please mark this as fixed? -- 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=28799 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |f144626e81c3b60bbc9a9c028e4 | |a8ec1f5d96c24 Status|NEW |RESOLVED Resolution| |FIXED --- Comment #8 from Austin English <austinenglish(a)gmail.com> 2012-05-31 13:01:00 CDT --- Fixed. -- 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=28799 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #9 from Alexandre Julliard <julliard(a)winehq.org> 2012-06-08 15:28:39 CDT --- Closing bugs fixed in 1.5.6. -- 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=28799 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |1.4.x -- 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=28799 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|1.4.x |--- -- 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