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

Unified Diff: runtime/bin/secure_socket_boringssl.cc

Issue 2683703004: [dart:io] A small boringssl cleanup from davidben@ (Closed)
Patch Set: Created 3 years, 10 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: runtime/bin/secure_socket_boringssl.cc
diff --git a/runtime/bin/secure_socket_boringssl.cc b/runtime/bin/secure_socket_boringssl.cc
index ff4a37149c6649bbb6a29c5dc52bc3e2a01ad58d..41a6198b590101274a681981058035675eaff906 100644
--- a/runtime/bin/secure_socket_boringssl.cc
+++ b/runtime/bin/secure_socket_boringssl.cc
@@ -91,8 +91,8 @@ static void FetchErrorString(const SSL* ssl, TextBuffer* text_buffer) {
error = ERR_get_error_line(&path, &line);
const char* file = strrchr(path, sep[0]);
path = file ? file + 1 : path;
- if ((ssl != NULL) &&
- (error == ERR_PACK(ERR_R_SSL_LIB, SSL_R_CERTIFICATE_VERIFY_FAILED))) {
+ if ((ssl != NULL) && (ERR_GET_LIB(error) == ERR_LIB_SSL) &&
+ (ERR_GET_REASON(error) == SSL_R_CERTIFICATE_VERIFY_FAILED)) {
intptr_t result = SSL_get_verify_result(ssl);
text_buffer->Printf("\n\t%s: %s (%s:%d)", ERR_reason_error_string(error),
X509_verify_cert_error_string(result), path, line);
« 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