]> git.ipfire.org Git - thirdparty/openvpn.git/commit
Fix '--cipher none --cipher' crash
authorSteffan Karger <steffan.karger@fox-it.com>
Tue, 26 Jul 2016 13:55:38 +0000 (15:55 +0200)
committerGert Doering <gert@greenie.muc.de>
Tue, 26 Jul 2016 15:13:12 +0000 (17:13 +0200)
commitdea8917a032e2303b39cce23e60f2348516d5a12
tree274d05742be2d84ebba8cb917393e49eb0f3e173
parent834f602fd069118b5d00a9042c9fdb20930257eb
Fix '--cipher none --cipher' crash

As reported in trac #699, OpenVPN crashes when an "--cipher none" option
is followed by "--cipher" (without arguments).  Fix this by removing the
redudant ciphername_defined and authname_defined members of struct options,
and remove support to specify --cipher or --auth without an argument.  That
not only fixes the issue, but also cleans up the code a bit.

v2: don't print a deprecating warning (we'll do that in the 2.3 branch),
    but just rip out support for --cipher and --auth without an argument.

Signed-off-by: Steffan Karger <steffan@karger.me>
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <1469541338-1530-1-git-send-email-steffan.karger@fox-it.com>
URL: http://article.gmane.org/gmane.network.openvpn.devel/12106
Signed-off-by: Gert Doering <gert@greenie.muc.de>
src/openvpn/crypto.c
src/openvpn/crypto.h
src/openvpn/init.c
src/openvpn/options.c
src/openvpn/options.h
src/openvpn/ssl.c