listen: Fix potential buffer overflow.

Sounds evil but would only be exploitable when getservbyport is accessing
a database that's under an attacker's control.  The /etc/services file
shipping with Fedora 34 has entries of a maximum length of 15 characters
so are just fine.
Fixed by not copying the string returned by getservbyport - which also is
faster.  At that point it becomes possible to reduce the length of str[]
to 6 bytes.

Found by the following GCC 11 warning:

gcc -DHAVE_CONFIG_H -I. -I..     -g -O2 -Wall -MT utils.o -MD -MP -MF .deps/utils.Tpo -c -o utils.o utils.c
utils.c: In function ‘servname’:
utils.c:124:17: warning: ‘strncpy’ specified bound 16 equals destination size [-Wstringop-truncation]
  124 |                 strncpy(str, serv->s_name, 16);
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Signed-off-by: Ralf Baechle <ralf@linux-mips.org>

0 files changed, 0 insertions, 0 deletions