C:SEGFAULT与char **上的realloc

时间:2013-11-05 06:42:27

标签: c segmentation-fault realloc

此代码中我的一系列问题中的另一个问题。我有以下函数,它将arg与字符串数组reference中的每个字符串进行比较:

char compare(char *arg)
{
        int iter=0;
        char retchar='0';

        while(iter < no_of_ref)
        {
        //      printf("arg : %s , reference : %s \n",arg,reference[iter]);
                if((strstr(reference[iter],arg) != NULL) || (strstr(arg,reference[iter]) != NULL))
                {
                        retchar='1';
                        break;
                }
          iter++;
        }
return retchar;
}

reference是全局char **,但在main中动态构建如下:

reference = calloc(CHUNK, sizeof(char *));

然后是一些代码:

                        temp_in[pre_pip+1]='\0';
                        reference[no_of_ref]=malloc(strlen(temp_in) + 1);
                        strcpy(reference[no_of_ref++],temp_in);
                        memset(&temp_in,'\0',sizeof(temp_in));
                        pre_pip = -1;
   printf("INDEX: %d, address : %p , val : %s\n",no_of_ref-1,reference[no_of_ref-1],reference[no_of_ref-1]);      //DEBUG
                }
                /*If allocated buffer is at brim, extend it for CHUNK char *  further*/
                if(no_of_ref == (tr*CHUNK - 2))
                {
                        current_size = tr*CHUNK*sizeof(char *);

                        char *retalloc = realloc(reference,current_size + CHUNK*sizeof(char *));
                                if(retalloc == NULL)
                                        perror("ERROR on realloc");
                                else
                                {
                                        printf("Realloced successfully: %p\n",retalloc);
                                        tr++;
                                }

对于不需要realloc的测试用例运行良好的代码,即输入字符串数小于CHUNK。在realloc的情况下,我从函数中获得SEGFAULT。以下是其中一个:

Program terminated with signal 11, Segmentation fault.
#0  __strstr_sse42 (s1=0x3839393433333230 <Address 0x3839393433333230 out of bounds>, s2=0x6020c0 <cmp> "8956549122") 

2 个答案:

答案 0 :(得分:1)

您需要在realloc()中将括号括起来作为

//---------------------------------v -------------------v
char *retalloc = realloc(reference,(current_size + CHUNK)*sizeof(char *));

假设CHUNK=100current_size=200,您的代码将在您想要200+100*8=1000 bytes

时分配(200+100)*8 = 2400 bytes

另外,请确保在重新分配后将retalloc分配给reference变量。

答案 1 :(得分:1)

realloc实际重新分配您传递给它的内存时,您作为参数传递的指针仍然指向内存区域。 realloc函数返回指向 new 内存的指针,因此您必须将其分配给例如 new 内存。 reference