116686 2013-05-23 12:10:14 -0700 [Soup] [Gstreamer] ASSERT in StreamingClient::getOrCreateReadBuffer 2013-05-31 03:25:02 -0700 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED http://www.publico.es/455878/espana-suma-11-banderas-azules-mas-a-sus-costas-y-se-mantiene-lider P2 Normal --- 1 svillar webkit-unassigned berto cdumez cgarcia commit-queue danw gtk-ews gustavo menard mrobinson pnormand rakuco rego+ews rego svillar xan.lopez oldest_to_newest 892983 0 svillar 2013-05-23 12:10:14 -0700 This is the backtrace of the crash visiting the above url. Program received signal SIGSEGV, Segmentation fault. 0x00007f26cc3787dc in WTFCrash () at ../../Source/WTF/wtf/Assertions.cpp:339 339 *(int *)(uintptr_t)0xbbadbeef = 0; (gdb) bt #0 0x00007f26cc3787dc in WTFCrash () at ../../Source/WTF/wtf/Assertions.cpp:339 #1 0x00007f26c8409364 in StreamingClient::getOrCreateReadBuffer (this=0x1eedf20, requestedSize=8192, actualSize=@0x7fff11795ea8: 139803140448136) at ../../Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:931 #2 0x00007f26c847e3d4 in WebCore::ResourceHandle::ensureReadBuffer (this=0x7f2674e0f7b0) at ../../Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp:311 #3 0x00007f26c848062d in WebCore::sendRequestCallback (result=0x7f26743dfdc0, data=0x7f2674e0f7b0) at ../../Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp:730 #4 0x00007f26c5ef1f6b in g_task_return_now (task=0x7f26743dfdc0) at gtask.c:1105 #5 0x00007f26c5ef271e in g_task_return (task=0x7f26743dfdc0, type=<optimized out>) at gtask.c:1158 #6 g_task_return (task=0x7f26743dfdc0, type=<optimized out>) at gtask.c:1126 #7 0x00007f26c603766c in http_input_stream_ready_cb (source=0x76b040, result=0x7f267443ec50, user_data=0x7f26743dfdc0) at soup-request-http.c:124 #8 0x00007f26c5ef1f6b in g_task_return_now (task=0x7f267443ec50) at gtask.c:1105 #9 0x00007f26c5ef271e in g_task_return (task=0x7f267443ec50, type=<optimized out>) at gtask.c:1158 #10 g_task_return (task=0x7f267443ec50, type=<optimized out>) at gtask.c:1126 #11 0x00007f26c603a332 in async_send_request_return_result (item=0x7f26743c3d20, stream=0xedbbc0, error=<optimized out>) at soup-session.c:3628 #12 0x00007f26c603f80f in send_async_maybe_complete (stream=0xedbbc0, item=0x7f26743c3d20) at soup-session.c:3742 #13 try_run_until_read (item=item@entry=0x7f26743c3d20) at soup-session.c:3766 #14 0x00007f26c603f95d in read_ready_cb (msg=<optimized out>, user_data=0x7f26743c3d20) at soup-session.c:3753 #15 0x00007f26c5d41e25 in g_main_dispatch (context=0x761e90) at gmain.c:3054 #16 g_main_context_dispatch (context=context@entry=0x761e90) at gmain.c:3630 #17 0x00007f26c5d42168 in g_main_context_iterate (context=0x761e90, block=block@entry=1, dispatch=dispatch@entry=1, self=<error reading variable: Unhandled dwarf expression opcode 0xfa>) at gmain.c:3701 #18 0x00007f26c5d425da in g_main_loop_run (loop=0x75e190) at gmain.c:3895 #19 0x00007f26c898b50a in WebCore::RunLoop::run () at ../../Source/WebCore/platform/gtk/RunLoopGtk.cpp:61 #20 0x00007f26c72533fc in WebKit::WebProcessMainGtk (argc=2, argv=0x7fff117963d8) at ../../Source/WebKit2/WebProcess/gtk/WebProcessMainGtk.cpp:78 #21 0x0000000000400a1c in main (argc=2, argv=0x7fff117963d8) at ../../Source/WebKit2/gtk/MainGtk.cpp:31 895475 1 berto 2013-05-30 04:11:38 -0700 Reproduced, I'll take a look. 895587 2 berto 2013-05-30 10:46:29 -0700 I actually don't understand why that assert is there, either it is wrong or the name of the method is misleading, because it's creating a new buffer every time (and we're most likely leaking memory). This is called from ResourceHandle::ensureReadBuffer(), which is used by several callbacks in ResourceHandle. This comes from bug 115364 and bug 105552. 895590 3 cgarcia 2013-05-30 10:58:10 -0700 (In reply to comment #2) > I actually don't understand why that assert is there, either it is > wrong or the name of the method is misleading, because it's creating a > new buffer every time (and we're most likely leaking memory). Memory is not leaking because buffer is a GRefPtr. > This is called from ResourceHandle::ensureReadBuffer(), which is used > by several callbacks in ResourceHandle. > > This comes from bug 115364 and bug 105552. Patch in bug 115364 doesn't change the logic of the gst buffer, it simply renames the method and moves the default buffer implementation from the cross platform code to the soup resource handle implementation 895600 4 203367 berto 2013-05-30 11:21:00 -0700 Created attachment 203367 Patch (In reply to comment #3) > Memory is not leaking because buffer is a GRefPtr. Yes, you're right. This patch removes the assertion and returns the buffer if it exists. I don't see any other crash or assertion being triggered and everytime a new buffer is created it seems to be cleared correctly afterwards. Then we have the thing that the requestedSize parameter is completely ignored if the buffer already exists, although in practice this is not happening since we're using a constant value that is defined in ResourceHandle::ensureReadBuffer() 895602 5 berto 2013-05-30 11:23:12 -0700 I think the original code is from Gustavo, adding him to Cc. 895604 6 203367 cgarcia 2013-05-30 11:30:28 -0700 Comment on attachment 203367 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=203367&action=review I wonder if we could reuse the buffer also for the case of not using the soup resource handle. > Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:936 > + mapGstBuffer(buffer); I don't know what map/unmap means in GST but we are unmapping the buffer for every data chunk in didReceiveData. 895606 7 cgarcia 2013-05-30 11:44:00 -0700 I had missed the leakPtr in didReceiveData, so we are in the end creating and mapping the buffer before every data chunk, and releasing it after. 895612 8 203367 gtk-ews 2013-05-30 11:57:32 -0700 Comment on attachment 203367 Patch Attachment 203367 did not pass gtk-ews (gtk): Output: http://webkit-queues.appspot.com/results/689307 895628 9 rego 2013-05-30 12:19:04 -0700 I've just tested manually the patch in gtk-wk2 and it worked now, so it should be some temporal issue. 895632 10 berto 2013-05-30 12:24:34 -0700 sendRequestCallback() is called twice, the first one is a redirection (there's an ensureReadBuffer() there). The second one is the call at the end of the function. 895636 11 cgarcia 2013-05-30 12:28:48 -0700 It seems that the assert was actually correct and tried to detect cases where the sequence getOrCreateBuffer -> didReceivedata doesn't happen. This currently happens in case of redirection because we changed the skip_async by read_async, but we don't really read anything and then didReceiveData is not called. So the fix should be either use skip_async now that we depend on recent glib, or use a different buffer in this particular case, a skipBuffer or something, so that we don't allocate a gst buffer for nothing. 895837 12 203427 berto 2013-05-31 02:40:55 -0700 Created attachment 203427 Patch Using skip_async() to handle the redirection fixes the problem. 895839 13 203427 cgarcia 2013-05-31 03:04:46 -0700 Comment on attachment 203427 Patch Thanks! 895847 14 203427 commit-queue 2013-05-31 03:24:58 -0700 Comment on attachment 203427 Patch Clearing flags on attachment: 203427 Committed r151013: <http://trac.webkit.org/changeset/151013> 895848 15 commit-queue 2013-05-31 03:25:02 -0700 All reviewed patches have been landed. Closing bug. 203367 2013-05-30 11:21:00 -0700 2013-05-31 02:40:55 -0700 Patch 116686.diff text/plain 1815 berto ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCBmZTg2Yjc5Li42ZDMyMmVhIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQg QEAKKzIwMTMtMDUtMzAgIEFsYmVydG8gR2FyY2lhICA8YWdhcmNpYUBpZ2FsaWEuY29tPgorCisg ICAgICAgIFtTb3VwXSBbR3N0cmVhbWVyXSBBU1NFUlQgaW4gU3RyZWFtaW5nQ2xpZW50OjpnZXRP ckNyZWF0ZVJlYWRCdWZmZXIKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i dWcuY2dpP2lkPTExNjY4NgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor CisgICAgICAgICogcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL1dlYktpdFdlYlNvdXJjZUdT dHJlYW1lci5jcHA6CisgICAgICAgIChTdHJlYW1pbmdDbGllbnQ6OmdldE9yQ3JlYXRlUmVhZEJ1 ZmZlcik6CisgICAgICAgIFJlbW92ZSBhc3NlcnRpb24gYW5kIHJldHVybiB0aGUgYnVmZmVyIGlm IGl0IGFscmVhZHkgZXhpc3RzLgorCiAyMDEzLTA1LTMwICBBbmRyZWFzIEtsaW5nICA8YWtsaW5n QGFwcGxlLmNvbT4KIAogICAgICAgICBBdm9pZCB1c2luZyBDU1NPTSBzdHlsZSBkZWNsYXJhdGlv bnMgaW4gSFRNTCBlZGl0aW5nLgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0v Z3JhcGhpY3MvZ3N0cmVhbWVyL1dlYktpdFdlYlNvdXJjZUdTdHJlYW1lci5jcHAgYi9Tb3VyY2Uv V2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvV2ViS2l0V2ViU291cmNlR1N0cmVh bWVyLmNwcAppbmRleCA3NjNiMWUwLi4zNjBjMWEyIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvV2ViS2l0V2ViU291cmNlR1N0cmVhbWVyLmNw cAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvV2ViS2l0 V2ViU291cmNlR1N0cmVhbWVyLmNwcApAQCAtOTI3LDE2ICs5MjcsMTggQEAgdm9pZCBTdHJlYW1p bmdDbGllbnQ6OmRpZFJlY2VpdmVEYXRhKFJlc291cmNlSGFuZGxlKiBoYW5kbGUsIGNvbnN0IGNo YXIqIGRhdGEsIGkKIGNoYXIqIFN0cmVhbWluZ0NsaWVudDo6Z2V0T3JDcmVhdGVSZWFkQnVmZmVy KHNpemVfdCByZXF1ZXN0ZWRTaXplLCBzaXplX3QmIGFjdHVhbFNpemUpCiB7CiAgICAgV2ViS2l0 V2ViU3JjUHJpdmF0ZSogcHJpdiA9IG1fc3JjLT5wcml2OworICAgIEdzdEJ1ZmZlciogYnVmZmVy OwogCi0gICAgQVNTRVJUKCFwcml2LT5idWZmZXIpOwotCi0gICAgR3N0QnVmZmVyKiBidWZmZXIg PSBnc3RfYnVmZmVyX25ld19hbmRfYWxsb2MocmVxdWVzdGVkU2l6ZSk7CisgICAgaWYgKCFwcml2 LT5idWZmZXIpIHsKKyAgICAgICAgYnVmZmVyID0gZ3N0X2J1ZmZlcl9uZXdfYW5kX2FsbG9jKHJl cXVlc3RlZFNpemUpOwogCiAjaWZkZWYgR1NUX0FQSV9WRVJTSU9OXzEKLSAgICBtYXBHc3RCdWZm ZXIoYnVmZmVyKTsKKyAgICAgICAgbWFwR3N0QnVmZmVyKGJ1ZmZlcik7CiAjZW5kaWYKIAotICAg IHByaXYtPmJ1ZmZlciA9IGFkb3B0R1JlZihidWZmZXIpOworICAgICAgICBwcml2LT5idWZmZXIg PSBhZG9wdEdSZWYoYnVmZmVyKTsKKyAgICB9IGVsc2UKKyAgICAgICAgYnVmZmVyID0gcHJpdi0+ YnVmZmVyLmdldCgpOwogCiAgICAgYWN0dWFsU2l6ZSA9IGdldEdzdEJ1ZmZlclNpemUoYnVmZmVy KTsKICAgICByZXR1cm4gZ2V0R3N0QnVmZmVyRGF0YVBvaW50ZXIoYnVmZmVyKTsK 203427 2013-05-31 02:40:55 -0700 2013-05-31 03:24:58 -0700 Patch 116686.diff text/plain 3911 berto ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCBmZTg2Yjc5Li5hOTJiYmY1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjQg QEAKKzIwMTMtMDUtMzEgIEFsYmVydG8gR2FyY2lhICA8YWdhcmNpYUBpZ2FsaWEuY29tPgorCisg ICAgICAgIFtTb3VwXSBbR3N0cmVhbWVyXSBBU1NFUlQgaW4gU3RyZWFtaW5nQ2xpZW50OjpnZXRP ckNyZWF0ZVJlYWRCdWZmZXIKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i dWcuY2dpP2lkPTExNjY4NgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor CisgICAgICAgIFdlIHdlcmUgdXNpbmcgZ19pbnB1dF9zdHJlYW1fcmVhZF9hc3luYygpIHRvIGhh bmRsZSByZWRpcmVjdGlvbnMKKyAgICAgICAgYmVjYXVzZSBvZiBHbm9tZSBidWcgIzY5MTQ4OS4K KworICAgICAgICBUaGlzIHByb2R1Y2VzIGFuIHVuZXhwZWN0ZWQgcmVhZCBhdHRlbXB0IHdoaWNo IHRyaWdnZXJzIGFuIGFzc2VydAorICAgICAgICBpbiBTdHJlYW1pbmdDbGllbnQ6OmdldE9yQ3Jl YXRlUmVhZEJ1ZmZlcigpLgorCisgICAgICAgIE5vdyB0aGF0IHRoZSBHbm9tZSBidWcgaXMgZml4 ZWQgd2UgY2FuIHJlcGxhY2UgcmVhZF9hc3luYygpIHdpdGgKKyAgICAgICAgc2tpcF9hc3luYygp LgorCisgICAgICAgICogcGxhdGZvcm0vbmV0d29yay9zb3VwL1Jlc291cmNlSGFuZGxlU291cC5j cHA6CisgICAgICAgIChXZWJDb3JlOjpSZXNvdXJjZUhhbmRsZTo6ZW5zdXJlUmVhZEJ1ZmZlcik6 CisgICAgICAgIChXZWJDb3JlOjpyZWRpcmVjdFNraXBDYWxsYmFjayk6CisgICAgICAgIChXZWJD b3JlOjpzZW5kUmVxdWVzdENhbGxiYWNrKToKKwogMjAxMy0wNS0zMCAgQW5kcmVhcyBLbGluZyAg PGFrbGluZ0BhcHBsZS5jb20+CiAKICAgICAgICAgQXZvaWQgdXNpbmcgQ1NTT00gc3R5bGUgZGVj bGFyYXRpb25zIGluIEhUTUwgZWRpdGluZy4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3Bs YXRmb3JtL25ldHdvcmsvc291cC9SZXNvdXJjZUhhbmRsZVNvdXAuY3BwIGIvU291cmNlL1dlYkNv cmUvcGxhdGZvcm0vbmV0d29yay9zb3VwL1Jlc291cmNlSGFuZGxlU291cC5jcHAKaW5kZXggNmNk ZDY5Ny4uNWE2MjAwOSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vbmV0d29y ay9zb3VwL1Jlc291cmNlSGFuZGxlU291cC5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvcGxhdGZv cm0vbmV0d29yay9zb3VwL1Jlc291cmNlSGFuZGxlU291cC5jcHAKQEAgLTg2LDYgKzg2LDcgQEAg aW5saW5lIHN0YXRpYyB2b2lkIHNvdXBMb2dQcmludGVyKFNvdXBMb2dnZXIqLCBTb3VwTG9nZ2Vy TG9nTGV2ZWwsIGNoYXIgZGlyZWN0aW8KIH0KIAogc3RhdGljIGJvb2wgbG9hZGluZ1N5bmNocm9u b3VzUmVxdWVzdCA9IGZhbHNlOworc3RhdGljIGNvbnN0IHNpemVfdCBkZWZhdWx0UmVhZEJ1ZmZl clNpemUgPSA4MTkyOwogCiBjbGFzcyBXZWJDb3JlU3luY2hyb25vdXNMb2FkZXIgOiBwdWJsaWMg UmVzb3VyY2VIYW5kbGVDbGllbnQgewogICAgIFdURl9NQUtFX05PTkNPUFlBQkxFKFdlYkNvcmVT eW5jaHJvbm91c0xvYWRlcik7CkBAIC0zMDYsNyArMzA3LDYgQEAgdm9pZCBSZXNvdXJjZUhhbmRs ZTo6ZW5zdXJlUmVhZEJ1ZmZlcigpCiB7CiAgICAgUmVzb3VyY2VIYW5kbGVJbnRlcm5hbCogZCA9 IGdldEludGVybmFsKCk7CiAKLSAgICBzdGF0aWMgY29uc3Qgc2l6ZV90IGRlZmF1bHRSZWFkQnVm ZmVyU2l6ZSA9IDgxOTI7CiAgICAgc2l6ZV90IGJ1ZmZlclNpemU7CiAgICAgY2hhciogYnVmZmVy UHRyID0gY2xpZW50KCktPmdldE9yQ3JlYXRlUmVhZEJ1ZmZlcihkZWZhdWx0UmVhZEJ1ZmZlclNp emUsIGJ1ZmZlclNpemUpOwogICAgIGlmIChidWZmZXJQdHIpIHsKQEAgLTUyNCw3ICs1MjQsNyBA QCBzdGF0aWMgdm9pZCByZWRpcmVjdFNraXBDYWxsYmFjayhHT2JqZWN0KiwgR0FzeW5jUmVzdWx0 KiBhc3luY1Jlc3VsdCwgZ3BvaW50ZXIgZAogCiAgICAgR093blB0cjxHRXJyb3I+IGVycm9yOwog ICAgIFJlc291cmNlSGFuZGxlSW50ZXJuYWwqIGQgPSBoYW5kbGUtPmdldEludGVybmFsKCk7Ci0g ICAgZ3NzaXplIGJ5dGVzU2tpcHBlZCA9IGdfaW5wdXRfc3RyZWFtX3JlYWRfZmluaXNoKGQtPm1f aW5wdXRTdHJlYW0uZ2V0KCksIGFzeW5jUmVzdWx0LCAmZXJyb3Iub3V0UHRyKCkpOworICAgIGdz c2l6ZSBieXRlc1NraXBwZWQgPSBnX2lucHV0X3N0cmVhbV9za2lwX2ZpbmlzaChkLT5tX2lucHV0 U3RyZWFtLmdldCgpLCBhc3luY1Jlc3VsdCwgJmVycm9yLm91dFB0cigpKTsKICAgICBpZiAoZXJy b3IpIHsKICAgICAgICAgaGFuZGxlLT5jbGllbnQoKS0+ZGlkRmFpbChoYW5kbGUuZ2V0KCksIFJl c291cmNlRXJyb3I6OmdlbmVyaWNHRXJyb3IoZXJyb3IuZ2V0KCksIGQtPm1fc291cFJlcXVlc3Qu Z2V0KCkpKTsKICAgICAgICAgY2xlYW51cFNvdXBSZXF1ZXN0T3BlcmF0aW9uKGhhbmRsZS5nZXQo KSk7CkBAIC01MzIsOCArNTMyLDcgQEAgc3RhdGljIHZvaWQgcmVkaXJlY3RTa2lwQ2FsbGJhY2so R09iamVjdCosIEdBc3luY1Jlc3VsdCogYXN5bmNSZXN1bHQsIGdwb2ludGVyIGQKICAgICB9CiAK ICAgICBpZiAoYnl0ZXNTa2lwcGVkID4gMCkgewotICAgICAgICBoYW5kbGUtPmVuc3VyZVJlYWRC dWZmZXIoKTsKLSAgICAgICAgZ19pbnB1dF9zdHJlYW1fcmVhZF9hc3luYyhkLT5tX2lucHV0U3Ry ZWFtLmdldCgpLCBkLT5tX3JlYWRCdWZmZXJQdHIsIGQtPm1fcmVhZEJ1ZmZlclNpemUsIEdfUFJJ T1JJVFlfREVGQVVMVCwKKyAgICAgICAgZ19pbnB1dF9zdHJlYW1fc2tpcF9hc3luYyhkLT5tX2lu cHV0U3RyZWFtLmdldCgpLCBkZWZhdWx0UmVhZEJ1ZmZlclNpemUsIEdfUFJJT1JJVFlfREVGQVVM VCwKICAgICAgICAgICAgIGQtPm1fY2FuY2VsbGFibGUuZ2V0KCksIHJlZGlyZWN0U2tpcENhbGxi YWNrLCBoYW5kbGUuZ2V0KCkpOwogICAgICAgICByZXR1cm47CiAgICAgfQpAQCAtNjgzLDExICs2 ODIsNyBAQCBzdGF0aWMgdm9pZCBzZW5kUmVxdWVzdENhbGxiYWNrKEdPYmplY3QqLCBHQXN5bmNS ZXN1bHQqIHJlc3VsdCwgZ3BvaW50ZXIgZGF0YSkKICAgICBpZiAoc291cE1lc3NhZ2UpIHsKICAg ICAgICAgaWYgKFNPVVBfU1RBVFVTX0lTX1JFRElSRUNUSU9OKHNvdXBNZXNzYWdlLT5zdGF0dXNf Y29kZSkgJiYgc2hvdWxkUmVkaXJlY3QoaGFuZGxlLmdldCgpKSkgewogICAgICAgICAgICAgZC0+ bV9pbnB1dFN0cmVhbSA9IGlucHV0U3RyZWFtOwotICAgICAgICAgICAgLy8gV2UgdXNlIHJlYWRf YXN5bmMoKSByYXRoZXIgdGhhbiBza2lwX2FzeW5jKCkgdG8gd29yayBhcm91bmQKLSAgICAgICAg ICAgIC8vIGh0dHBzOi8vYnVnemlsbGEuZ25vbWUub3JnL3Nob3dfYnVnLmNnaT9pZD02OTE0ODkg dW50aWwgd2UgY2FuCi0gICAgICAgICAgICAvLyBkZXBlbmQgb24gZ2xpYiA+IDIuMzUuNAotICAg ICAgICAgICAgaGFuZGxlLT5lbnN1cmVSZWFkQnVmZmVyKCk7Ci0gICAgICAgICAgICBnX2lucHV0 X3N0cmVhbV9yZWFkX2FzeW5jKGQtPm1faW5wdXRTdHJlYW0uZ2V0KCksIGQtPm1fcmVhZEJ1ZmZl clB0ciwgZC0+bV9yZWFkQnVmZmVyU2l6ZSwgR19QUklPUklUWV9ERUZBVUxULAorICAgICAgICAg ICAgZ19pbnB1dF9zdHJlYW1fc2tpcF9hc3luYyhkLT5tX2lucHV0U3RyZWFtLmdldCgpLCBkZWZh dWx0UmVhZEJ1ZmZlclNpemUsIEdfUFJJT1JJVFlfREVGQVVMVCwKICAgICAgICAgICAgICAgICBk LT5tX2NhbmNlbGxhYmxlLmdldCgpLCByZWRpcmVjdFNraXBDYWxsYmFjaywgaGFuZGxlLmdldCgp KTsKICAgICAgICAgICAgIHJldHVybjsKICAgICAgICAgfQo=