Title: [280747] trunk/Source/WebCore
Revision
280747
Author
[email protected]
Date
2021-08-06 18:16:27 -0700 (Fri, 06 Aug 2021)

Log Message

CrashTracer: com.apple.WebKit.WebContent at com.apple.WebCore: WebCore::CryptoKeyRSA::exportJwk const
https://bugs.webkit.org/show_bug.cgi?id=228878
<rdar://problem/60147231>

Reviewed by Brent Fulgham.

exportData() could return nullptr. We should check the value of
rsaComponents before trying to use it.

* crypto/keys/CryptoKeyRSA.cpp:
(WebCore::CryptoKeyRSA::exportJwk const):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (280746 => 280747)


--- trunk/Source/WebCore/ChangeLog	2021-08-07 00:57:57 UTC (rev 280746)
+++ trunk/Source/WebCore/ChangeLog	2021-08-07 01:16:27 UTC (rev 280747)
@@ -1,3 +1,17 @@
+2021-08-06  Kate Cheney  <[email protected]>
+
+        CrashTracer: com.apple.WebKit.WebContent at com.apple.WebCore: WebCore::CryptoKeyRSA::exportJwk const
+        https://bugs.webkit.org/show_bug.cgi?id=228878
+        <rdar://problem/60147231>
+
+        Reviewed by Brent Fulgham.
+
+        exportData() could return nullptr. We should check the value of
+        rsaComponents before trying to use it.
+
+        * crypto/keys/CryptoKeyRSA.cpp:
+        (WebCore::CryptoKeyRSA::exportJwk const):
+
 2021-08-06  Alexander Mikhaylenko  <[email protected]>
 
         REGRESSION(r279987): [GTK][WPE] It caused 2 new test failures

Modified: trunk/Source/WebCore/crypto/keys/CryptoKeyRSA.cpp (280746 => 280747)


--- trunk/Source/WebCore/crypto/keys/CryptoKeyRSA.cpp	2021-08-07 00:57:57 UTC (rev 280746)
+++ trunk/Source/WebCore/crypto/keys/CryptoKeyRSA.cpp	2021-08-07 01:16:27 UTC (rev 280747)
@@ -139,6 +139,9 @@
 
     auto rsaComponents = exportData();
 
+    if (!rsaComponents)
+        return result;
+
     // public key
     result.n = base64URLEncodeToString(rsaComponents->modulus());
     result.e = base64URLEncodeToString(rsaComponents->exponent());
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to