diff options
author | Antonio Quartulli | 2019-08-05 11:25:24 +0200 |
---|---|---|
committer | Gert Doering | 2019-08-15 20:21:57 +0200 |
commit | 8a05f860af386cd135ee1aacb0d5eccc49c466bb (patch) | |
tree | e20241fc0aa1bc9fa00f971fec53814a7c83febd /src/openvpn/socket.c | |
parent | 26e5b80096a78ec7d5e7bcd15f81155be5fde2ec (diff) | |
download | openvpn-8a05f860af386cd135ee1aacb0d5eccc49c466bb.zip openvpn-8a05f860af386cd135ee1aacb0d5eccc49c466bb.tar.gz |
sitnl: harden strncpy() by forcing arguments to have the same length
At the moment a strcpy() (without length check!) is performed between a
string long IFNAMSIZ bytes and one of 16 bytes. This is ok right now
because IFNAMSIZ is defined as 16, however this bit is not under our
control and may change in he future without us being warned.
For this reason, force both strings to use IFNAMSIZ as size and, since
this constant may not exist on every platform, ensure it is always
defined.
Signed-off-by: Antonio Quartulli <antonio@openvpn.net>
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <20190805092529.9467-2-a@unstable.cc>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg18722.html
Signed-off-by: Gert Doering <gert@greenie.muc.de>
Diffstat (limited to 'src/openvpn/socket.c')
0 files changed, 0 insertions, 0 deletions