Log how often DTLS negotiation failed because of incompatible ciphersuites.

webrtc/base/opensslstreamadapter.cc

 /*
  *  Copyright 2004 The WebRTC Project Authors. All rights reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 836 matching lines @@
       }
       break;
 
     case SSL_ERROR_WANT_WRITE:
       LOG(LS_VERBOSE) << " -- error want write";
       break;
 
     case SSL_ERROR_ZERO_RETURN:
     default:
       LOG(LS_VERBOSE) << " -- error " << code;
+      int err_code = ERR_peek_last_error();
+      int err_reason = 0;

[Zhi Huang, 2016/07/27 21:04:14]

I can leave a comment, like "Use 0 to represent unknown error.", here before
submitting.

+      if (err_code != 0 && ERR_GET_REASON(err_code) == SSL_R_NO_SHARED_CIPHER) {
+        err_reason = SSL_R_NO_SHARED_CIPHER;
+      }
+      SignalSSLHandshakeError(err_reason);

[honghaiz3, 2016/07/29 15:49:54]

Should this just be 
int err_reason = ERR_GET_REASON(err_code);
SignalSSLHandshakeError(err_reason);  
?

[honghaiz3, 2016/08/01 17:19:19]

If you only want to log the error SSL_R_NO_SHARED_CIPHER, that is fine. 

       return (ssl_error != 0) ? ssl_error : -1;
   }
 
   return 0;
 }
 
 void OpenSSLStreamAdapter::Error(const char* context, int err, bool signal) {
   LOG(LS_WARNING) << "OpenSSLStreamAdapter::Error("
                   << context << ", " << err << ")";
   state_ = SSL_ERROR;
@@ skipping 352 matching lines @@
         return true;
     }
   }
 
   return false;
 }
 
 }  // namespace rtc
 
 #endif  // HAVE_OPENSSL_SSL_H