[webkit-changes] [WebKit/WebKit] 55dbfa: Bug 269902 - [Curl] Add null check in WebSocketTas...

Kenji Shukuwa noreply at github.com
Sun Feb 25 19:03:35 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 55dbfac13e1f1133bb160bae62cfaffed01659f3
      https://github.com/WebKit/WebKit/commit/55dbfac13e1f1133bb160bae62cfaffed01659f3
  Author: Kenji Shukuwa <kenji.shukuwa at open-tec.co.jp>
  Date:   2024-02-25 (Sun, 25 Feb 2024)

  Changed paths:
    M Source/WebKit/NetworkProcess/curl/NetworkDataTaskCurl.cpp
    M Source/WebKit/NetworkProcess/curl/NetworkSessionCurl.cpp
    M Source/WebKit/NetworkProcess/curl/WebSocketTaskCurl.cpp

  Log Message:
  -----------
  Bug 269902 - [Curl] Add null check in WebSocketTaskCurl/NetworkDataTask/NetworkSession
https://bugs.webkit.org/show_bug.cgi?id=269902

Reviewed by Fujii Hironori.

m_channel.session() referenced in WebSocketTaskCurl is defined as WeakPtr<NetworkSession>.
Therefore, nullptr may be returned depending on the timing, so check for null before using them.

Also, NetworkSession::networkStorageSession() referenced by NetworkDataTaskCurl and NetworkSession
can return nullptr, so do a null check before using them.

* Source/WebKit/NetworkProcess/curl/NetworkDataTaskCurl.cpp:
* Source/WebKit/NetworkProcess/curl/NetworkSessionCurl.cpp:
* Source/WebKit/NetworkProcess/curl/WebSocketTaskCurl.cpp:

Canonical link: https://commits.webkit.org/275304@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list