Skip to contentSkip to conversation

Update cipher-related man page text

PATCH In review Steffan Karger 2 participants 2 messages Sep 11, 2016 – Oct 4, 2016

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Update cipher-related man page text

As reported in trac #732, the man page text for --cipher is no longer accurate. Update the text to represent current knowledge, about NCP and SWEET32.

This does not hint at changing the default cipher, because we did not make a decision on that yet. If we do change the default cipher, we'll have to update the text to reflect that.

---
 doc/openvpn.8 | 36 +++++++++++++++++++++++++-----------
 1 file changed, 25 insertions(+), 11 deletions(-)
doc/openvpn.8 +25 -11
index 163bdf4..f86851c 100644
@@ -4110,25 +4110,26 @@ Encrypt data channel packets with cipher algorithm
 The default is
 .B BF-CBC,
 an abbreviation for Blowfish in Cipher Block Chaining mode.
-Blowfish has the advantages of being fast, very secure, and allowing key sizes
-of up to 448 bits.  Blowfish is designed to be used in situations where
-keys are changed infrequently.
 
-For more information on blowfish, see
-.I http://www.counterpane.com/blowfish.html
+Using BF-CBC is no longer recommended, because of it's 64-bit block size.  This
+small block size allows attacks based on collisions, as demonstrated by SWEET32.
 
-To see other ciphers that are available with
-OpenVPN, use the
+To see other ciphers that are available with OpenVPN, use the
 .B \-\-show\-ciphers
 option.
 
-OpenVPN supports the CBC, CFB, and OFB cipher modes,
-however CBC is recommended and CFB and OFB should
-be considered advanced modes.
-
 Set
 .B alg=none
 to disable encryption.
+
+As of OpenVPN 2.4, cipher negotiation (NCP) can override the cipher specified by
+.B \-\-cipher\fR.
+See
+.B \-\-ncp-ciphers
+and
+.B \-\-ncp-disable
+for more on NCP.
+
 .\"*********************************************************
 .TP
 .B \-\-ncp\-ciphers cipher_list
@@ -4141,6 +4142,19 @@ is a colon-separated list of ciphers, and defaults to
 For servers, the first cipher from
 .B cipher_list
 will be pushed to clients that support cipher negotiation.
+
+Cipher negotiation is enabled in client-server mode only.  I.e. if
+.B \-\-mode
+is set to 'server' (server-side, implied by setting
+.B \-\-server
+), or if
+.B \-\-pull
+is specified (client-side, implied by setting \-\-client).
+
+If both peers support and do not disable NCP, the negotiated cipher will
+override the cipher specified by
+.B \-\-cipher\fR.
+
 .\"*********************************************************
 .TP
 .B \-\-ncp\-disable
Signature · 1 line
2.7.4
#1 lore raw

ACK.

Your patch has been applied to the master branch.

commit 5a1daf533ae283e258732260c96461e820e61fe6

Author: Steffan Karger
Date:   Sun Sep 11 16:50:31 2016 +0200
     Update cipher-related man page text

     Signed-off-by: Steffan Karger <[email protected]...>
     Acked-by: Gert Doering <[email protected]...>
     Message-Id: <[email protected]...>
     URL: https://www.mail-archive.com/[email protected]/msg12439.html
     Signed-off-by: Gert Doering <[email protected]...>

--
kind regards,

Gert Doering