Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(615)

Unified Diff: components/webcrypto/algorithms/rsa_oaep.cc

Issue 1634383002: Change calling style for some Boring SSL functions which (now) return just 0 or 1. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/webcrypto/algorithms/rsa_oaep.cc
diff --git a/components/webcrypto/algorithms/rsa_oaep.cc b/components/webcrypto/algorithms/rsa_oaep.cc
index 27e834bce71b1e642736581124b73371f0633479..667a3cd66d784d4aa274d6825b1f4e4e6a3d9d91 100644
--- a/components/webcrypto/algorithms/rsa_oaep.cc
+++ b/components/webcrypto/algorithms/rsa_oaep.cc
@@ -51,9 +51,9 @@ Status CommonEncryptDecrypt(InitFunc init_func,
crypto::ScopedEVP_PKEY_CTX ctx(EVP_PKEY_CTX_new(pkey, NULL));
if (!init_func(ctx.get()) ||
- 1 != EVP_PKEY_CTX_set_rsa_padding(ctx.get(), RSA_PKCS1_OAEP_PADDING) ||
- 1 != EVP_PKEY_CTX_set_rsa_oaep_md(ctx.get(), digest) ||
- 1 != EVP_PKEY_CTX_set_rsa_mgf1_md(ctx.get(), digest)) {
+ !EVP_PKEY_CTX_set_rsa_padding(ctx.get(), RSA_PKCS1_OAEP_PADDING) ||
+ !EVP_PKEY_CTX_set_rsa_oaep_md(ctx.get(), digest) ||
+ !EVP_PKEY_CTX_set_rsa_mgf1_md(ctx.get(), digest)) {
return Status::OperationError();
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698