90168 2012-06-28 06:24:20 -0700 [GTK] [WK2] Memory leak in ResourceHandleSoup.cpp 2012-06-28 09:43:43 -0700 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 naginenis naginenis danw gustavo mrobinson rakuco webkit.review.bot oldest_to_newest 659457 0 naginenis 2012-06-28 06:24:20 -0700 ==11611== 160 bytes in 10 blocks are definitely lost in loss record 18,502 of 25,443 ==11611== at 0x4C29DB4: calloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) ==11611== by 0x10603AA0: g_malloc0 (gmem.c:189) ==11611== by 0x105FE2A9: g_main_loop_new (gmain.c:3212) ==11611== by 0x6CF4942: WebCore::WebCoreSynchronousLoader::WebCoreSynchronousLoader(WebCore::ResourceError&, WebCore::ResourceResponse&, _SoupSession*, WTF::Vector<char, 0ul>&) (ResourceHandleSoup.cpp:92) ==11611== by 0x6CF3F03: WebCore::ResourceHandle::loadResourceSynchronously(WebCore::NetworkingContext*, WebCore::ResourceRequest const&, WebCore::StoredCredentials, WebCore::ResourceError&, WebCore::ResourceResponse&, WTF::Vector<char, 0ul>&) (ResourceHandleSoup.cpp:808) ==11611== by 0x6B0B70A: WebCore::FrameLoader::loadResourceSynchronously(WebCore::ResourceRequest const&, WebCore::StoredCredentials, WebCore::ResourceError&, WebCore::ResourceResponse&, WTF::Vector<char, 0ul>&) (FrameLoader.cpp:2521) ==11611== by 0x6AF9F6E: WebCore::DocumentThreadableLoader::loadRequest(WebCore::ResourceRequest const&, WebCore::SecurityCheckPolicy) (DocumentThreadableLoader.cpp:398) ==11611== by 0x6AF7F25: WebCore::DocumentThreadableLoader::DocumentThreadableLoader(WebCore::Document*, WebCore::ThreadableLoaderClient*, WebCore::DocumentThreadableLoader::BlockingBehavior, WebCore::ResourceRequest const&, WebCore::ThreadableLoaderOptions const&) (DocumentThreadableLoader.cpp:89) ==11611== by 0x6AF7B29: WebCore::DocumentThreadableLoader::loadResourceSynchronously(WebCore::Document*, WebCore::ResourceRequest const&, WebCore::ThreadableLoaderClient&, WebCore::ThreadableLoaderOptions const&) (DocumentThreadableLoader.cpp:60) ==11611== by 0x6B564AC: WebCore::ThreadableLoader::loadResourceSynchronously(WebCore::ScriptExecutionContext*, WebCore::ResourceRequest const&, WebCore::ThreadableLoaderClient&, WebCore::ThreadableLoaderOptions const&) (ThreadableLoader.cpp:69) ==11611== by 0x712B865: WebCore::XMLHttpRequest::createRequest(int&) (XMLHttpRequest.cpp:732) ==11611== by 0x712AD0C: WebCore::XMLHttpRequest::send(WTF::String const&, int&) (XMLHttpRequest.cpp:601) ==11611== by 0x712A732: WebCore::XMLHttpRequest::send(int&) (XMLHttpRequest.cpp:541) ==11611== by 0x65068C2: WebCore::JSXMLHttpRequest::send(JSC::ExecState*) (JSXMLHttpRequestCustom.cpp:120) ==11611== by 0x7575AC2: WebCore::jsXMLHttpRequestPrototypeFunctionSend(JSC::ExecState*) (JSXMLHttpRequest.cpp:568) 659464 1 149940 naginenis 2012-06-28 06:45:43 -0700 Created attachment 149940 Patch Fixed a memory leak. 659566 2 149940 webkit.review.bot 2012-06-28 09:43:39 -0700 Comment on attachment 149940 Patch Clearing flags on attachment: 149940 Committed r121437: <http://trac.webkit.org/changeset/121437> 659567 3 webkit.review.bot 2012-06-28 09:43:43 -0700 All reviewed patches have been landed. Closing bug. 149940 2012-06-28 06:45:43 -0700 2012-06-28 09:43:39 -0700 Patch 90168.patch text/plain 1594 naginenis ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCAzZmEyMTQ0Li42ZDY2NTdhIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTgg QEAKKzIwMTItMDYtMjggIFN1ZGFyc2FuYSBOYWdpbmVuaSAgPHN1ZGFyc2FuYS5uYWdpbmVuaUBs aW51eC5pbnRlbC5jb20+CisKKyAgICAgICAgW0dUS10gW1dLMl0gTWVtb3J5IGxlYWsgaW4gUmVz b3VyY2VIYW5kbGVTb3VwLmNwcAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93 X2J1Zy5jZ2k/aWQ9OTAxNjgKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4K KworICAgICAgICBGaXhlZCBhIG1lbW9yeSBsZWFrIGluIFdlYkNvcmVTeW5jaHJvbm91c0xvYWRl ciBieSB1c2luZyBhZG9wdEdSZWYKKyAgICAgICAgaW5zdGVhZCBvZiBqdXN0IGdldHRpbmcgbmV3 IHJlZmVyZW5jZSBvZiBHTWFpbkxvb3AuCisKKyAgICAgICAgTm8gbmV3IHRlc3RzLiBObyBjaGFu Z2UgaW4gYmVoYXZpb3IuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9uZXR3b3JrL3NvdXAvUmVzb3Vy Y2VIYW5kbGVTb3VwLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OldlYkNvcmVTeW5jaHJvbm91c0xv YWRlcjo6V2ViQ29yZVN5bmNocm9ub3VzTG9hZGVyKToKKwogMjAxMi0wNi0yOCAgQmFsYXpzIEtl bGVtZW4gIDxrYmFsYXpzQHdlYmtpdC5vcmc+CiAKICAgICAgICAgW1F0XSBwbHVnaW4gaXMgbG9h ZGVkIHRvIHRoZSB3ZWIgcHJvY2VzcyB2aWEgTWFpblJlc291cmNlTG9hZGVyOjpzdWJzdGl0dXRl TUlNRVR5cGVGcm9tUGx1Z2luRGF0YWJhc2UKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3Bs YXRmb3JtL25ldHdvcmsvc291cC9SZXNvdXJjZUhhbmRsZVNvdXAuY3BwIGIvU291cmNlL1dlYkNv cmUvcGxhdGZvcm0vbmV0d29yay9zb3VwL1Jlc291cmNlSGFuZGxlU291cC5jcHAKaW5kZXggMTBj NmZhOS4uMWEwOGU1YiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vbmV0d29y ay9zb3VwL1Jlc291cmNlSGFuZGxlU291cC5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvcGxhdGZv cm0vbmV0d29yay9zb3VwL1Jlc291cmNlSGFuZGxlU291cC5jcHAKQEAgLTg4LDcgKzg4LDcgQEAg cHVibGljOgogICAgICAgICBsb2FkaW5nU3luY2hyb25vdXNSZXF1ZXN0ID0gdHJ1ZTsKICAgICAg ICAgR1JlZlB0cjxHTWFpbkNvbnRleHQ+IGlubmVyTWFpbkNvbnRleHQgPSBhZG9wdEdSZWYoZ19t YWluX2NvbnRleHRfbmV3KCkpOwogICAgICAgICBnX21haW5fY29udGV4dF9wdXNoX3RocmVhZF9k ZWZhdWx0KGlubmVyTWFpbkNvbnRleHQuZ2V0KCkpOwotICAgICAgICBtX21haW5Mb29wID0gZ19t YWluX2xvb3BfbmV3KGlubmVyTWFpbkNvbnRleHQuZ2V0KCksIGZhbHNlKTsKKyAgICAgICAgbV9t YWluTG9vcCA9IGFkb3B0R1JlZihnX21haW5fbG9vcF9uZXcoaW5uZXJNYWluQ29udGV4dC5nZXQo KSwgZmFsc2UpKTsKIAogICAgICAgICBhZGp1c3RNYXhDb25uZWN0aW9ucygxKTsKICAgICB9Cg==