Commit f9ce26c5 authored by Kees Cook's avatar Kees Cook Committed by Jonathan Corbet
Browse files

docs: networking: Replace strncpy() with strscpy() 

Replace example code's use of strncpy() with strscpy() functions. Using
strncpy() is considered deprecated:
https://www.kernel.org/doc/html/latest/process/deprecated.html#strncpy-on-nul-terminated-strings



Signed-off-by: default avatarKees Cook <keescook@chromium.org>
Acked-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
Link: https://lore.kernel.org/r/20210602202914.4079123-1-keescook@chromium.org


Signed-off-by: default avatarJonathan Corbet <corbet@lwn.net>
parent 709dedfd

Documentation/input/joydev/joystick-api.rst

+1 −1
Original line number Diff line number Diff line
@@ -263,7 +263,7 @@ possible overrun should the name be too long:: 


	char name[128];

	if (ioctl(fd, JSIOCGNAME(sizeof(name)), name) < 0)

		strncpy(name, "Unknown", sizeof(name));

		strscpy(name, "Unknown", sizeof(name));

	printf("Name: %s\n", name);

Documentation/networking/packet_mmap.rst

+1 −1
Original line number Diff line number Diff line
@@ -153,7 +153,7 @@ As capture, each frame contains two parts:: 
    struct ifreq s_ifr;

    ...



    strncpy (s_ifr.ifr_name, "eth0", sizeof(s_ifr.ifr_name));

    strscpy_pad (s_ifr.ifr_name, "eth0", sizeof(s_ifr.ifr_name));



    /* get interface index of eth0 */

    ioctl(this->socket, SIOCGIFINDEX, &s_ifr);

Documentation/networking/tuntap.rst

+1 −1
Original line number Diff line number Diff line
@@ -107,7 +107,7 @@ Note that the character pointer becomes overwritten with the real device name 
       */

      ifr.ifr_flags = IFF_TUN;

      if( *dev )

	 strncpy(ifr.ifr_name, dev, IFNAMSIZ);

	 strscpy_pad(ifr.ifr_name, dev, IFNAMSIZ);



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

	 close(fd);