[Bug 40114] New: mshtml:style regression (crash)
https://bugs.winehq.org/show_bug.cgi?id=40114 Bug ID: 40114 Summary: mshtml:style regression (crash) Product: Wine Version: 1.9.3 Hardware: x86 OS: Windows Status: NEW Severity: normal Priority: P2 Component: mshtml Assignee: wine-bugs(a)winehq.org Reporter: fgouget(a)codeweavers.com The commit below causes mshtml:style to crash on machines with Internet Explorer 6 such as the fg-winxp-1sp and fg-winxp-2p VMs. As a comparison point one can look at fg-winxp-2pie7 which is the same Windows XP SP2 machine with IE 7 installed. Crash: style.c:2938: Test failed: Could not get IHTMLCurrentStyle4 iface: 80004002 style.c:2938: this is the last test seen before the exception style: unhandled exception c0000005 at 0046287F test failed: crash This points the finger at the IHTMLCurrentStyle4_get_minWidth() call near the end of test_current_style(). mshtml:style results on test.winehq.org: https://test.winehq.org/data/tests/mshtml:style.html commit 27182667ccbe6f903ab1986bf89f9e4b392c9fb8 Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Jan 11 13:52:07 2016 +0100 mshtml: Added IHTMLCurrentStyle4::get_maxHeight implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40114 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download, source, testcase URL| |https://test.winehq.org/dat | |a/tests/mshtml:style.html Regression SHA1| |27182667ccbe6f903ab1986bf89 | |f9e4b392c9fb8 -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40114 Jacek Caban <jacek(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jacek(a)codeweavers.com --- Comment #1 from Jacek Caban <jacek(a)codeweavers.com> --- Patch sent, thanks for the report: http://source.winehq.org/patches/data/118921 -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40114 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40114 Jacek Caban <jacek(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |433f82c74e5b203d35b00f01584 | |1ec2b448542eb Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #2 from Jacek Caban <jacek(a)codeweavers.com> --- Fixed in Git. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40114 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #3 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 1.9.4. -- 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