laforge submitted this change.

View Change

Approvals: laforge: Looks good to me, approved; Verified dexter: Looks good to me, approved
ensure zero termination of tun device name

Change-Id: I4b783c6b98ddfb30a4ca4532ef8cffa8e5f3a12f
---
M src/tuntap.c
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/tuntap.c b/src/tuntap.c
index 8109067..6ae16c7 100644
--- a/src/tuntap.c
+++ b/src/tuntap.c
@@ -29,7 +29,7 @@
*/
ifr.ifr_flags = IFF_TUN|IFF_NO_PI;
if( *dev )
- strncpy(ifr.ifr_name, dev, IFNAMSIZ);
+ snprintf(ifr.ifr_name, IFNAMSIZ, "%s", dev);

if( (err = ioctl(fd, TUNSETIFF, (void *) &ifr)) < 0 ){
close(fd);

To view, visit change 28591. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-tetra
Gerrit-Branch: master
Gerrit-Change-Id: I4b783c6b98ddfb30a4ca4532ef8cffa8e5f3a12f
Gerrit-Change-Number: 28591
Gerrit-PatchSet: 1
Gerrit-Owner: wbokslag <w.bokslag@midnightblue.nl>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged