Question

我想在GMP中获取一个大整数的二进制表示。我将1和0存储在一个名为 expBinary 的数组中。我使用malloc来分配大小为“int”的内存，然后在添加新位时使用 realloc 来增加此内存。转换运行完美没有任何问题，但是当我尝试在while循环后使用malloc分配更多内存时，当我第二次调用相同的代码时，它给了我分段错误 ，第一次它没有给我分段错误。我已经检查过并且“expBinary”没有存储任何超出范围的内容，我已经给出了下面的代码

int binarySize = 0;                                                        
int * expBinary = malloc(sizeof(int));                                 
int i = 0;                                                                  

// Run until exp == 0                                                       
while(mpz_cmp_ui(exp,0) != 0)                                               
{                                                                           
    binarySize++;                                                           
    expBinary = (int*) realloc(expBinary,(binarySize));                     
    // Getting LSB of exp                                                   
    if(mpz_even_p(exp) != 0)                                                
        expBinary[i] = 0;                                                   
    else                                                                    
        expBinary[i] = 1;                                                                                                                                 
    // Incrmenting variables                                                
    i++;                                                                    
    // Dividing exponent by 2                                               
    mpz_tdiv_q_ui(exp,exp,2);                                               
}                                                                           

// This line is giving error
int * temp = malloc(sizeof(int));

Answer 1

如果您使用的是int数组，那么这是错误的

expBinary = (int*) realloc(expBinary,(binarySize));

应该是

expBinary = realloc(expBinary, binarySize * sizeof(*expBinary));

或等效，

expBinary = realloc(expBinary, binarySize * sizeof(int));

出于显而易见的原因，我更喜欢sizeof(*expBinary)，而且，如果realloc()失败，则会忽略对前一指针的引用，所以我建议这样做

void *tmp;
tmp = realloc(expBinary, binarySize * sizeof(int));
if (tmp == NULL)
    handleFailureHereAndDontContinueToTheNextLineAndFree_expBinary_Please();
expBinary = tmp;

现在，如果您想使用任何printf("%s\n", expBinary);打印表示，则应使用char *代替，在这种情况下，您应该始终考虑sizeof(char) == 1，并且您将需要一个额外的字节在1和0＆＃39}的末尾，其值为'\0'。

C中的内存分配错误

1 个答案: