老是提示这个子程序有问题，麻烦大家帮我找找

#include <stdlib.h>
#include <string.h>

char *strdup(char *s)
{
    char *p;

    p = (char *) malloc(strlen(s) + 1);
    if(p != NULL)
        strcpy(p, s);
    return p;
}

原来是这样啊！谢谢