The initial logic for allocating the new string was wrong, the conversion

to strndup(3) duplicated the same mistake, actually strdup(3) is good enough
to allocate the new string.

1 files changed, 2 insertions, 5 deletions

diff --git a/pw/pw_conf.c b/pw/pw_conf.c
index 46474a1..e988f4b 100644
--- a/pw/pw_conf.c
+++ b/pw/pw_conf.c
@@ -213,15 +213,12 @@ char           *
 newstr(char const * p)
 {
 	char	*q;
-	size_t	 l;
 
 	if ((p = unquote(p)) == NULL)
 		return (NULL);
 
-	l = strlen(p) + 1;
-
-	if ((q = strndup(p, l)) == NULL)
-		err(1, "strndup()");
+	if ((q = strdup(p)) == NULL)
+		err(1, "strdup()");
 
 	return (q);
 }