在for循环中重新分配

Question

我正在尝试创建一个函数,它应该将所有参数作为连接的char数组返回.不幸的是,我在使用该函数时遇到了"无效指针"错误.我是C的新手,所以也许我以这种方式使用realloc是错误的.

char* concat(int argc, ...) {
    char* result;

    va_list args;
    va_start(args, argc);

    int i;
    for (i = 0; i < argc; i++) {
        char* s = va_arg(args, char*);
        int length = (result) ? strlen(result) : 1;

        char* tmp = (char*)realloc(result, sizeof(char) * (strlen(s) + length - 1));
        if (tmp == NULL) {
            throw_error("Realloc failed in `concat`.");
        }

        result = tmp;
        memcpy(&(result[length-1]), s, strlen(s));

        printf("result: %s\n", s);
    }

    va_end(args);
    return result;
}

错误消息,如果它可以帮助:

*** glibc detected *** ./pascc: realloc(): invalid pointer: 0x00000000006060c8 ***
======= Backtrace: =========
/lib/x86_64-linux-gnu/libc.so.6(+0x76bb6)[0x7f9953be8bb6]
/lib/x86_64-linux-gnu/libc.so.6(realloc+0x338)[0x7f9953beed58]
./pascc[0x401368]
./pascc[0x400f85]
./pascc[0x4019bb]
./pascc[0x403c49]
/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xff)[0x7f9953b90eff]
./pascc[0x400e29]
======= Memory map: ========
00400000-00406000 r-xp 00000000 08:05 2885504 

为了避免内存泄漏,我在另一个线程中读到我不应该像这样重新分配一些东西:

result = (char*)realloc(result, sizeof(char) * (strlen(s) + length - 1));

所以我试图找到这样做的好方法.一些帮助将不胜感激.

Answer 1

你是realloc一个未初始化的指针.

char* result;

要使用realloc您必须使用NULL指针或指向由指定的内存的指针malloc.

char* result = NULL;