Skip to content

Commit

Permalink
man: add security considerations to --compress section
Browse files Browse the repository at this point in the history
As Ahamed Nafeez reported to the OpenVPN security team, we did not
sufficiently inform our users about the risks of combining encryption
and compression.  This patch adds a "Security Considerations" paragraph
to the --compress section of the manpage to point the risks out to our
users.

Signed-off-by: Steffan Karger <steffan@karger.me>
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <1528020718-12721-1-git-send-email-steffan@karger.me>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg16919.html
Signed-off-by: Gert Doering <gert@greenie.muc.de>
  • Loading branch information
syzzer authored and cron2 committed Jun 3, 2018
1 parent 1394192 commit a59fd14
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions doc/openvpn.8
Original file line number Diff line number Diff line change
Expand Up @@ -2516,6 +2516,16 @@ If the
parameter is empty, compression will be turned off, but the packet
framing for compression will still be enabled, allowing a different
setting to be pushed later.

.B Security Considerations

Compression and encryption is a tricky combination. If an attacker knows or is
able to control (parts of) the plaintext of packets that contain secrets, the
attacker might be able to extract the secret if compression is enabled. See
e.g. the CRIME and BREACH attacks on TLS which also leverage compression to
break encryption. If you are not entirely sure that the above does not apply
to your traffic, you are advised to *not* enable compression.

.\"*********************************************************
.TP
.B \-\-comp\-lzo [mode]
Expand Down

0 comments on commit a59fd14

Please sign in to comment.