C - 附加到字符串 - 可能的内存错误

Question

我正在尝试创建一个函数，该函数不断将字符串附加到 char 变量。但是，有时它有效，有时则无效。我想知道错误在哪里？

char *final_output = NULL;
void add_string(const char *);

int main(void) {
    add_string("Hello world\n");
    add_string("This is my new function!\n");

    /* Let's print */
    while (final_output && *final_output) {
        printf("%c", *final_output);
        *final_output++;
    }
}

void add_string(const char *text) {
    if (final_output == NULL) {
        final_output = malloc(strlen(text) + 1);
    }
    else {
        final_output = (char *) realloc(final_output, strlen(final_output) + strlen(text) + 2);
    }

    strncat(final_output, text, strlen(text));
}

Answer 1

问题出在函数 add_string 中。您不要在语句之后附加带有终止零的分配或复制的数组

final_output = malloc(strlen(text) + 1);

和

strncat(final_output, text, strlen(text));

按如下方式重写函数

void add_string( const char *s ) 
{
    if ( final_output == NULL ) 
    {
        final_output = malloc( strlen( s ) + 1 );
        final_output[0] = '\0';
    }
    else 
    {
        final_output = realloc( final_output, strlen( final_output ) + strlen( s ) + 1 );
    }

    strcat( final_output, s );
}