New issue
Advanced search Search tips

Issue 654143 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Replace scoped_openssl_types.h with BoringSSL scopers

Project Member Reported by davidben@chromium.org, Oct 8 2016

Issue description

BoringSSL has scopers now. They're much more forward-decl friendly and can be shared with other BoringSSL consumers. Switch everything to them.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 11 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d80c12cc3341c364f67c04b1a6153a1ee34be25d

commit d80c12cc3341c364f67c04b1a6153a1ee34be25d
Author: davidben <davidben@chromium.org>
Date: Tue Oct 11 00:13:49 2016

Use BoringSSL scopers in //net.

BUG= 654143 

Review-Url: https://codereview.chromium.org/2400033005
Cr-Commit-Position: refs/heads/master@{#424298}

[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/base/keygen_handler_openssl.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/base/keygen_handler_unittest.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/base/openssl_private_key_store_android.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/cert/cert_database_openssl.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/cert/cert_verify_proc_ios.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/cert/cert_verify_proc_openssl.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/cert/ct_log_verifier.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/cert/ct_objects_extractor.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/cert/internal/verify_name_match.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/cert/internal/verify_signed_data.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/cert/jwk_serializer.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/cert/x509_certificate_ios.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/cert/x509_certificate_openssl.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/cert/x509_util_openssl.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/quic/chromium/crypto/proof_source_chromium.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/quic/core/crypto/channel_id.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/quic/core/crypto/p256_key_exchange.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/quic/core/crypto/p256_key_exchange.h
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/quic/test_tools/crypto_test_utils.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/socket/ssl_client_socket_impl.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/socket/ssl_client_socket_impl.h
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/socket/ssl_client_socket_unittest.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/socket/ssl_server_socket_impl.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/socket/ssl_server_socket_impl.h
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/socket/ssl_server_socket_unittest.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/openssl_client_key_store.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/openssl_client_key_store_unittest.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/openssl_ssl_util.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/openssl_ssl_util.h
[delete] https://crrev.com/6be4d9fbc0b33047dfe901e2bbe2c322efda30cd/net/ssl/scoped_openssl_types.h
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/ssl_client_session_cache.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/ssl_client_session_cache.h
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/ssl_client_session_cache_unittest.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/ssl_platform_key_android.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/ssl_platform_key_chromecast.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/ssl_platform_key_mac.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/ssl_platform_key_nss.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/ssl_platform_key_win.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/test_ssl_private_key.cc
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/test_ssl_private_key.h
[modify] https://crrev.com/d80c12cc3341c364f67c04b1a6153a1ee34be25d/net/ssl/token_binding.cc

Project Member

Comment 2 by bugdroid1@chromium.org, Oct 11 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7e7722810efc072552797165546f7f77f245c396

commit 7e7722810efc072552797165546f7f77f245c396
Author: davidben <davidben@chromium.org>
Date: Tue Oct 11 00:29:06 2016

Use new BoringSSL scopers in //components.

In doing so, switch some heap-allocated things to stack-allocated and
avoid needless bouncing on refcounts with EVP_PKEY_get1_*.

BUG= 654143 

Review-Url: https://codereview.chromium.org/2407633002
Cr-Commit-Position: refs/heads/master@{#424303}

[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/gcm_driver/crypto/p256_key_util.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/aes_cbc.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/aes_ctr.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/aes_gcm.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/aes_kw.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/asymmetric_key_util.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/asymmetric_key_util.h
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/ec.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/ecdh.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/ecdsa.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/pbkdf2.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/rsa.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/rsa_oaep.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/rsa_sign.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/sha.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/algorithms/util.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/blink_key_handle.cc
[modify] https://crrev.com/7e7722810efc072552797165546f7f77f245c396/components/webcrypto/blink_key_handle.h

Status: Fixed (was: Assigned)

Sign in to add a comment