crypto: comment change on disabling compression
authorAhamed Nafeez <ahamed.nafeez@gmail.com>
Sat, 14 Dec 2013 22:47:00 +0000 (04:17 +0530)
committerFedor Indutny <fedor.indutny@gmail.com>
Sun, 15 Dec 2013 10:24:51 +0000 (14:24 +0400)
This is a comment change, where it originally says disabling TLS
Compression protects against BEAST attack. But in fact, it is the
CRIME attack(Compression Ratio Info-leak Made Easy) that makes use
of TLS Compression and not BEAST.

BEAST(Browser Exploit Against SSL/TLS) is an entirely another variant
making use of the chosen boundary attack against CBC mode in
encryption.

Just making sure, that the exact reason for disabling TLS compression
must be made clear and not be misleading with some other attack.

src/node_crypto.cc

index d146752..ebd935a 100644 (file)
@@ -4060,7 +4060,7 @@ void InitCryptoOnce() {
   CRYPTO_set_locking_callback(crypto_lock_cb);
   CRYPTO_THREADID_set_callback(crypto_threadid_cb);
 
-  // Turn off compression. Saves memory and protects against BEAST attacks.
+  // Turn off compression. Saves memory and protects against CRIME attacks.
 #if !defined(OPENSSL_NO_COMP)
 #if OPENSSL_VERSION_NUMBER < 0x00908000L
   STACK_OF(SSL_COMP)* comp_methods = SSL_COMP_get_compression_method();