Fix strncpy warning by making it more obviously dangerous

The function as a whole is boundschecked after the fact, but as its
comments state, this function really needs its API changed to carry
buffer lengths in from the outside.

sc2/src/uqm/commglue.c

@@ -284,7 +284,7 @@ construct_response (UNICODE *buf, int R /* promoted from RESPONSE_REF */, ...)
 		if (name)
 		{
 			len = strlen (name);
-			strncpy (buf, name, len);
+			strcpy (buf, name);
 			buf += len;
 			
 			/*