Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 33777aff18f133fb902b6f6b08438fe36f2bbeaa
      
https://github.com/WebKit/WebKit/commit/33777aff18f133fb902b6f6b08438fe36f2bbeaa
  Author: Javier Fernandez <jfernan...@igalia.com>
  Date:   2025-08-27 (Wed, 27 Aug 2025)

  Changed paths:
    M LayoutTests/platform/glib/TestExpectations
    M Source/WebCore/crypto/gcrypt/CryptoKeyOKPGCrypt.cpp

  Log Message:
  -----------
  Rename the key-pair elements returned in the gcryptGenerateX25519Keys method
https://bugs.webkit.org/show_bug.cgi?id=289693

Reviewed by Carlos Garcia Campos.

The gcryptGenerateX25519Keys method returns a std::pair of KeyMaterial variables
which should contain the public and private key. The caller expects the first
element to be the public key.

We were returning the data in the wrong order, so among other issues, it caused
that the public key generated by the JWK export of the private key doesn't match
the original key-pair's public key material.

This change fixes the issue by switching the variable names used to hold the
private and public key pair. This is also more consistent with the nomenclature
used in the Ed25519 generateKey() method..

Regarding tests, this change make the successes_X25519.https.any.html tests to
pass, so updating also the platform-specific TestExpectations file.

* LayoutTests/platform/glib/TestExpectations:
* Source/WebCore/crypto/gcrypt/CryptoKeyOKPGCrypt.cpp:
(WebCore::gcryptGenerateX25519Keys):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to