157682 2016-05-13 14:10:54 -0700 Runaway malloc memory usage in this simple JSC program 2016-05-13 15:21:50 -0700 1 1 1 Unclassified WebKit New Bugs WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 ggaren ggaren commit-queue fpizlo keith_miller kling mark.lam msaboff saam oldest_to_newest 1192875 0 ggaren 2016-05-13 14:10:54 -0700 Runaway malloc memory usage in this simple JSC program 1192878 1 278865 ggaren 2016-05-13 14:17:33 -0700 Created attachment 278865 Patch 1192879 2 278866 ggaren 2016-05-13 14:20:43 -0700 Created attachment 278866 Patch 1192880 3 278866 mark.lam 2016-05-13 14:21:38 -0700 Comment on attachment 278866 Patch r=me 1192883 4 ggaren 2016-05-13 14:23:42 -0700 Committed r200884: <http://trac.webkit.org/changeset/200884> 1192903 5 ggaren 2016-05-13 15:21:25 -0700 <rdar://problem/25942874> 1192904 6 ggaren 2016-05-13 15:21:50 -0700 BTW, here is the program: #include <JavaScriptCore/JavaScriptCore.h> #include <stdio.h> #include <sys/time.h> static double currentTime() { struct timeval tv; gettimeofday(&tv, NULL); return tv.tv_sec + tv.tv_usec / 1000. / 1000.; } int main(int argc, char** argv) { JSContextGroupRef group; double before, after; unsigned i; group = JSContextGroupCreate(); before = currentTime(); for (i = 0; i < 100000; ++i) JSGlobalContextRelease(JSGlobalContextCreateInGroup(group, NULL)); after = currentTime(); printf("That took %lf ms.\n", (after - before) * 1000.); return 0; } 278865 2016-05-13 14:17:33 -0700 2016-05-13 14:20:39 -0700 Patch bug-157682-20160513141826.patch text/plain 1676 ggaren SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291 cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMjAwODgxKQorKysgU291cmNl L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIxIEBA CisyMDE2LTA1LTEzICBHZW9mZnJleSBHYXJlbiAgPGdnYXJlbkBhcHBsZS5jb20+CisKKyAgICAg ICAgUnVuYXdheSBtYWxsb2MgbWVtb3J5IHVzYWdlIGluIHRoaXMgc2ltcGxlIEpTQyBwcm9ncmFt CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNTc2ODIK KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGhlYXAv V2Vha1NldC5jcHA6CisgICAgICAgIChKU0M6OldlYWtTZXQ6OnN3ZWVwKTogV2hlbmV2ZXIgd2Ug bWlnaHQgYWRkIGEgYmxvY2sgdG8KKyAgICAgICAgbV9sb2dpY2FsbHlFbXB0eVdlYWtCbG9ja3Ms IGJ1dCBzdXJlIGFsc28gdG8gc3dlZXAgYSBibG9jayBpbgorICAgICAgICBtX2xvZ2ljYWxseUVt cHR5V2Vha0Jsb2Nrcy4gT3RoZXJ3aXNlLCBhZGRpdGlvbnMgbWlnaHQgb3V0cGFjZSByZW1vdmFs cworICAgICAgICBldmVuIHdoZW4gYWxsIG1lbW9yeSBpcyBmcmVlZC4KKworICAgICAgICBXZSBk byB0aGlzIHdoZW5ldmVyIHdlIG1pZ2h0IGFkZCBhIGJsb2NrIHJhdGhlciB0aGFuIHdoZW5ldmVy IHdlIGRvIGFkZAorICAgICAgICBhIGJsb2NrIGJlY2F1c2Ugd2UnZCBsaWtlIHRvIHN3ZWVwIHRo ZSBlbnRyaWVzIGluCisgICAgICAgIG1fbG9naWNhbGx5RW1wdHlXZWFrQmxvY2tzIHByb21wdGx5 IGV2ZW4gaWYgaXQncyBub3QgZ3Jvd2luZywgYW5kIHRoaXMKKyAgICAgICAgaXMgYSByZWFzb25h Ymx5IHJhdGUtbGltaXRlZCBvcHBvcnR1bml0eSB0byBkbyBzby4KKwogMjAxNi0wNS0xMyAgTWFy ayBMYW0gIDxtYXJrLmxhbUBhcHBsZS5jb20+CiAKICAgICAgICAgV2Ugc2hvdWxkIGhhdmUgb25l IGNhbGxlZVNhdmVSZWdpc3RlcnNCdWZmZXIgcGVyIFZNRW50cnlGcmFtZSwgbm90IG9uZSBwZXIg Vk0uCkluZGV4OiBTb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVhcC9XZWFrU2V0LmNwcAo9PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09Ci0tLSBTb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVhcC9XZWFrU2V0LmNwcAkocmV2aXNpb24g MjAwNzgwKQorKysgU291cmNlL0phdmFTY3JpcHRDb3JlL2hlYXAvV2Vha1NldC5jcHAJKHdvcmtp bmcgY29weSkKQEAgLTQ2LDYgKzQ2LDggQEAgV2Vha1NldDo6fldlYWtTZXQoKQogdm9pZCBXZWFr U2V0Ojpzd2VlcCgpCiB7CiAgICAgZm9yIChXZWFrQmxvY2sqIGJsb2NrID0gbV9ibG9ja3MuaGVh ZCgpOyBibG9jazspIHsKKyAgICAgICAgaGVhcCgpLT5zd2VlcE5leHRMb2dpY2FsbHlFbXB0eVdl YWtCbG9jaygpOworCiAgICAgICAgIFdlYWtCbG9jayogbmV4dEJsb2NrID0gYmxvY2stPm5leHQo KTsKICAgICAgICAgYmxvY2stPnN3ZWVwKCk7CiAgICAgICAgIGlmIChibG9jay0+aXNMb2dpY2Fs bHlFbXB0eUJ1dE5vdEZyZWUoKSkgewo= 278866 2016-05-13 14:20:43 -0700 2016-05-13 14:21:38 -0700 Patch bug-157682-20160513142136.patch text/plain 1682 ggaren SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291 cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMjAwODgxKQorKysgU291cmNl L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIxIEBA CisyMDE2LTA1LTEzICBHZW9mZnJleSBHYXJlbiAgPGdnYXJlbkBhcHBsZS5jb20+CisKKyAgICAg ICAgUnVuYXdheSBtYWxsb2MgbWVtb3J5IHVzYWdlIGluIHRoaXMgc2ltcGxlIEpTQyBwcm9ncmFt CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNTc2ODIK KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGhlYXAv V2Vha1NldC5jcHA6CisgICAgICAgIChKU0M6OldlYWtTZXQ6OnN3ZWVwKTogV2hlbmV2ZXIgd2Ug bWlnaHQgYWRkIGEgYmxvY2sgdG8KKyAgICAgICAgbV9sb2dpY2FsbHlFbXB0eVdlYWtCbG9ja3Ms IGJlIHN1cmUgYWxzbyB0byBzd2VlcCBhIGJsb2NrIGluCisgICAgICAgIG1fbG9naWNhbGx5RW1w dHlXZWFrQmxvY2tzLiBPdGhlcndpc2UsIGFkZGl0aW9ucyBtaWdodCBvdXRwYWNlIHJlbW92YWxz CisgICAgICAgIGV2ZW4gd2hlbiBhbGwgbWVtb3J5IGlzIGZyZWVkLgorCisgICAgICAgIFdlIGRv IHRoaXMgd2hlbmV2ZXIgd2UgKm1pZ2h0KiBhZGQgYSBibG9jayBhbmQgbm90IGp1c3Qgd2hlbmV2 ZXIgd2UgKmRvKgorICAgICAgICBhZGQgYSBibG9jayBiZWNhdXNlIHdlJ2QgbGlrZSB0byBzd2Vl cCB0aGUgZW50cmllcyBpbgorICAgICAgICBtX2xvZ2ljYWxseUVtcHR5V2Vha0Jsb2NrcyBwcm9t cHRseSBldmVuIHdoZW4gaXQncyBub3QgZ3Jvd2luZywgYW5kIHRoaXMKKyAgICAgICAgaXMgYSBy ZWFzb25hYmx5IHJhdGUtbGltaXRlZCBvcHBvcnR1bml0eSB0byBkbyBzby4KKwogMjAxNi0wNS0x MyAgTWFyayBMYW0gIDxtYXJrLmxhbUBhcHBsZS5jb20+CiAKICAgICAgICAgV2Ugc2hvdWxkIGhh dmUgb25lIGNhbGxlZVNhdmVSZWdpc3RlcnNCdWZmZXIgcGVyIFZNRW50cnlGcmFtZSwgbm90IG9u ZSBwZXIgVk0uCkluZGV4OiBTb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVhcC9XZWFrU2V0LmNwcAo9 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09Ci0tLSBTb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVhcC9XZWFrU2V0LmNwcAkocmV2 aXNpb24gMjAwNzgwKQorKysgU291cmNlL0phdmFTY3JpcHRDb3JlL2hlYXAvV2Vha1NldC5jcHAJ KHdvcmtpbmcgY29weSkKQEAgLTQ2LDYgKzQ2LDggQEAgV2Vha1NldDo6fldlYWtTZXQoKQogdm9p ZCBXZWFrU2V0Ojpzd2VlcCgpCiB7CiAgICAgZm9yIChXZWFrQmxvY2sqIGJsb2NrID0gbV9ibG9j a3MuaGVhZCgpOyBibG9jazspIHsKKyAgICAgICAgaGVhcCgpLT5zd2VlcE5leHRMb2dpY2FsbHlF bXB0eVdlYWtCbG9jaygpOworCiAgICAgICAgIFdlYWtCbG9jayogbmV4dEJsb2NrID0gYmxvY2st Pm5leHQoKTsKICAgICAgICAgYmxvY2stPnN3ZWVwKCk7CiAgICAgICAgIGlmIChibG9jay0+aXNM b2dpY2FsbHlFbXB0eUJ1dE5vdEZyZWUoKSkgewo=