Attachment 343832 Details for Bug 187150 – Patch

[patch] Patch

bug-187150-20180628121111.patch (text/plain), 4.35 KB, created by youenn fablet on 2018-06-28 12:11:11 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: youenn fablet

Created: 2018-06-28 12:11:11 PDT

Size: 4.35 KB

patch

obsolete

>Subversion Revision: 233256
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 992ad68b3707e52189fdb213703b6665c531fb2f..51b72c81c3e227443dacc555e0c8f5995ca72a88 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,26 @@
>+2018-06-28  Youenn Fablet  <youenn@apple.com>
>+
>+        Handle the case of registerMDNSNameCallback called several times
>+        https://bugs.webkit.org/show_bug.cgi?id=187150
>+        <rdar://problem/41329832>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        This is a speculative fix on the basis that registerMDNSNameCallback may be called several times.
>+        In that case, we would have freed the context after the first call and would reuse it for the second call.
>+
>+        Instead, keep a map of pending requests and pass to registerMDNSNameCallback an identifier to that map.
>+        If the map has no value for that identifier, return early.
>+
>+        * NetworkProcess/webrtc/NetworkMDNSRegister.cpp:
>+        (WebKit::NetworkMDNSRegister::~NetworkMDNSRegister):
>+        (WebKit::pendingRegistrationRequests):
>+        (WebKit::registerMDNSNameCallback):
>+        (WebKit::NetworkMDNSRegister::clearPendingRequests):
>+        (WebKit::NetworkMDNSRegister::registerMDNSName):
>+        * NetworkProcess/webrtc/NetworkMDNSRegister.h:
>+        (): Deleted.
>+
> 2018-06-28  Youenn Fablet  <youenn@apple.com>
> 
>         Add sandbox to microdone plugin
>diff --git a/Source/WebKit/NetworkProcess/webrtc/NetworkMDNSRegister.cpp b/Source/WebKit/NetworkProcess/webrtc/NetworkMDNSRegister.cpp
>index 8f9ca1065d193052403616b77780af98c0d910f9..90234fb06ac8d178b67bb60f90fa5ec8f6f0c057 100644
>--- a/Source/WebKit/NetworkProcess/webrtc/NetworkMDNSRegister.cpp
>+++ b/Source/WebKit/NetworkProcess/webrtc/NetworkMDNSRegister.cpp
>@@ -79,9 +79,20 @@ struct PendingRegistrationRequest {
>     DNSRecordRef record;
> };
> 
>+
>+static uintptr_t pendingRegistrationRequestCount = 1;
>+static HashMap<uintptr_t, std::unique_ptr<PendingRegistrationRequest>>& pendingRegistrationRequests()
>+{
>+    ASSERT(RunLoop::isMain());
>+    static NeverDestroyed<HashMap<uintptr_t, std::unique_ptr<PendingRegistrationRequest>>> map;
>+    return map;
>+}
>+
> static void registerMDNSNameCallback(DNSServiceRef, DNSRecordRef record, DNSServiceFlags, DNSServiceErrorType errorCode, void *context)
> {
>-    std::unique_ptr<PendingRegistrationRequest> request { static_cast<PendingRegistrationRequest*>(context) };
>+    auto request = pendingRegistrationRequests().take(reinterpret_cast<uintptr_t>(context));
>+    if (!request)
>+        return;
> 
>     RELEASE_LOG_IF_ALLOWED_IN_CALLBACK(request->sessionID, "registerMDNSNameCallback with error %d", errorCode);
> 
>@@ -115,7 +126,7 @@ void NetworkMDNSRegister::registerMDNSName(uint64_t requestIdentifier, PAL::Sess
>         service = iterator->value;
> 
>     String baseName = createCanonicalUUIDString();
>-    String name = makeString(baseName, String::number(++m_registrationCount), ".local");
>+    String name = makeString(baseName, String::number(pendingRegistrationRequestCount), ".local");
> 
>     auto ip = inet_addr(ipAddress.utf8().data());
> 
>@@ -138,13 +149,13 @@ void NetworkMDNSRegister::registerMDNSName(uint64_t requestIdentifier, PAL::Sess
>         &ip,
>         0,
>         registerMDNSNameCallback,
>-        pendingRequest.get());
>+        reinterpret_cast<void*>(pendingRegistrationRequestCount));
>     if (error) {
>         RELEASE_LOG_IF_ALLOWED(sessionID, "registerMDNSName DNSServiceRegisterRecord error %d", error);
>         m_connection.connection().send(Messages::WebMDNSRegister::FinishedRegisteringMDNSName { requestIdentifier, makeUnexpected(MDNSRegisterError::DNSSD) }, 0);
>         return;
>     }
>-    pendingRequest.release();
>+    pendingRegistrationRequests().add(pendingRegistrationRequestCount++, WTFMove(pendingRequest));
> }
> 
> struct PendingResolutionRequest {
>diff --git a/Source/WebKit/NetworkProcess/webrtc/NetworkMDNSRegister.h b/Source/WebKit/NetworkProcess/webrtc/NetworkMDNSRegister.h
>index 081138808c07a0990b6fce6b83173d8e77d6bd58..7ab0b7bd952875b26e5d480524e6afedcdd7b9e1 100644
>--- a/Source/WebKit/NetworkProcess/webrtc/NetworkMDNSRegister.h
>+++ b/Source/WebKit/NetworkProcess/webrtc/NetworkMDNSRegister.h
>@@ -71,8 +71,6 @@ private:
>     NetworkConnectionToWebProcess& m_connection;
> #if ENABLE_MDNS
>     HashMap<WebCore::DocumentIdentifier, DNSServiceRef> m_services;
>-
>-    uint64_t m_registrationCount { 0 };
> #endif
> };
>

Actions: View | Formatted Diff | Diff

Attachments on bug 187150: 343831 | 343832