29407 2009-09-18 07:32:22 -0700 Webkit on SPARC Solaris has wrong endian 2011-06-08 10:14:47 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Other Other RESOLVED FIXED P2 Normal --- 1 vestbo ademar ademar bugs-webkit diegohcg hausmann kent.hansen kling nils.roscher-nielsen pierre.rossi tonikitoo webkit.review.bot oldest_to_newest 148112 0 vestbo 2009-09-18 07:32:22 -0700 This bug report originated from issue QTBUG-4078 <http://bugreports.qt.nokia.com/browse/QTBUG-4078> --- Description --- Webkit on SPARC Solaris have wrong endian. This results in a run time crash. 158494 1 nils.roscher-nielsen 2009-10-28 02:25:26 -0700 Here is a stack trace from the crash: #0 0xfe8de3f8 in WTF::HashSet<WebCore::StringImpl*, WebCore::StringHash, WTF::HashTraits<WebCore::StringImpl*> >::add<WebCore::UCharBuffer, WebCore::UCharBufferTranslator> () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #1 0xfe247bc4 in WebCore::AtomicString::add () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #2 0xfedd914c in WebCore::CSSParserString::operator WebCore::AtomicString () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #3 0xfe695604 in cssyyparse () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #4 0xfdfd6d8c in WebCore::CSSParser::parseSheet () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #5 0xfe027050 in WebCore::CSSStyleSheet::parseString () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #6 0xfdffd4e4 in WebCore::parseUASheet () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #7 0xfdffd50c in WebCore::parseUASheet () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #8 0xfe025f90 in WebCore::loadSimpleDefaultStyle () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #9 0xfe026228 in WebCore::CSSStyleSelector::CSSStyleSelector () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #10 0xfe03dc58 in WebCore::Document::attach () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #11 0xfe22c158 in WebCore::Frame::setDocument () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQt---Type <return> to continue, or q <return> to quit--- WebKit.so.4 #12 0xfe1b8fcc in WebCore::FrameLoader::begin () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #13 0xfe1c8390 in WebCore::FrameLoader::init () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #14 0xfe39893c in QWebFramePrivate::init () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #15 0xfe39b0d0 in QWebFrame::QWebFrame () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #16 0xfe39f4d0 in QWebPagePrivate::createMainFrame () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #17 0xfe39f5b4 in QWebPage::mainFrame () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #18 0xfe39f5e4 in QWebPage::setViewportSize () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #19 0xfe39f6f4 in QWebPage::setView () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #20 0xfe39f778 in QWebPage::QWebPage () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #21 0xfe3a3cb0 in QWebView::page () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #22 0xfe3a3cd8 in QWebView::load () from /export/disc2/workspaces/branch_3_9/workspace/ExternLibs/Qt4/x11-src/qt-all-commercial-src-4.5.3/lib//libQtWebKit.so.4 #23 0x00013970 in MainWindow::MainWindow () #24 0x00012e74 in main () (gdb) (gdb) 200364 2 diegohcg 2010-03-16 12:29:05 -0700 In this bug the OS should be Other, not Mac OS 200694 3 kent.hansen 2010-03-17 05:01:26 -0700 This is not Qt-specific. Anyways, I think it might be fixed already, since trunk now has more cases for detecting SPARC (in particular, 32-bit) than it did in the version shipped with Qt 4.5. It's possible that he was building on 32-bit SPARC, in which case WTF_PLATFORM_BIG_ENDIAN would not be correctly defined with that version. Nils, could you perhaps try with a more recent WebKit (the one in 4.6 seems to have the proper defines too)? 201013 4 hausmann 2010-03-17 15:49:29 -0700 Assigning to Niels, since he has a bugzilla account. Niels, can you quickly verify this on your Solaris box with at least Qt 4.6? The original bug report says Qt 4.5 was affected. Thanks! 225286 5 jesus 2010-05-13 07:40:47 -0700 (In reply to comment #4) > Niels, can you quickly verify this on your Solaris box with at least Qt 4.6? The original bug report says Qt 4.5 was affected. Any updates on this? Thanks! 414842 6 bugs-webkit 2011-06-03 06:35:59 -0700 We just saw this crash with qt-4.7.3 on Solaris 8/sparc built with gcc. It seems to be a bus error which can be fixed by fixing the // FIXME comments about going 4 bytes at a time in src/3rdparty/webkit/WebCore/platform/text/AtomicString.cpp and src/3rdparty/webkit/WebCore/platform/text/StringHash.h We also built webkitgtk-1.3.10 on Solaris 8, which appears to be based on a newer version of webkit than that shipped with qt-4.7, and did not see this problem, so it is probably fixed in later qtwebkit versions also. 414855 7 ademar 2011-06-03 07:06:12 -0700 We had a fix for a similar problem a while ago (see bug 57256), but I have the impression that there are still a few missing bugs. I don't have a sparc machine to test... Are you able to build a recent webkit to test this? (such as our next release: qtwebkit-2.2, see http://trac.webkit.org/wiki/QtWebKitRelease22) Anyway, cooking a patch to include a couple of missing checks. Will upload it in a second. 414859 8 95908 ademar 2011-06-03 07:14:00 -0700 Created attachment 95908 patch 417269 9 95908 kling 2011-06-08 09:02:52 -0700 Comment on attachment 95908 patch LGTM. 417320 10 95908 webkit.review.bot 2011-06-08 10:14:42 -0700 Comment on attachment 95908 patch Clearing flags on attachment: 95908 Committed r88355: <http://trac.webkit.org/changeset/88355> 417321 11 webkit.review.bot 2011-06-08 10:14:47 -0700 All reviewed patches have been landed. Closing bug. 95908 2011-06-03 07:14:00 -0700 2011-06-08 10:14:42 -0700 patch 0001-Webkit-on-SPARC-Solaris-has-wrong-endian.patch text/plain 2918 ademar RnJvbSBhMWFlNWJjZDE2YmUxZmE4NmM3NDRjODA1ZmM0Zjk5NWYwYjEwODYxIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBBZGVtYXIgZGUgU291emEgUmVpcyBKciA8YWRlbWFyLnJlaXNA b3BlbmJvc3NhLm9yZz4KRGF0ZTogRnJpLCAzIEp1biAyMDExIDExOjEzOjAwIC0wMzAwClN1Ympl Y3Q6IFtQQVRDSF0gV2Via2l0IG9uIFNQQVJDIFNvbGFyaXMgaGFzIHdyb25nIGVuZGlhbgoKaHR0 cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI5NDA3Ci0tLQogU291cmNlL0ph dmFTY3JpcHRDb3JlL0NoYW5nZUxvZyAgICAgICAgICAgICB8ICAgMjEgKysrKysrKysrKysrKysr KysrKysrCiBTb3VyY2UvSmF2YVNjcmlwdENvcmUvcnVudGltZS9VU3RyaW5nLmggICAgIHwgICAg MiArLQogU291cmNlL0phdmFTY3JpcHRDb3JlL3d0Zi90ZXh0L1N0cmluZ0hhc2guaCB8ICAgIDIg Ky0KIDMgZmlsZXMgY2hhbmdlZCwgMjMgaW5zZXJ0aW9ucygrKSwgMiBkZWxldGlvbnMoLSkKCmRp ZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFT Y3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCAyOTA2M2Y1Li45N2M4YjJmIDEwMDc1NQotLS0gYS9T b3VyY2UvSmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29y ZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwyNCBAQAorMjAxMS0wNi0wMyAgQWRlbWFyIGRlIFNvdXph IFJlaXMgSnIuICA8YWRlbWFyLnJlaXNAb3BlbmJvc3NhLm9yZz4KKworICAgICAgICBSZXZpZXdl ZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBXZWJraXQgb24gU1BBUkMgU29sYXJpcyBo YXMgd3JvbmcgZW5kaWFuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn LmNnaT9pZD0yOTQwNworCisgICAgICAgIEJ1ZyA1NzI1NiBmaXhlZCBvbmUgY3Jhc2ggb24gbWlz YWxpZ25lZCByZWFkcyBvbiBzcGFyYy9zb2xhcmlzLCBidXQKKyAgICAgICAgdGhlcmUgYXJlIG1v cmUgb2N1cnJlbmNlcyBvZiB0aGUgc2FtZSBjb2RlIHBhdHRlcm4gaW4gd2Via2l0LgorCisgICAg ICAgIFRoaXMgcGF0Y2ggaW5jbHVkZXMgdGhlIGNoZWNrIG9uIHRoZXNlIG90aGVyIHBhcnRzIG9m IHRoZSBjb2RlLgorCisgICAgICAgIFRoaXMgaXMgYSBzcGVjdWxhdGl2ZSBmaXgsIEkgZG9uJ3Qg aGF2ZSBhIHNwYXJjIG1hY2hpbmUgdG8gdGVzdCBhbmQKKyAgICAgICAgZG9uJ3Qga25vdyB3aGlj aCBraW5kIG9mIHRlc3Qgd291bGQgdHJpZ2dlciBhIGNyYXNoIChidXQgaXQncyBxdWl0ZQorICAg ICAgICBvYnZpb3VzIHRoYXQgaXQncyB0aGUgc2FtZSBjb2RlIGR1cGxpY2F0ZWQgaW4gZGlmZmVy ZW50IGZpbGVzKS4KKworICAgICAgICAqIHJ1bnRpbWUvVVN0cmluZy5oOgorICAgICAgICAoSlND OjpVU3RyaW5nSGFzaDo6ZXF1YWwpOgorICAgICAgICAqIHd0Zi90ZXh0L1N0cmluZ0hhc2guaDoK KyAgICAgICAgKFdURjo6U3RyaW5nSGFzaDo6ZXF1YWwpOgorCiAyMDExLTA2LTAzICBBbGV4aXMg TWVuYXJkICA8YWxleGlzLm1lbmFyZEBvcGVuYm9zc2Eub3JnPgogCiAgICAgICAgIFJldmlld2Vk IGJ5IEJlbmphbWluIFBvdWxhaW4uCmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlwdENvcmUv cnVudGltZS9VU3RyaW5nLmggYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvcnVudGltZS9VU3RyaW5n LmgKaW5kZXggYjk4ZTdiNC4uMmQ3NjgwOSAxMDA2NDQKLS0tIGEvU291cmNlL0phdmFTY3JpcHRD b3JlL3J1bnRpbWUvVVN0cmluZy5oCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9ydW50aW1l L1VTdHJpbmcuaApAQCAtMjAyLDcgKzIwMiw3IEBAIHN0cnVjdCBVU3RyaW5nSGFzaCB7CiAKICAg ICAgICAgLy8gRklYTUU6IHBlcmhhcHMgd2Ugc2hvdWxkIGhhdmUgYSBtb3JlIGFic3RyYWN0IG1h Y3JvIHRoYXQgaW5kaWNhdGVzIHdoZW4KICAgICAgICAgLy8gZ29pbmcgNCBieXRlcyBhdCBhIHRp bWUgaXMgdW5zYWZlCi0jaWYgQ1BVKEFSTSkgfHwgQ1BVKFNINCkgfHwgQ1BVKE1JUFMpCisjaWYg Q1BVKEFSTSkgfHwgQ1BVKFNINCkgfHwgQ1BVKE1JUFMpIHx8IENQVShTUEFSQykKICAgICAgICAg Y29uc3QgVUNoYXIqIGFDaGFycyA9IGEtPmNoYXJhY3RlcnMoKTsKICAgICAgICAgY29uc3QgVUNo YXIqIGJDaGFycyA9IGItPmNoYXJhY3RlcnMoKTsKICAgICAgICAgZm9yICh1bnNpZ25lZCBpID0g MDsgaSAhPSBhTGVuZ3RoOyArK2kpIHsKZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0Q29y ZS93dGYvdGV4dC9TdHJpbmdIYXNoLmggYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvd3RmL3RleHQv U3RyaW5nSGFzaC5oCmluZGV4IGM0N2MxZDguLjQ2MzdjM2QgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9K YXZhU2NyaXB0Q29yZS93dGYvdGV4dC9TdHJpbmdIYXNoLmgKKysrIGIvU291cmNlL0phdmFTY3Jp cHRDb3JlL3d0Zi90ZXh0L1N0cmluZ0hhc2guaApAQCAtNTUsNyArNTUsNyBAQCBuYW1lc3BhY2Ug V1RGIHsKIAogICAgICAgICAgICAgLy8gRklYTUU6IHBlcmhhcHMgd2Ugc2hvdWxkIGhhdmUgYSBt b3JlIGFic3RyYWN0IG1hY3JvIHRoYXQgaW5kaWNhdGVzIHdoZW4KICAgICAgICAgICAgIC8vIGdv aW5nIDQgYnl0ZXMgYXQgYSB0aW1lIGlzIHVuc2FmZQotI2lmIENQVShBUk0pIHx8IENQVShTSDQp IHx8IENQVShNSVBTKQorI2lmIENQVShBUk0pIHx8IENQVShTSDQpIHx8IENQVShNSVBTKSB8fCBD UFUoU1BBUkMpCiAgICAgICAgICAgICBjb25zdCBVQ2hhciogYUNoYXJzID0gYS0+Y2hhcmFjdGVy cygpOwogICAgICAgICAgICAgY29uc3QgVUNoYXIqIGJDaGFycyA9IGItPmNoYXJhY3RlcnMoKTsK ICAgICAgICAgICAgIGZvciAodW5zaWduZWQgaSA9IDA7IGkgIT0gYUxlbmd0aDsgKytpKSB7Ci0t IAoxLjcuMy40Cgo=