Revert "glapi: s/strcpy/strncpy/"
authorBrian Paul <brianp@vmware.com>
Mon, 3 May 2010 19:16:11 +0000 (13:16 -0600)
committerBrian Paul <brianp@vmware.com>
Mon, 3 May 2010 19:16:11 +0000 (13:16 -0600)
This reverts commit 9446fd8f69564e09ffd0f28735a99c510f84bb62.

It doesn't make sense to replace strcpy(a,b) with strncpy(a,b,strlen(b)).
The preceeding code effectively does bounds checking, btw.

src/mesa/glapi/glapi_getproc.c

index ec96ab3..c73e8dd 100644 (file)
@@ -265,8 +265,7 @@ str_dup(const char *str)
    copy = (char*) malloc(strlen(str) + 1);
    if (!copy)
       return NULL;
-   strncpy(copy, str, strlen(str));
-   copy[strlen(str)] = '\0';
+   strcpy(copy, str);
    return copy;
 }