为什么 strcpy 的这种使用被认为是不好的？

Question

我发现了以下一段C代码，标记为 BAD（又名缓冲区溢出错误）。问题是我不太明白为什么？输入字符串长度在分配等之前被捕获。

char *my_strdup(const char *s)
{
    size_t len = strlen(s) + 1;
    char *c = malloc(len);
    if (c) {
        strcpy(c, s);  // BAD
    }
    return c;
}

评论更新：

• “BAD”标记不精确，代码还不错，效率不高是的，有风险（如下）是的，
• 为什么有风险？+1在 strlen() 调用之后需要安全地分配堆上的空间，该空间也将保留字符串终止符 ( '\0')

Answer 1

您的示例函数中没有错误。

但是，为了让未来的读者（无论是人类还是机械）清楚地知道不存在错误，您应该将调用替换strcpy为memcpy：

char *my_strdup(const char *s)
{
    size_t len = strlen(s) + 1;
    char *c = malloc(len);
    if (c) {
        memcpy(c, s, len);
    }
    return c;
}

无论哪种方式，len都会分配字节并len复制字节，但memcpy这一事实对读者来说更加清晰。