114610 2013-04-15 02:18:56 -0700 [BlackBerry] Filesystem callback notifyCreateSnapshotFileAndReadMetadata can be called on the wring thread 2013-04-15 03:10:17 -0700 1 1 1 Unclassified WebKit WebKit BlackBerry 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 cgarcia webkit-unassigned commit-queue leoyang mifenton rwlbuis tonikitoo xan.lopez oldest_to_newest 875526 0 cgarcia 2013-04-15 02:18:56 -0700 This causes an assert in debug builds: Program terminated with signal 11, Segmentation fault. #0 0x794068ba in JSC::Heap::allocateWithImmortalStructureDestructor (this=0x8447e40, bytes=64) at /home/cgarcia/rim/webkit/Source/JavaScriptCore/heap/Heap.h:377 377 ASSERT(isValidAllocation(bytes)); (gdb) bt #0 0x794068ba in JSC::Heap::allocateWithImmortalStructureDestructor (this=0x8447e40, bytes=64) at /home/cgarcia/rim/webkit/Source/JavaScriptCore/heap/Heap.h:377 #1 0x7940cece in JSC::allocateCell<JSC::Structure> (heap=..., size=64) at /home/cgarcia/rim/webkit/Source/JavaScriptCore/runtime/JSCellInlines.h:92 #2 0x7940b67a in JSC::allocateCell<JSC::Structure> (heap=...) at /home/cgarcia/rim/webkit/Source/JavaScriptCore/runtime/JSCellInlines.h:104 #3 0x7940a262 in JSC::Structure::create (globalData=..., globalObject=0x89bf838, prototype=..., typeInfo=..., classInfo=0x7faac148, indexingType=0 '\000', inlineCapacity=0) at /home/cgarcia/rim/webkit/Source/JavaScriptCore/runtime/StructureInlines.h:38 #4 0x7d4e3d04 in WebCore::JSBlobPrototype::createStructure (globalData=..., globalObject=0x89bf838, prototype=...) at /home/cgarcia/rim/webkit/WebKitBuild/armle-v7/Debug/DerivedSources/WebCore/JSBlob.h:104 #5 0x7d4e322e in WebCore::JSBlob::createPrototype (exec=0x89bf9a0, globalObject=0x89bf838) at /home/cgarcia/rim/webkit/WebKitBuild/armle-v7/Debug/DerivedSources/WebCore/JSBlob.cpp:146 #6 0x7d2de2c0 in WebCore::getDOMStructure<WebCore::JSBlob> (exec=0x89bf9a0, globalObject=0x89bf838) at /home/cgarcia/rim/webkit/Source/WebCore/bindings/js/JSDOMBinding.h:116 #7 0x7d4e4062 in WebCore::getDOMPrototype<WebCore::JSBlob> (exec=0x89bf9a0, globalObject=0x89bf838) at /home/cgarcia/rim/webkit/Source/WebCore/bindings/js/JSDOMBinding.h:127 #8 0x7d4e308a in WebCore::JSBlobPrototype::self (exec=0x89bf9a0, globalObject=0x89bf838) at /home/cgarcia/rim/webkit/WebKitBuild/armle-v7/Debug/DerivedSources/WebCore/JSBlob.cpp:110 #9 0x7d4e4c02 in WebCore::JSFile::createPrototype (exec=0x89bf9a0, globalObject=0x89bf838) at /home/cgarcia/rim/webkit/WebKitBuild/armle-v7/Debug/DerivedSources/WebCore/JSFile.cpp:121 #10 0x7d2de44c in WebCore::getDOMStructure<WebCore::JSFile> (exec=0x89bf9a0, globalObject=0x89bf838) at /home/cgarcia/rim/webkit/Source/WebCore/bindings/js/JSDOMBinding.h:116 #11 0x7d2de1d4 in WebCore::createWrapper<WebCore::JSFile, WebCore::File> (exec=0x89bf9a0, globalObject=0x89bf838, node=0x82d8a70) at /home/cgarcia/rim/webkit/Source/WebCore/bindings/js/JSDOMBinding.h:186 #12 0x7d2ddbf8 in WebCore::wrap<WebCore::JSFile, WebCore::File> (exec=0x89bf9a0, globalObject=0x89bf838, domObject=0x82d8a70) at /home/cgarcia/rim/webkit/Source/WebCore/bindings/js/JSDOMBinding.h:199 #13 0x7d4e500c in WebCore::toJS (exec=0x89bf9a0, globalObject=0x89bf838, impl=0x82d8a70) at /home/cgarcia/rim/webkit/WebKitBuild/armle-v7/Debug/DerivedSources/WebCore/JSFile.cpp:171 #14 0x7d3ad830 in WebCore::JSFileCallback::handleEvent (this=0x8b03b50, file=0x82d8a70) at /home/cgarcia/rim/webkit/WebKitBuild/armle-v7/Debug/DerivedSources/WebCore/JSFileCallback.cpp:68 #15 0x7c5397ae in WebCore::(anonymous namespace)::SnapshotFileCallback::didCreateSnapshotFile (this=0x84b5998, metadata=..., snapshot=...) at /home/cgarcia/rim/webkit/Source/WebCore/Modules/filesystem/DOMFileSystem.cpp:174 #16 0x7d879198 in WebCore::PlatformAsyncFileSystemCallbacks::notifyCreateSnapshotFileAndReadMetadata (this=0x83ecdb8, fileInfo=...) at /home/cgarcia/rim/webkit/Source/WebCore/platform/blackberry/PlatformAsyncFileSystemCallbacks.cpp:75 875529 1 198039 cgarcia 2013-04-15 02:22:49 -0700 Created attachment 198039 Patch 875541 2 198039 xan.lopez 2013-04-15 02:36:46 -0700 Comment on attachment 198039 Patch Makes sense, seems every other thing in the file is following the same pattern. 875554 3 198039 commit-queue 2013-04-15 03:10:15 -0700 Comment on attachment 198039 Patch Clearing flags on attachment: 198039 Committed r148425: <http://trac.webkit.org/changeset/148425> 875555 4 commit-queue 2013-04-15 03:10:17 -0700 All reviewed patches have been landed. Closing bug. 198039 2013-04-15 02:22:49 -0700 2013-04-15 03:10:14 -0700 Patch bb-filesystem-crash.diff text/plain 5195 cgarcia ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCBmZDA3YTYzLi43MTcyNmI4IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjAg QEAKKzIwMTMtMDQtMTUgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t PgorCisgICAgICAgIFtCbGFja0JlcnJ5XSBGaWxlc3lzdGVtIGNhbGxiYWNrIG5vdGlmeUNyZWF0 ZVNuYXBzaG90RmlsZUFuZFJlYWRNZXRhZGF0YSBjYW4gYmUgY2FsbGVkIG9uIHRoZSB3cmluZyB0 aHJlYWQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEx NDYxMAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIE92 ZXJyaWRlIG5vdGlmeUNyZWF0ZVNuYXBzaG90RmlsZUFuZFJlYWRNZXRhZGF0YSB0byBtYWtlIHN1 cmUgdGhlCisgICAgICAgIGNhbGxiYWNrIGlzIGNhbGxlZCBvbiB0aGUgcmlnaHQgdGhyZWFkLgor CisgICAgICAgIEZpeGVzIGEgY3Jhc2ggaW4gc2V2ZXJhbCBmYXN0L2ZpbGVzeXN0ZW0gdGVzdCBp biBkZWJ1ZyBidWlsZHMuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ibGFja2JlcnJ5L1dvcmtlclBs YXRmb3JtQXN5bmNGaWxlU3lzdGVtQ2FsbGJhY2tzLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6Oldv cmtlclBsYXRmb3JtQXN5bmNGaWxlU3lzdGVtQ2FsbGJhY2tzOjpub3RpZnlDcmVhdGVTbmFwc2hv dEZpbGVBbmRSZWFkTWV0YWRhdGEpOgorICAgICAgICAoV2ViQ29yZTo6V29ya2VyUGxhdGZvcm1B c3luY0ZpbGVTeXN0ZW1DYWxsYmFja3M6Om5vdGlmeUNyZWF0ZVNuYXBzaG90RmlsZUFuZFJlYWRN ZXRhZGF0YU9uV29ya2VyVGhyZWFkKToKKyAgICAgICAgKiBwbGF0Zm9ybS9ibGFja2JlcnJ5L1dv cmtlclBsYXRmb3JtQXN5bmNGaWxlU3lzdGVtQ2FsbGJhY2tzLmg6CisKIDIwMTMtMDQtMTEgIEFy dmlkIE5pbHNzb24gIDxhbmlsc3NvbkByaW0uY29tPgogCiAgICAgICAgIFtCbGFja0JlcnJ5XVtD U1MgRmlsdGVyc10gQmx1ciBmaWx0ZXIgZmFpbHMgdG8gcmVjb21wdXRlIGJsdXIgc2l6ZSB3aGVu IGxheWVyIHNpemUgY2hhbmdlcwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0v YmxhY2tiZXJyeS9Xb3JrZXJQbGF0Zm9ybUFzeW5jRmlsZVN5c3RlbUNhbGxiYWNrcy5jcHAgYi9T b3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ibGFja2JlcnJ5L1dvcmtlclBsYXRmb3JtQXN5bmNGaWxl U3lzdGVtQ2FsbGJhY2tzLmNwcAppbmRleCBiOGI1OWM4Li4wYmM4MDdhIDEwMDY0NAotLS0gYS9T b3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ibGFja2JlcnJ5L1dvcmtlclBsYXRmb3JtQXN5bmNGaWxl U3lzdGVtQ2FsbGJhY2tzLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ibGFja2Jl cnJ5L1dvcmtlclBsYXRmb3JtQXN5bmNGaWxlU3lzdGVtQ2FsbGJhY2tzLmNwcApAQCAtMTMwLDYg KzEzMCwxOSBAQCB2b2lkIFdvcmtlclBsYXRmb3JtQXN5bmNGaWxlU3lzdGVtQ2FsbGJhY2tzOjpu b3RpZnlSZWFkTWV0YWRhdGEoY29uc3QgQmxhY2tCZXJyeQogICAgIG1fbXV0ZXgudW5sb2NrKCk7 CiB9CiAKK3ZvaWQgV29ya2VyUGxhdGZvcm1Bc3luY0ZpbGVTeXN0ZW1DYWxsYmFja3M6Om5vdGlm eUNyZWF0ZVNuYXBzaG90RmlsZUFuZFJlYWRNZXRhZGF0YShjb25zdCBCbGFja0JlcnJ5OjpQbGF0 Zm9ybTo6V2ViRmlsZUluZm8mIGZpbGVJbmZvKQoreworICAgIG1fbXV0ZXgubG9jaygpOworICAg IGlmICghbV9jb250ZXh0KSB7CisgICAgICAgIG1fbXV0ZXgudW5sb2NrKCk7CisgICAgICAgIFBs YXRmb3JtQXN5bmNGaWxlU3lzdGVtQ2FsbGJhY2tzOjpkZWxldGVNZSgpOworICAgICAgICByZXR1 cm47CisgICAgfQorCisgICAgcG9zdFRhc2tUb1dvcmtlclRocmVhZChjcmVhdGVDYWxsYmFja1Rh c2soJm5vdGlmeUNyZWF0ZVNuYXBzaG90RmlsZUFuZFJlYWRNZXRhZGF0YU9uV29ya2VyVGhyZWFk LCB0aGlzLCBmaWxlSW5mbykpOworICAgIG1fbXV0ZXgudW5sb2NrKCk7Cit9CisKIHZvaWQgV29y a2VyUGxhdGZvcm1Bc3luY0ZpbGVTeXN0ZW1DYWxsYmFja3M6Om5vdGlmeVJlYWREaXJlY3Rvcnko Y29uc3Qgc3RkOjp2ZWN0b3I8QmxhY2tCZXJyeTo6UGxhdGZvcm06OldlYkZpbGVTeXN0ZW1FbnRy eT4mIGVudHJpZXMsIGJvb2wgaGFzTW9yZSkKIHsKICAgICBtX211dGV4LmxvY2soKTsKQEAgLTIw MCw2ICsyMTMsMTEgQEAgdm9pZCBXb3JrZXJQbGF0Zm9ybUFzeW5jRmlsZVN5c3RlbUNhbGxiYWNr czo6bm90aWZ5UmVhZE1ldGFkYXRhT25Xb3JrZXJUaHJlYWQoU2MKICAgICBjYWxsYmFja3MtPlBs YXRmb3JtQXN5bmNGaWxlU3lzdGVtQ2FsbGJhY2tzOjpub3RpZnlSZWFkTWV0YWRhdGEoZmlsZUlu Zm8pOwogfQogCit2b2lkIFdvcmtlclBsYXRmb3JtQXN5bmNGaWxlU3lzdGVtQ2FsbGJhY2tzOjpu b3RpZnlDcmVhdGVTbmFwc2hvdEZpbGVBbmRSZWFkTWV0YWRhdGFPbldvcmtlclRocmVhZChTY3Jp cHRFeGVjdXRpb25Db250ZXh0KiwgV29ya2VyUGxhdGZvcm1Bc3luY0ZpbGVTeXN0ZW1DYWxsYmFj a3MqIGNhbGxiYWNrcywgY29uc3QgQmxhY2tCZXJyeTo6UGxhdGZvcm06OldlYkZpbGVJbmZvJiBm aWxlSW5mbykKK3sKKyAgICBjYWxsYmFja3MtPlBsYXRmb3JtQXN5bmNGaWxlU3lzdGVtQ2FsbGJh Y2tzOjpub3RpZnlDcmVhdGVTbmFwc2hvdEZpbGVBbmRSZWFkTWV0YWRhdGEoZmlsZUluZm8pOwor fQorCiB2b2lkIFdvcmtlclBsYXRmb3JtQXN5bmNGaWxlU3lzdGVtQ2FsbGJhY2tzOjpub3RpZnlS ZWFkRGlyZWN0b3J5RW50cnlPbldvcmtlclRocmVhZChTY3JpcHRFeGVjdXRpb25Db250ZXh0Kiwg V29ya2VyUGxhdGZvcm1Bc3luY0ZpbGVTeXN0ZW1DYWxsYmFja3MqIGNhbGxiYWNrcywgY29uc3Qg c3RkOjp2ZWN0b3I8QmxhY2tCZXJyeTo6UGxhdGZvcm06OldlYkZpbGVTeXN0ZW1FbnRyeT4mIGVu dHJpZXMsIGJvb2wgaGFzTW9yZSkKIHsKICAgICBjYWxsYmFja3MtPlBsYXRmb3JtQXN5bmNGaWxl U3lzdGVtQ2FsbGJhY2tzOjpub3RpZnlSZWFkRGlyZWN0b3J5KGVudHJpZXMsIGhhc01vcmUpOwpk aWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vYmxhY2tiZXJyeS9Xb3JrZXJQbGF0 Zm9ybUFzeW5jRmlsZVN5c3RlbUNhbGxiYWNrcy5oIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0v YmxhY2tiZXJyeS9Xb3JrZXJQbGF0Zm9ybUFzeW5jRmlsZVN5c3RlbUNhbGxiYWNrcy5oCmluZGV4 IGYyYzA5NDYuLmVjNzA4ZTUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2Js YWNrYmVycnkvV29ya2VyUGxhdGZvcm1Bc3luY0ZpbGVTeXN0ZW1DYWxsYmFja3MuaAorKysgYi9T b3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ibGFja2JlcnJ5L1dvcmtlclBsYXRmb3JtQXN5bmNGaWxl U3lzdGVtQ2FsbGJhY2tzLmgKQEAgLTg0LDYgKzg0LDcgQEAgcHVibGljOgogICAgIHZpcnR1YWwg dm9pZCBub3RpZnlTdWNjZWVkKCk7CiAgICAgdmlydHVhbCB2b2lkIG5vdGlmeUZhaWwoQmxhY2tC ZXJyeTo6UGxhdGZvcm06OldlYkZpbGVFcnJvcik7CiAgICAgdmlydHVhbCB2b2lkIG5vdGlmeVJl YWRNZXRhZGF0YShjb25zdCBCbGFja0JlcnJ5OjpQbGF0Zm9ybTo6V2ViRmlsZUluZm8mKTsKKyAg ICB2aXJ0dWFsIHZvaWQgbm90aWZ5Q3JlYXRlU25hcHNob3RGaWxlQW5kUmVhZE1ldGFkYXRhKGNv bnN0IEJsYWNrQmVycnk6OlBsYXRmb3JtOjpXZWJGaWxlSW5mbyYpOwogICAgIHZpcnR1YWwgdm9p ZCBub3RpZnlSZWFkRGlyZWN0b3J5KGNvbnN0IHN0ZDo6dmVjdG9yPEJsYWNrQmVycnk6OlBsYXRm b3JtOjpXZWJGaWxlU3lzdGVtRW50cnk+JiBlbnRyaWVzLCBib29sIGhhc01vcmUpOwogICAgIHZp cnR1YWwgdm9pZCBub3RpZnlDcmVhdGVGaWxlV3JpdGVyKEJsYWNrQmVycnk6OlBsYXRmb3JtOjpX ZWJGaWxlV3JpdGVyKiBwbGF0Zm9ybVdyaXRlciwgbG9uZyBsb25nIGxlbmd0aCk7CiAgICAgdmly dHVhbCBQYXNzT3duUHRyPEFzeW5jRmlsZVN5c3RlbT4gY3JlYXRlQXN5bmNGaWxlU3lzdGVtKFBh c3NPd25QdHI8QmxhY2tCZXJyeTo6UGxhdGZvcm06OldlYkZpbGVTeXN0ZW0+IHBsYXRmb3JtRmls ZVN5c3RlbSk7CkBAIC05Nyw2ICs5OCw3IEBAIHByaXZhdGU6CiAgICAgc3RhdGljIHZvaWQgbm90 aWZ5U3VjY2VlZE9uV29ya2VyVGhyZWFkKFNjcmlwdEV4ZWN1dGlvbkNvbnRleHQqLCBXb3JrZXJQ bGF0Zm9ybUFzeW5jRmlsZVN5c3RlbUNhbGxiYWNrcyopOwogICAgIHN0YXRpYyB2b2lkIG5vdGlm eUZhaWxPbldvcmtlclRocmVhZChTY3JpcHRFeGVjdXRpb25Db250ZXh0KiwgV29ya2VyUGxhdGZv cm1Bc3luY0ZpbGVTeXN0ZW1DYWxsYmFja3MqLCBCbGFja0JlcnJ5OjpQbGF0Zm9ybTo6V2ViRmls ZUVycm9yKTsKICAgICBzdGF0aWMgdm9pZCBub3RpZnlSZWFkTWV0YWRhdGFPbldvcmtlclRocmVh ZChTY3JpcHRFeGVjdXRpb25Db250ZXh0KiwgV29ya2VyUGxhdGZvcm1Bc3luY0ZpbGVTeXN0ZW1D YWxsYmFja3MqLCBjb25zdCBCbGFja0JlcnJ5OjpQbGF0Zm9ybTo6V2ViRmlsZUluZm8mKTsKKyAg ICBzdGF0aWMgdm9pZCBub3RpZnlDcmVhdGVTbmFwc2hvdEZpbGVBbmRSZWFkTWV0YWRhdGFPbldv cmtlclRocmVhZChTY3JpcHRFeGVjdXRpb25Db250ZXh0KiwgV29ya2VyUGxhdGZvcm1Bc3luY0Zp bGVTeXN0ZW1DYWxsYmFja3MqLCBjb25zdCBCbGFja0JlcnJ5OjpQbGF0Zm9ybTo6V2ViRmlsZUlu Zm8mKTsKICAgICBzdGF0aWMgdm9pZCBub3RpZnlSZWFkRGlyZWN0b3J5RW50cnlPbldvcmtlclRo cmVhZChTY3JpcHRFeGVjdXRpb25Db250ZXh0KiwgV29ya2VyUGxhdGZvcm1Bc3luY0ZpbGVTeXN0 ZW1DYWxsYmFja3MqLCBjb25zdCBzdGQ6OnZlY3RvcjxCbGFja0JlcnJ5OjpQbGF0Zm9ybTo6V2Vi RmlsZVN5c3RlbUVudHJ5PiYgZW50cmllcywgYm9vbCBoYXNNb3JlKTsKICAgICBzdGF0aWMgdm9p ZCBub3RpZnlDcmVhdGVGaWxlV3JpdGVyT25Xb3JrZXJUaHJlYWQoU2NyaXB0RXhlY3V0aW9uQ29u dGV4dCosIFdvcmtlclBsYXRmb3JtQXN5bmNGaWxlU3lzdGVtQ2FsbGJhY2tzKiwgQmxhY2tCZXJy eTo6UGxhdGZvcm06OldlYkZpbGVXcml0ZXIqIHBsYXRmb3JtV3JpdGVyLCBsb25nIGxvbmcgbGVu Z3RoKTsKIAo=