使用 strncpy 复制字符串时遇到异常 [关闭]

Question

我有一个字符串，我正在迭代寻找一个特定的单词，它恰好在两个空格之间。

例如：

// where the word that I'm looking for is /docs/index.html
const char* c = "GET /docs/index.html HTTP/1.1\r\n";

我找到这个词如下；

const char* wbeg = strchr(c, ' ') + 1; // points to '/'
const char* wend = strchr(wbeg, ' ') -1; // points to 'l'

如果我想将该单词存储到另一个位置，我使用 strncpy 实现了这一点

char word[256];
strncpy(word, wbeg, wend - wbeg);

我收到以下错误

在 0x00007FFAAE8C4A74 (ucrtbased.dll) 处引发异常 ConsoleApplication1.exe: 0xC0000005: 访问冲突写入位置 0x0000000000000000.

Answer 1

当您在帖子中展示的要点在一个简单的main() 程序中运行时，...

int main()
{
    const char* c = "GET /docs/index.html HTTP/1.1\r\n";
    const char* wbeg = strchr(c, ' ') + 1; // points to '/'
    const char* wend = strchr(wbeg, ' ') -1; // points to 'l'
    char word[256];
    strncpy(word, wbeg, wend - wbeg);

    printf("%s", word);

    return 0;
}

...在我的环境中没有观察到故障。因此，除了发布其余相关代码之外，唯一的建议都是围绕确保您没有调用 UB。

1) 在您的声明中：

strncpy(word, wbeg, wend - wbeg);

`wend - wbeg` is == 15

而 /docs/index.html 的长度为 16 个字符。
将您的声明更改为：

strncpy(word, wbeg, (wend - wbeg)+1);

2) 从初始化变量开始：

  char word[SIZE] = ""  

3) strncpy 不会 NULL 终止。如果您要复制到的目标在使用前尚未初始化，或者您在使用后没有明确地为 null 终止，则可能会发生 UB。 示例：

char target[];  //contents of target are not guaranteed
char source[]="abcdefghijklmnopqrstuv";
strncpy(target, source, 3);

有可能得到以下结果：

|a|b|c|?|?|?|?|?|?|?|?|?|?|?|?|?|...

在哪里？可以是任何东西。

如果要保证 ASCII NUL 字节位于复制字节的末尾，可以使用以下内容：

strncpy (target, source, 3);
target[3] = 0;
|a|b|c|\0|?|?|?|?|?|?|?|?|?|?|?|?|...

4) 如果复制发生在两个重叠的对象之间，则行为未定义。确保在使用strncpy() 中的结果之前检查strchr() 函数的结果

Answer 2

strncpy 是一个糟糕的函数。如果 source 比 count 参数长，它不会正确终止字符串：

char s[] = "AAAAA";
strncpy(s, "BB", 2);
// s is now "BBAAA", not "BB"

您需要在复制后显式终止字符串。

char word[SIZE];
ptrdiff_t count = wend - wbeg + 1;
if(count < SIZE) {
    memcpy(word, wbeg, count); // might as well use memcpy
    word[count] = '\0';
}
else // handle error