c - c中的这个缓冲区溢出解决方案有什么问题？

Question

我只是因为这个 c 中缓冲区溢出的错误解决方案而在类(class)作业上被扣分，但没有得到关于它是如何错误的反馈。有人可以让我知道问题是什么吗？谢谢。

如果将超过 16 的字符串传递给此函数，该问题将提供解决方案:

void function(char *str)
{
    char buffer[16];
    strcpy(buffer, str);
}

这是我的解决方案

void function(char *str)
{
    size_t str_length = strlen(str);

    char buffer[str_length];
    strcpy(buffer, str);
}

谢谢

Answer 1

您需要考虑终止字符串的空字符:

char buffer[str_length + 1];

void_ptr 指出以上还不够。所以要更健壮:

void function(char *str)
{
    size_t str_length = strlen(str);
    char *buffer = malloc(str_length + 1);
    if (buffer == NULL)
        return;
    memcpy(buffer, str, str_length+1); // Thanks chux
    // Do something with with buffer...
    free(buffer);
}

或者也许教授只是在寻找这个:

void function(char *str)
{
    char buffer[16];
    strncpy(buffer, str, sizeof(buffer));
    buffer[sizeof(buffer)-1] = '\0';
}