strncpy -> strlcpy

quoting strncpy(3)

     strncpy() only NUL terminates the destination string when the
     length of the source string is less than the length parameter.

strlcpy is more intuitive.

this is another warning gcc 8 found that clang didn't.

1 files changed, 2 insertions, 2 deletions

diff --git a/server.c b/server.c
index 5bc0ff5..d07a642 100644
--- a/server.c
+++ b/server.c
@@ -492,9 +492,9 @@ found:
 
 err:
 	if (servname != NULL)
-		strncpy(c->req, servname, sizeof(c->req));
+		strlcpy(c->req, servname, sizeof(c->req));
 	else
-		strncpy(c->req, "null", sizeof(c->req));
+		strlcpy(c->req, "null", sizeof(c->req));
 
 	start_reply(c, BAD_REQUEST, "Wrong/malformed host or missing SNI");
 }