Title: [229012] releases/WebKitGTK/webkit-2.20/Source/WebCore
Revision
229012
Author
[email protected]
Date
2018-02-26 05:22:20 -0800 (Mon, 26 Feb 2018)

Log Message

Merge r228882 - [GCrypt] Remove unsound assertion from CryptoKeyRSA::exportData().
https://bugs.webkit.org/show_bug.cgi?id=183001

Reviewed by Philippe Normand.

This function is called from structured cloning, in particular when storing
a key in IndexedDB. This would trip the assertion if the key in question is
non-exportable.

The assertion was copied from the macOS implementation in r172389; it was
subsequently removed there in r172898 to handle this case.

Test: crypto/subtle/rsa-indexeddb-non-exportable.html

* crypto/gcrypt/CryptoKeyRSAGCrypt.cpp:
(WebCore::CryptoKeyRSA::exportData const): remove assertion.

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.20/Source/WebCore/ChangeLog (229011 => 229012)


--- releases/WebKitGTK/webkit-2.20/Source/WebCore/ChangeLog	2018-02-26 13:22:16 UTC (rev 229011)
+++ releases/WebKitGTK/webkit-2.20/Source/WebCore/ChangeLog	2018-02-26 13:22:20 UTC (rev 229012)
@@ -1,3 +1,22 @@
+2018-02-21  Ms2ger  <[email protected]>
+
+        [GCrypt] Remove unsound assertion from CryptoKeyRSA::exportData().
+        https://bugs.webkit.org/show_bug.cgi?id=183001
+
+        Reviewed by Philippe Normand.
+
+        This function is called from structured cloning, in particular when storing
+        a key in IndexedDB. This would trip the assertion if the key in question is
+        non-exportable.
+
+        The assertion was copied from the macOS implementation in r172389; it was
+        subsequently removed there in r172898 to handle this case.
+
+        Test: crypto/subtle/rsa-indexeddb-non-exportable.html
+
+        * crypto/gcrypt/CryptoKeyRSAGCrypt.cpp:
+        (WebCore::CryptoKeyRSA::exportData const): remove assertion.
+
 2018-02-21  Philippe Normand  <[email protected]>
 
         [GStreamer] We need to adopt GstGlDisplays after GStreamer 1.13.1

Modified: releases/WebKitGTK/webkit-2.20/Source/WebCore/crypto/gcrypt/CryptoKeyRSAGCrypt.cpp (229011 => 229012)


--- releases/WebKitGTK/webkit-2.20/Source/WebCore/crypto/gcrypt/CryptoKeyRSAGCrypt.cpp	2018-02-26 13:22:16 UTC (rev 229011)
+++ releases/WebKitGTK/webkit-2.20/Source/WebCore/crypto/gcrypt/CryptoKeyRSAGCrypt.cpp	2018-02-26 13:22:20 UTC (rev 229012)
@@ -645,8 +645,6 @@
 
 std::unique_ptr<CryptoKeyRSAComponents> CryptoKeyRSA::exportData() const
 {
-    ASSERT(extractable());
-
     switch (type()) {
     case CryptoKeyType::Public:
         return CryptoKeyRSAComponents::createPublic(getRSAKeyParameter(m_platformKey, "n"), getRSAKeyParameter(m_platformKey, "e"));
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to