Attachment 373646 Details for Bug 199580 – Patch

[patch] Patch

bug-199580-20190708111921.patch (text/plain), 1.68 KB, created by Alex Christensen on 2019-07-08 11:19:21 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Christensen

Created: 2019-07-08 11:19:21 PDT

Size: 1.68 KB

patch

obsolete

>Subversion Revision: 247102
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index e1c36e910e82b7445e4c679ddba90f0a7fa1ef08..da8c6c72b3e142dcb0e6f82e34af8fba527fa5bf 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,16 @@
>+2019-07-08  Alex Christensen  <achristensen@webkit.org>
>+
>+        Prevent null dereferencing in SubresourceLoader::init's lambda
>+        https://bugs.webkit.org/show_bug.cgi?id=199580
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Crash logs indicate that sometimes m_documentLoader is null.
>+        This is similar to https://bugs.webkit.org/show_bug.cgi?id=187360
>+
>+        * loader/SubresourceLoader.cpp:
>+        (WebCore::SubresourceLoader::init):
>+
> 2019-07-03  Sam Weinig  <weinig@apple.com>
> 
>         Adopt simple structured bindings in more places
>diff --git a/Source/WebCore/loader/SubresourceLoader.cpp b/Source/WebCore/loader/SubresourceLoader.cpp
>index e524d5720bbf84f24bfbd25eb14cf1230f112830..4ea00a51d5429d6652d3cea3ff4ae8dc397629a0 100644
>--- a/Source/WebCore/loader/SubresourceLoader.cpp
>+++ b/Source/WebCore/loader/SubresourceLoader.cpp
>@@ -164,6 +164,10 @@ void SubresourceLoader::init(ResourceRequest&& request, CompletionHandler<void(b
>     ResourceLoader::init(WTFMove(request), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)] (bool initialized) mutable {
>         if (!initialized)
>             return completionHandler(false);
>+        if (!m_documentLoader) {
>+            ASSERT_NOT_REACHED();
>+            return completionHandler(false);
>+        }
>         ASSERT(!reachedTerminalState());
>         m_state = Initialized;
>         m_documentLoader->addSubresourceLoader(this);

Actions: View | Formatted Diff | Diff

Attachments on bug 199580: 373646 | 373667