realloc引起的分割错误？

Question

嘿，我试图解决这个学校练习..

编写一个程序，不断读取字符串并连接它们（将它们添加到单个字符串）。串联应该在函数中发生，如果成功则返回1，否则返回0。内存分配只使用realloc！

我在调试程序时没有收到任何错误，但是当我尝试运行程序时，在插入字符串后，出现的唯一内容是“Segmentation Fault”，它会是什么？这是代码：

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

int cat(char **, char *); 

int main(void) 
{ 
    char string[51]; 
    char *output=NULL; 
    char choice; 
    do 
    { 
    printf("Please enter a string [<50 chars]: "); 
    fgets(string,50,stdin); 
    if(string[strlen(string)-1]=='\n') /* if newline was read as well */ 
     string[strlen(string)-1]=0;  /* discard it */ 
    if(cat(&output,string)) 
     printf("\n\nThe string now contains:\n%s\n",output); 
    else 
    { 
     printf("error: memory (re-)allocation failed!\n\n"); 
     return 1; /* exit with error */ 
    } 
    printf("Continue? (y/n) - "); 
    fgets(string,3,stdin); /* read input from keyboard - leave a safety buffer to account for read newline */ 
    choice=string[0]; /* use the first character from the previous read as the choice */ 
    } while(choice=='y' || choice=='Y'); 

    free(output); 
    return 0; 
} 

int cat(char **dest, char *src) 
{ 

    int i; 
    int length1=strlen(src); 
    int length2=strlen(*dest); 
    int length3=length1+length2; 
    *dest=(char*)realloc(NULL,sizeof(*src)); 
    printf("%p", *dest); 
    if(*dest==NULL) return 0; /* if allocation failed */ 
    for(i=0;i<=length3;i++) 
    { 
     if(i<=length1) 
     (*dest)[i]=(*dest)[i]; 
     else 
     (*dest)[i]=(src)[i]; 
    } 
    free(src); 
    return 1; 
} 

Answer 1

至少有5个问题与您的代码：

1）你应该只free你自己分配在堆上。不要free(src)，因为你通过src指向堆栈内存（char string[51];自动释放）。

2）你可能打算重新分配dest，3）你打算分配内存大小为length3（+1 null-terminator）。当*dest是NULL最初

*dest=(char*)realloc(*dest, length3 + 1); 

4）strlen(*dest)会崩溃。

int length2=(*dest)?strlen(*dest):0; 

5）我不认为你的for-loop是正确的。它不会连接字符串，您的偏移量计算关闭。

Answer 2

指针output的初始值为NULL。但是在函数内部没有检查指针是否等于NULL。因此，将函数strlen应用于指针会导致未定义的行为。

此外，您还需要为终止零点保留一个字符。

内存在函数中没有正确重新分配。而且sizeof(*src)等于一个字节。

本声明

if(i<=length1) 
    (*dest)[i]=(*dest)[i]; 

没有任何重大的意义。重新分配的内存提供它已正确重新分配已包含原始字符串。

您不应该释放指针src，因为它不指向动态分配的内存。

该功能可以按照演示程序中显示的以下方式进行查看。

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

int cat(char **dest, const char *src) 
{ 
    size_t n = strlen(src) + (*dest == NULL ? 0 : strlen(*dest)); 

    char *tmp = realloc(*dest, n + 1); 
    int success = tmp != NULL; 

    if (success) 
    { 
     if (*dest == NULL) *tmp = '\0'; 
     *dest = tmp; 

     while (*tmp) ++tmp; 

     while ((*tmp++ = *src++)); 
    }  

    return success; 
} 

#define N 50 

int main(void) 
{ 
    char *output = NULL; 
    char choice = 'n'; 

    do 
    { 
     char string[N]; 


     printf("Please enter a string [<%d chars]: ", N); 
     fgets(string, sizeof(string),stdin); 

     string[strcspn(string, "\n")] = '\0'; 

     if (cat(&output, string)) 
     { 
      printf("\nThe string now contains:\n\"%s\"\n\n", output); 
     }   
     else 
     { 
      printf("error: memory (re-)allocation failed!\n\n"); 
      return 1; /* exit with error */ 
     } 

     printf("Continue? (y/n) - "); 
     fgets(string, 3, stdin); /* read input from keyboard - leave a safety buffer to account for read newline */ 
     choice = string[0]; /* use the first character from the previous read as the choice */ 
    } while (choice == 'y' || choice == 'Y'); 

    free(output); 

    return 0; 
} 

它的输出可能看起来像

Please enter a string [<50 chars]: Hi Stefano Feltre 

The string now contains: 
"Hi Stefano Feltre" 

Continue? (y/n) - y 
Please enter a string [<50 chars]: 

The string now contains: 
"Hi Stefano Feltre " 

Continue? (y/n) - y 
Please enter a string [<50 chars]: Let's learn C 

The string now contains: 
"Hi Stefano Feltre Let's learn C" 

Continue? (y/n) - n