204460 2019-11-21 10:55:55 -0800 Null check callback in NetworkConnectionToWebProcess::didDeliverMessagePortMessages 2019-11-22 15:15:22 -0800 1 1 1 Unclassified WebKit New Bugs WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 achristensen achristensen cdumez mjs webkit-bug-importer youennf oldest_to_newest 1592441 0 achristensen 2019-11-21 10:55:55 -0800 Null check callback in NetworkConnectionToWebProcess::didDeliverMessagePortMessages 1592443 1 384072 achristensen 2019-11-21 10:56:38 -0800 Created attachment 384072 Patch 1592483 2 384072 mjs 2019-11-21 12:17:04 -0800 Comment on attachment 384072 Patch Do we understand why the callback can be null here? 1592484 3 384072 cdumez 2019-11-21 12:18:28 -0800 Comment on attachment 384072 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=384072&action=review > Source/WebKit/ChangeLog:5 > + <rdar://57448618> This does not look like a valid radar number? 1592485 4 384072 cdumez 2019-11-21 12:21:06 -0800 Comment on attachment 384072 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=384072&action=review > Source/WebKit/NetworkProcess/NetworkConnectionToWebProcess.cpp:992 > + if (auto callback = m_messageBatchDeliveryCompletionHandlers.take(messageBatchIdentifier)) This should not be possible in theory, I think we need to find the root cause. m_messageBatchDeliveryCompletionHandlers is only modified here and in takeAllMessagesForPort(). So either we're getting an unexpected didDeliverMessagePortMessages or the NetworkConnectionToWebProcess is stale. 1592486 5 384072 cdumez 2019-11-21 12:23:59 -0800 Comment on attachment 384072 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=384072&action=review >> Source/WebKit/NetworkProcess/NetworkConnectionToWebProcess.cpp:992 >> + if (auto callback = m_messageBatchDeliveryCompletionHandlers.take(messageBatchIdentifier)) > > This should not be possible in theory, I think we need to find the root cause. > > m_messageBatchDeliveryCompletionHandlers is only modified here and in takeAllMessagesForPort(). So either we're getting an unexpected didDeliverMessagePortMessages or the NetworkConnectionToWebProcess is stale. Maybe if the network process crashes in between the takeAllMessagesForPort() call and the didDeliverMessagePortMessages() one? Then it'd be a new NetworkConnectionToWebProcess. The WebProcess would send the DidDeliverMessagePortMessages IPC to the new network process, which does not know about the takeAllMessagesForPort request. 1592488 6 cdumez 2019-11-21 12:24:41 -0800 (In reply to Chris Dumez from comment #5) > Comment on attachment 384072 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=384072&action=review > > >> Source/WebKit/NetworkProcess/NetworkConnectionToWebProcess.cpp:992 > >> + if (auto callback = m_messageBatchDeliveryCompletionHandlers.take(messageBatchIdentifier)) > > > > This should not be possible in theory, I think we need to find the root cause. > > > > m_messageBatchDeliveryCompletionHandlers is only modified here and in takeAllMessagesForPort(). So either we're getting an unexpected didDeliverMessagePortMessages or the NetworkConnectionToWebProcess is stale. > > Maybe if the network process crashes in between the takeAllMessagesForPort() > call and the didDeliverMessagePortMessages() one? Then it'd be a new > NetworkConnectionToWebProcess. The WebProcess would send the > DidDeliverMessagePortMessages IPC to the new network process, which does not > know about the takeAllMessagesForPort request. If that's what's doing on, it may be possible to write a test for this. 1592927 7 384072 achristensen 2019-11-22 11:35:15 -0800 Comment on attachment 384072 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=384072&action=review >> Source/WebKit/ChangeLog:5 >> + <rdar://57448618> > > This does not look like a valid radar number? I have no idea where that came from. Should be rdar://problem/57348618 and will update. >>>> Source/WebKit/NetworkProcess/NetworkConnectionToWebProcess.cpp:992 >>>> + if (auto callback = m_messageBatchDeliveryCompletionHandlers.take(messageBatchIdentifier)) >>> >>> This should not be possible in theory, I think we need to find the root cause. >>> >>> m_messageBatchDeliveryCompletionHandlers is only modified here and in takeAllMessagesForPort(). So either we're getting an unexpected didDeliverMessagePortMessages or the NetworkConnectionToWebProcess is stale. >> >> Maybe if the network process crashes in between the takeAllMessagesForPort() call and the didDeliverMessagePortMessages() one? Then it'd be a new NetworkConnectionToWebProcess. The WebProcess would send the DidDeliverMessagePortMessages IPC to the new network process, which does not know about the takeAllMessagesForPort request. > > If that's what's doing on, it may be possible to write a test for this. I think that's what's going on and that was mentioned in https://bugs.webkit.org/show_bug.cgi?id=201299#c4 , but there doesn't seem to be a good way to call _terminateNetworkProcess exactly between these two calls. 1592938 8 384072 cdumez 2019-11-22 11:59:33 -0800 Comment on attachment 384072 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=384072&action=review >>>>> Source/WebKit/NetworkProcess/NetworkConnectionToWebProcess.cpp:992 >>>>> + if (auto callback = m_messageBatchDeliveryCompletionHandlers.take(messageBatchIdentifier)) >>>> >>>> This should not be possible in theory, I think we need to find the root cause. >>>> >>>> m_messageBatchDeliveryCompletionHandlers is only modified here and in takeAllMessagesForPort(). So either we're getting an unexpected didDeliverMessagePortMessages or the NetworkConnectionToWebProcess is stale. >>> >>> Maybe if the network process crashes in between the takeAllMessagesForPort() call and the didDeliverMessagePortMessages() one? Then it'd be a new NetworkConnectionToWebProcess. The WebProcess would send the DidDeliverMessagePortMessages IPC to the new network process, which does not know about the takeAllMessagesForPort request. >> >> If that's what's doing on, it may be possible to write a test for this. > > I think that's what's going on and that was mentioned in https://bugs.webkit.org/show_bug.cgi?id=201299#c4 , but there doesn't seem to be a good way to call _terminateNetworkProcess exactly between these two calls. Then the ChangeLog should state so. Also maybe a comment in the code indicating why this can happen. 1592959 9 384185 achristensen 2019-11-22 12:46:15 -0800 Created attachment 384185 Patch 1593030 10 384185 cdumez 2019-11-22 14:53:13 -0800 Comment on attachment 384185 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=384185&action=review > Source/WebKit/ChangeLog:10 > + a web process may have an identifier that becomes stale. In this case, we should not crsah again. Typo: crash > Source/WebKit/NetworkProcess/NetworkConnectionToWebProcess.cpp:1001 > + ASSERT_NOT_REACHED(); Since we know this can happen in practice and we have a valid explanation for it, I do not think we should ASSERT. 1593031 11 384201 achristensen 2019-11-22 14:54:58 -0800 Created attachment 384201 Patch 1593050 12 achristensen 2019-11-22 15:14:26 -0800 http://trac.webkit.org/r252809 1593053 13 webkit-bug-importer 2019-11-22 15:15:22 -0800 <rdar://problem/57442544> 384072 2019-11-21 10:56:38 -0800 2019-11-22 12:46:14 -0800 Patch bug-204460-20191121105638.patch text/plain 1574 achristensen SW5kZXg6IFNvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJL aXQvQ2hhbmdlTG9nCShyZXZpc2lvbiAyNTI3NDEpCisrKyBTb3VyY2UvV2ViS2l0L0NoYW5nZUxv Zwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE0IEBACisyMDE5LTExLTIxICBBbGV4IENocmlz dGVuc2VuICA8YWNocmlzdGVuc2VuQHdlYmtpdC5vcmc+CisKKyAgICAgICAgTnVsbCBjaGVjayBj YWxsYmFjayBpbiBOZXR3b3JrQ29ubmVjdGlvblRvV2ViUHJvY2Vzczo6ZGlkRGVsaXZlck1lc3Nh Z2VQb3J0TWVzc2FnZXMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu Y2dpP2lkPTIwNDQ2MAorICAgICAgICA8cmRhcjovLzU3NDQ4NjE4PgorCisgICAgICAgIFJldmll d2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogTmV0d29ya1Byb2Nlc3MvTmV0d29y a0Nvbm5lY3Rpb25Ub1dlYlByb2Nlc3MuY3BwOgorICAgICAgICAoV2ViS2l0OjpOZXR3b3JrQ29u bmVjdGlvblRvV2ViUHJvY2Vzczo6ZGlkRGVsaXZlck1lc3NhZ2VQb3J0TWVzc2FnZXMpOgorCiAy MDE5LTExLTIwICBDaGFuZ1Nlb2sgT2ggIDxjaGFuZ3Nlb2tAd2Via2l0Lm9yZz4KIAogICAgICAg ICBbR1RLXSBBZGQgQU5HTEUgYmFja2VuZCB0byBHVEsgcG9ydApJbmRleDogU291cmNlL1dlYktp dC9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrQ29ubmVjdGlvblRvV2ViUHJvY2Vzcy5jcHAKPT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PQotLS0gU291cmNlL1dlYktpdC9OZXR3b3JrUHJvY2Vzcy9OZXR3b3JrQ29ubmVjdGlvblRv V2ViUHJvY2Vzcy5jcHAJKHJldmlzaW9uIDI1MTgxMykKKysrIFNvdXJjZS9XZWJLaXQvTmV0d29y a1Byb2Nlc3MvTmV0d29ya0Nvbm5lY3Rpb25Ub1dlYlByb2Nlc3MuY3BwCSh3b3JraW5nIGNvcHkp CkBAIC05ODksOSArOTg5LDggQEAgdm9pZCBOZXR3b3JrQ29ubmVjdGlvblRvV2ViUHJvY2Vzczo6 dGFrZQogCiB2b2lkIE5ldHdvcmtDb25uZWN0aW9uVG9XZWJQcm9jZXNzOjpkaWREZWxpdmVyTWVz c2FnZVBvcnRNZXNzYWdlcyh1aW50NjRfdCBtZXNzYWdlQmF0Y2hJZGVudGlmaWVyKQogewotICAg IGF1dG8gY2FsbGJhY2sgPSBtX21lc3NhZ2VCYXRjaERlbGl2ZXJ5Q29tcGxldGlvbkhhbmRsZXJz LnRha2UobWVzc2FnZUJhdGNoSWRlbnRpZmllcik7Ci0gICAgQVNTRVJUKGNhbGxiYWNrKTsKLSAg ICBjYWxsYmFjaygpOworICAgIGlmIChhdXRvIGNhbGxiYWNrID0gbV9tZXNzYWdlQmF0Y2hEZWxp dmVyeUNvbXBsZXRpb25IYW5kbGVycy50YWtlKG1lc3NhZ2VCYXRjaElkZW50aWZpZXIpKQorICAg ICAgICBjYWxsYmFjaygpOwogfQogCiB2b2lkIE5ldHdvcmtDb25uZWN0aW9uVG9XZWJQcm9jZXNz Ojpwb3N0TWVzc2FnZVRvUmVtb3RlKE1lc3NhZ2VXaXRoTWVzc2FnZVBvcnRzJiYgbWVzc2FnZSwg Y29uc3QgTWVzc2FnZVBvcnRJZGVudGlmaWVyJiBwb3J0KQo= 384185 2019-11-22 12:46:15 -0800 2019-11-22 14:54:57 -0800 Patch bug-204460-20191122124614.patch text/plain 2128 achristensen SW5kZXg6IFNvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJL aXQvQ2hhbmdlTG9nCShyZXZpc2lvbiAyNTI3OTMpCisrKyBTb3VyY2UvV2ViS2l0L0NoYW5nZUxv Zwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE4IEBACisyMDE5LTExLTIyICBBbGV4IENocmlz dGVuc2VuICA8YWNocmlzdGVuc2VuQHdlYmtpdC5vcmc+CisKKyAgICAgICAgTnVsbCBjaGVjayBj YWxsYmFjayBpbiBOZXR3b3JrQ29ubmVjdGlvblRvV2ViUHJvY2Vzczo6ZGlkRGVsaXZlck1lc3Nh Z2VQb3J0TWVzc2FnZXMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu Y2dpP2lkPTIwNDQ2MAorICAgICAgICA8cmRhcjovL3Byb2JsZW0vNTczNDg2MTg+CisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgV2hlbiB0aGUgbmV0d29y ayBwcm9jZXNzIGNyYXNoZXMgZHVyaW5nIGEgbWVzc2FnZSBwb3J0IGNvbm5lY3Rpb24gZXN0YWJs aXNobWVudCwKKyAgICAgICAgYSB3ZWIgcHJvY2VzcyBtYXkgaGF2ZSBhbiBpZGVudGlmaWVyIHRo YXQgYmVjb21lcyBzdGFsZS4gIEluIHRoaXMgY2FzZSwgd2Ugc2hvdWxkIG5vdCBjcnNhaCBhZ2Fp bi4KKyAgICAgICAgTm8gdGVzdCBiZWNhdXNlIHRoaXMgb25seSBoYXBwZW5zIGR1cmluZyBhIHJh cmUgcmFjZSBjb25kaXRpb24gYWZ0ZXIgYSBuZXR3b3JrIHByb2Nlc3MgY3Jhc2gsIGJ1dCB0aGUg cmFkYXIgc2hvd3MgcmVjb3JkcyBvZiB0aGlzIGhhcHBlbmluZy4KKworICAgICAgICAqIE5ldHdv cmtQcm9jZXNzL05ldHdvcmtDb25uZWN0aW9uVG9XZWJQcm9jZXNzLmNwcDoKKyAgICAgICAgKFdl YktpdDo6TmV0d29ya0Nvbm5lY3Rpb25Ub1dlYlByb2Nlc3M6OmRpZERlbGl2ZXJNZXNzYWdlUG9y dE1lc3NhZ2VzKToKKwogMjAxOS0xMS0yMiAgQ29ucmFkIFNodWx0eiAgPGNvbnJhZF9zaHVsdHpA YXBwbGUuY29tPgogCiAgICAgICAgIFJlbW92ZSAtW19XS1RodW1ibmFpbFZpZXcgdXNlc1NuYXBz aG90L3NldFVzZXNTbmFwc2hvdDpdCkluZGV4OiBTb3VyY2UvV2ViS2l0L05ldHdvcmtQcm9jZXNz L05ldHdvcmtDb25uZWN0aW9uVG9XZWJQcm9jZXNzLmNwcAo9PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2Uv V2ViS2l0L05ldHdvcmtQcm9jZXNzL05ldHdvcmtDb25uZWN0aW9uVG9XZWJQcm9jZXNzLmNwcAko cmV2aXNpb24gMjUyNzgxKQorKysgU291cmNlL1dlYktpdC9OZXR3b3JrUHJvY2Vzcy9OZXR3b3Jr Q29ubmVjdGlvblRvV2ViUHJvY2Vzcy5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTk5NCw5ICs5OTQs MTEgQEAgdm9pZCBOZXR3b3JrQ29ubmVjdGlvblRvV2ViUHJvY2Vzczo6dGFrZQogCiB2b2lkIE5l dHdvcmtDb25uZWN0aW9uVG9XZWJQcm9jZXNzOjpkaWREZWxpdmVyTWVzc2FnZVBvcnRNZXNzYWdl cyh1aW50NjRfdCBtZXNzYWdlQmF0Y2hJZGVudGlmaWVyKQogewotICAgIGF1dG8gY2FsbGJhY2sg PSBtX21lc3NhZ2VCYXRjaERlbGl2ZXJ5Q29tcGxldGlvbkhhbmRsZXJzLnRha2UobWVzc2FnZUJh dGNoSWRlbnRpZmllcik7Ci0gICAgQVNTRVJUKGNhbGxiYWNrKTsKLSAgICBjYWxsYmFjaygpOwor ICAgIC8vIE51bGwgY2hlY2sgb25seSBuZWNlc3NhcnkgZm9yIHJhcmUgY29uZGl0aW9uIHdoZXJl IG5ldHdvcmsgcHJvY2VzcyBjcmFzaGVzIGR1cmluZyBtZXNzYWdlIHBvcnQgY29ubmVjdGlvbiBl c3RhYmxpc2htZW50LgorICAgIGlmIChhdXRvIGNhbGxiYWNrID0gbV9tZXNzYWdlQmF0Y2hEZWxp dmVyeUNvbXBsZXRpb25IYW5kbGVycy50YWtlKG1lc3NhZ2VCYXRjaElkZW50aWZpZXIpKQorICAg ICAgICBjYWxsYmFjaygpOworICAgIGVsc2UKKyAgICAgICAgQVNTRVJUX05PVF9SRUFDSEVEKCk7 CiB9CiAKIHZvaWQgTmV0d29ya0Nvbm5lY3Rpb25Ub1dlYlByb2Nlc3M6OnBvc3RNZXNzYWdlVG9S ZW1vdGUoTWVzc2FnZVdpdGhNZXNzYWdlUG9ydHMmJiBtZXNzYWdlLCBjb25zdCBNZXNzYWdlUG9y dElkZW50aWZpZXImIHBvcnQpCg== 384201 2019-11-22 14:54:58 -0800 2019-11-22 15:12:33 -0800 Patch bug-204460-20191122145457.patch text/plain 2047 achristensen SW5kZXg6IFNvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJL aXQvQ2hhbmdlTG9nCShyZXZpc2lvbiAyNTI3OTgpCisrKyBTb3VyY2UvV2ViS2l0L0NoYW5nZUxv Zwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE4IEBACisyMDE5LTExLTIyICBBbGV4IENocmlz dGVuc2VuICA8YWNocmlzdGVuc2VuQHdlYmtpdC5vcmc+CisKKyAgICAgICAgTnVsbCBjaGVjayBj YWxsYmFjayBpbiBOZXR3b3JrQ29ubmVjdGlvblRvV2ViUHJvY2Vzczo6ZGlkRGVsaXZlck1lc3Nh Z2VQb3J0TWVzc2FnZXMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu Y2dpP2lkPTIwNDQ2MAorICAgICAgICA8cmRhcjovL3Byb2JsZW0vNTczNDg2MTg+CisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgV2hlbiB0aGUgbmV0d29y ayBwcm9jZXNzIGNyYXNoZXMgZHVyaW5nIGEgbWVzc2FnZSBwb3J0IGNvbm5lY3Rpb24gZXN0YWJs aXNobWVudCwKKyAgICAgICAgYSB3ZWIgcHJvY2VzcyBtYXkgaGF2ZSBhbiBpZGVudGlmaWVyIHRo YXQgYmVjb21lcyBzdGFsZS4gIEluIHRoaXMgY2FzZSwgd2Ugc2hvdWxkIG5vdCBjcmFzaCBhZ2Fp bi4KKyAgICAgICAgTm8gdGVzdCBiZWNhdXNlIHRoaXMgb25seSBoYXBwZW5zIGR1cmluZyBhIHJh cmUgcmFjZSBjb25kaXRpb24gYWZ0ZXIgYSBuZXR3b3JrIHByb2Nlc3MgY3Jhc2gsIGJ1dCB0aGUg cmFkYXIgc2hvd3MgcmVjb3JkcyBvZiB0aGlzIGhhcHBlbmluZy4KKworICAgICAgICAqIE5ldHdv cmtQcm9jZXNzL05ldHdvcmtDb25uZWN0aW9uVG9XZWJQcm9jZXNzLmNwcDoKKyAgICAgICAgKFdl YktpdDo6TmV0d29ya0Nvbm5lY3Rpb25Ub1dlYlByb2Nlc3M6OmRpZERlbGl2ZXJNZXNzYWdlUG9y dE1lc3NhZ2VzKToKKwogMjAxOS0xMS0yMiAgQWxleCBDaHJpc3RlbnNlbiAgPGFjaHJpc3RlbnNl bkB3ZWJraXQub3JnPgogCiAgICAgICAgIFJldmVydCByMjUwNDIxCkluZGV4OiBTb3VyY2UvV2Vi S2l0L05ldHdvcmtQcm9jZXNzL05ldHdvcmtDb25uZWN0aW9uVG9XZWJQcm9jZXNzLmNwcAo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci0tLSBTb3VyY2UvV2ViS2l0L05ldHdvcmtQcm9jZXNzL05ldHdvcmtDb25uZWN0aW9u VG9XZWJQcm9jZXNzLmNwcAkocmV2aXNpb24gMjUyNzgxKQorKysgU291cmNlL1dlYktpdC9OZXR3 b3JrUHJvY2Vzcy9OZXR3b3JrQ29ubmVjdGlvblRvV2ViUHJvY2Vzcy5jcHAJKHdvcmtpbmcgY29w eSkKQEAgLTk5NCw5ICs5OTQsOSBAQCB2b2lkIE5ldHdvcmtDb25uZWN0aW9uVG9XZWJQcm9jZXNz Ojp0YWtlCiAKIHZvaWQgTmV0d29ya0Nvbm5lY3Rpb25Ub1dlYlByb2Nlc3M6OmRpZERlbGl2ZXJN ZXNzYWdlUG9ydE1lc3NhZ2VzKHVpbnQ2NF90IG1lc3NhZ2VCYXRjaElkZW50aWZpZXIpCiB7Ci0g ICAgYXV0byBjYWxsYmFjayA9IG1fbWVzc2FnZUJhdGNoRGVsaXZlcnlDb21wbGV0aW9uSGFuZGxl cnMudGFrZShtZXNzYWdlQmF0Y2hJZGVudGlmaWVyKTsKLSAgICBBU1NFUlQoY2FsbGJhY2spOwot ICAgIGNhbGxiYWNrKCk7CisgICAgLy8gTnVsbCBjaGVjayBvbmx5IG5lY2Vzc2FyeSBmb3IgcmFy ZSBjb25kaXRpb24gd2hlcmUgbmV0d29yayBwcm9jZXNzIGNyYXNoZXMgZHVyaW5nIG1lc3NhZ2Ug cG9ydCBjb25uZWN0aW9uIGVzdGFibGlzaG1lbnQuCisgICAgaWYgKGF1dG8gY2FsbGJhY2sgPSBt X21lc3NhZ2VCYXRjaERlbGl2ZXJ5Q29tcGxldGlvbkhhbmRsZXJzLnRha2UobWVzc2FnZUJhdGNo SWRlbnRpZmllcikpCisgICAgICAgIGNhbGxiYWNrKCk7CiB9CiAKIHZvaWQgTmV0d29ya0Nvbm5l Y3Rpb25Ub1dlYlByb2Nlc3M6OnBvc3RNZXNzYWdlVG9SZW1vdGUoTWVzc2FnZVdpdGhNZXNzYWdl UG9ydHMmJiBtZXNzYWdlLCBjb25zdCBNZXNzYWdlUG9ydElkZW50aWZpZXImIHBvcnQpCg==