107970 2013-01-25 11:42:14 -0800 New NetworkProcess can ASSERT after old NetworkProcess crashes 2013-01-25 11:47:02 -0800 1 1 1 Unclassified WebKit WebKit2 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 beidson beidson oldest_to_newest 816797 0 beidson 2013-01-25 11:42:14 -0800 New NetworkProcess can ASSERT after old NetworkProcess crashes Steps: 1 - Go to a site with HTTP auth, so Safari drops down the auth sheet 2 - `killall -9 NetworkProcess` in a terminal 3 - The WebProcess responds to the news that the NetworkProcess crashed by failing all of its outstanding ResourceLoaders 4 - In that process, it tells the new NetworkProcess to remove a load identifier for a failed loader 5 - The new NetworkProcess has no record of that new load identifier so it ASSERTs in: void NetworkResourceLoadScheduler::removeLoader(SchedulableLoader* loader) { ASSERT(loader); ... } 816798 1 184783 beidson 2013-01-25 11:43:53 -0800 Created attachment 184783 Patch v1 816805 2 beidson 2013-01-25 11:47:02 -0800 http://trac.webkit.org/changeset/140852 184783 2013-01-25 11:43:53 -0800 2013-01-25 11:44:20 -0800 Patch v1 patch text/plain 1490 beidson ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No YW5nZUxvZwppbmRleCAzNjgwNDMzLi40NDY3Njc4IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQg QEAKKzIwMTMtMDEtMjUgIEJyYWR5IEVpZHNvbiAgPGJlaWRzb25AYXBwbGUuY29tPgorCisgICAg ICAgIE5ldyBOZXR3b3JrUHJvY2VzcyBjYW4gQVNTRVJUIGFmdGVyIG9sZCBOZXR3b3JrUHJvY2Vz cyBjcmFzaGVzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p ZD0xMDc5NzAKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg ICAqIE5ldHdvcmtQcm9jZXNzL05ldHdvcmtDb25uZWN0aW9uVG9XZWJQcm9jZXNzLmNwcDoKKyAg ICAgICAgKFdlYktpdDo6TmV0d29ya0Nvbm5lY3Rpb25Ub1dlYlByb2Nlc3M6OnJlbW92ZUxvYWRJ ZGVudGlmaWVyKTogRG9uJ3QgdGVsbCB0aGUgc2NoZWR1bGVyCisgICAgICAgICAgdG8gcmVtb3Zl IGEgbG9hZGVyIHRoYXQgZG9lc24ndCBleGlzdCwgYXMgd2UgbWlnaHQgYmUgcmVjb3ZlcmluZyBm cm9tIGEgY3Jhc2guCisKIDIwMTMtMDEtMjUgIFphbiBEb2JlcnNlayAgPHpkb2JlcnNla0BpZ2Fs aWEuY29tPgogCiAgICAgICAgIEZ1cnRoZXIgR1RLIGJ1aWxkIGZpeGluZy4KZGlmZiAtLWdpdCBh L1NvdXJjZS9XZWJLaXQyL05ldHdvcmtQcm9jZXNzL05ldHdvcmtDb25uZWN0aW9uVG9XZWJQcm9j ZXNzLmNwcCBiL1NvdXJjZS9XZWJLaXQyL05ldHdvcmtQcm9jZXNzL05ldHdvcmtDb25uZWN0aW9u VG9XZWJQcm9jZXNzLmNwcAppbmRleCBiNTljNjdjLi5lNTk0ZDFlIDEwMDY0NAotLS0gYS9Tb3Vy Y2UvV2ViS2l0Mi9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrQ29ubmVjdGlvblRvV2ViUHJvY2Vzcy5j cHAKKysrIGIvU291cmNlL1dlYktpdDIvTmV0d29ya1Byb2Nlc3MvTmV0d29ya0Nvbm5lY3Rpb25U b1dlYlByb2Nlc3MuY3BwCkBAIC0xMjQsNiArMTI0LDEwIEBAIHZvaWQgTmV0d29ya0Nvbm5lY3Rp b25Ub1dlYlByb2Nlc3M6OnJlbW92ZUxvYWRJZGVudGlmaWVyKFJlc291cmNlTG9hZElkZW50aWZp ZXIKICAgICBpZiAoIWxvYWRlcikKICAgICAgICAgbG9hZGVyID0gbV9zeW5jTmV0d29ya1Jlc291 cmNlTG9hZGVycy50YWtlKGlkZW50aWZpZXIpOwogCisgICAgLy8gSXQncyBwb3NzaWJsZSB3ZSBo YXZlIG5vIGxvYWRlciBmb3IgdGhpcyBpZGVudGlmaWVyIGlmIHRoZSBOZXR3b3JrUHJvY2VzcyBj cmFzaGVkIGFuZCB0aGlzIHdhcyBhIHJlc3Bhd25lZCBOZXR3b3JrUHJvY2Vzcy4KKyAgICBpZiAo IWxvYWRlcikKKyAgICAgICAgcmV0dXJuOworCiAgICAgTmV0d29ya1Byb2Nlc3M6OnNoYXJlZCgp Lm5ldHdvcmtSZXNvdXJjZUxvYWRTY2hlZHVsZXIoKS5yZW1vdmVMb2FkZXIobG9hZGVyLmdldCgp KTsKIH0KIAo=