C: 在 0x0FFB0BA0 (ucrtbased.dll) 处抛出异常

Question

我正在开发一个程序，要求用户输入一个单词以及用户想要从该单词中复制的字母数。该程序在我使用 Compile Online 时有效，但是当我在 Micrsoft Visual Studio 中运行该程序时，在我输入要复制的单词后程序冻结。我运行调试器，发现如下所示的错误。我认为，from googling，我正在写超过为我的数组预留的内存量？我应该使用 malloc 来解决这个问题吗？下面贴出错误和代码（链接到原始stackoverflow thread。

在 lab1113problem7.exe 中的 0x0FFB0BA0 (ucrtbased.dll) 处引发异常：0xC0000005：访问冲突写入位置 0x00B80000。

lab1113problem7.exe 中 0xFEFEFEFE 处的未处理异常：0xC00001A5：检测到无效的异常处理程序例程（参数：0x00000003）。

程序“[7196] lab1113problem7.exe”已退出，代码为 0 (0x0)。

程序：

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

char *copywords(char *dest, const char *source, size_t n);

int main(void) {
    char words[50];
    char newwords[50];
    int num;

    for (;;) {
        printf("Type a word, or type 'quit' to quit: ");
        if (scanf("%49s", words) != 1) {
            printf("Invalid input!\n");
            return 0; 
        }
        if (!strcmp(words, "quit")) {
            printf("Good bye!\n");
            return 0; 
        }
        printf("Type the # of chars to copy: ");
         if (scanf("%d", &num) != 1) {
            printf("Invalid input!\n");
            return 0; 
        }
        copywords(newwords, words, num);
        printf("The word was %s\n", words);
        printf("and the copied word is %s\n", newwords);
    }
}

char *copywords(char *dest, const char *source, size_t n) {
    size_t i;
    for (i = 0; i < n && source[i] != '\0'; i++) {
        dest[i] = source[i];
    }
    dest[i] = '\0';
    return dest;
}

Answer 1

一些调试代码以确保您的字符串按预期工作：

// Place this at the top of your file, after your `#include` statements 
// and use it in place of the 50s in your array defines.
#define MAX_ARRAY_SIZE 50

char *copywords(char *dest, const char *source, size_t n) {
    size_t i;
    for (i = 0; i < n && source[i] != '\0'; i++) {
        if(i >= MAX_ARRAY_SIZE)
        {
            printf("I just blew up memory at %i\n", %i);
        }
        dest[i] = source[i];
    }
    dest[i] = '\0';
    return dest;
}

注意：您可以在 for 循环中检查每次迭代的数组大小，以确保您没有超出缓冲区。

不是直接的答案，而是比使用 cmets 更容易尝试的方法。如果 printf 被触发，你就知道发生了不好的事情......

更好的是，因为您将大小作为n 传递，您可以在每次循环开始之前检查数组大小并立即处理错误数据。

char *copywords(char *dest, const char *source, size_t n) {
    if(n >= MAX_ARRAY_SIZE) 
    {
        printf("Array will overflow by %d bytes- truncating\n", n - MAX_ARRAY_SIZE);
        n = MAX_ARRAY_SIZE;
    }