Обнаружен glibc free (): неверный следующий размер (быстро) - PullRequest
Новая
       19

Обнаружен glibc free (): неверный следующий размер (быстро)

5 голосов
/ 31 января 2012

Этот код генерирует случайные числа, а затем выдает гистограмму на основе ввода в функции относительно интервалов.«bin» представляет интервалы гистограммы, а «bin_counts» содержит число случайных чисел в данном интервале.

Я просмотрел несколько постов, посвященных аналогичным проблемам, и я понимаю, что я вышел за пределыгде-то в памяти, но GBD указывает мне только на «free (bins)»;в конце кода.Я дважды проверил длину моего массива, и я думаю, что все они правильны с точки зрения отсутствия доступа к элементам, которые не существуют / запись в память не выделена.Странно то, что код работает так, как задумано, он генерирует точную гистограмму, теперь мне просто нужно помочь устранить эту недопустимую ошибку следующего размера free ().Если у кого-то есть какие-либо предложения, я был бы очень благодарен.Весь вывод:

обнаружен glibc ./file: free (): недопустимый следующий размер (быстрый): 0x8429008

, за которым следует несколько адресов в памяти,отделены Backtrace и картой памяти.Backtrace только указывает мне на линию 129, которая является «свободной (мусорные ведра);».Заранее спасибо 

    #include "stdio.h"
    #include "string.h"
    #include "stdlib.h"

    void histo(int N, double m, double M, int nbins, int *bin_counts, double *bins);

     int main(int argc, char* argv[])
     {

     int *ptr_bin_counts;
     double *ptr_bins; 

     histo(5,0.0,11.0,4, ptr_bin_counts, ptr_bins);

     return 0;
     } 

     void histo(int N, double m, double M, int nbins, int *bin_counts, double *bins)
     {

     srand(time(NULL));
     int i,j,k,x,y;
     double interval;
     int randoms[N-1];
     int temp_M = (int)M;
     int temp_m = (int)m;
     interval = (M-m) /((double)nbins);


     //allocating mem to arrays
     bins =(double*)malloc(nbins * sizeof(double));
     bin_counts =(int*)malloc((nbins-1) * sizeof(int));

     //create bins from intervals
     for(j=0; j<=(nbins); j++)
     {
            bins[j] = m + (j*interval); 
     } 

      //generate "bin_counts[]" with all 0's
      for(y=0; y<=(nbins-1); y++)
       {
         bin_counts[y] = 0; 
       }


      //Generate "N" random numbers in "randoms[]" array
      for(k =0; k<=(N-1); k++)
      {
          randoms[k] = rand() % (temp_M + temp_m);
          printf("The random number is %d \n", randoms[k]);
      }

       //histogram code 
       for(i=0; i<=(N-1); i++)
        {
         for(x=0; x<=(nbins-1); x++)
         {
              if( (double)randoms[i]<=bins[x+1] && (double)randoms[i]>=bins[x] )
               {
                    bin_counts[x] = bin_counts[x] + 1; 
               }
         }
         }
         free(bins);
         free(bin_counts);
         }

1 Ответ

10 голосов
/ 31 января 2012 
bins =(double*)malloc(nbins * sizeof(double));
bin_counts =(int*)malloc((nbins-1) * sizeof(int));

//create bins from intervals
for(j=0; j<=(nbins); j++)
{
    bins[j] = m + (j*interval); 
} 

//generate "bin_counts[]" with all 0's
for(y=0; y<=(nbins-1); y++)
{
    bin_counts[y] = 0; 
}

Вы переступаете через свои массивы, вы выделяете место для nbins удваиваний, но записываете в nbins+1 местоположения и используете nbins местоположения для bin_counts, но выделите только nbins-1.

...