Программа "иногда" вылетает при использовании бесплатного

Question

Моя программа иногда падает, а иногда нет, когда я вызываю free () для элемента из моего массива.Элементы в массиве являются структурой.Я покажу с некоторым кодом:

//This first part might be a bit messy, and hard to understand but it is working
int main(int argc, char *argv[]){
      komplexNumber komplexNumbers[antal-1]; 
      int i;

      float temp;
      int realCounter=0;
      int imaginarCounter=0;
      int numberOfElements=6 // this variable is set by the user using scanf, and can be any number>=2. Each komplexNumber consists of two numbers (i.e 9 -3j is a complex number)

       for(i=0;i<numberOfElements*2;i++){ // 
           printf("Enter number %i\n", i+1);
           scanf("%f", &temp); 
           if(i%2==0){  //a complex number consts of two parts. first entered number is first part of number1, second is second part of number1, third is first part of nr 2 etc
               komplexNumbers[realCounter].real=temp;
               realCounter++;
            }
            else{
              komplexNumbers[imaginarCounter].imaginar=temp;
              imaginarCounter++;
              }
//The struct
typedef struct komplexNumber{ 
  float real;
  float imaginar;
} komplexNumber;




//The method that mallocs memory for each element:
void calculation(float a1, float a2, float b1, float b2, komplexNumber komplexNumbers[]){ 

    float temp1 = (a1*a2)-(b1*b2);     
    float temp2 = (a1*b2)+(a2*b1);

    komplexNumber *k;
    k=(komplexNumber*)malloc(sizeof(komplexNumber));              
    k->real=temp1;
    k->imaginar=temp2;
    komplexNumbers[0]=*k;
}

//The loop, in which im calling free each iteration:

int counter=1;
for(i=0;i<(numberOfIterations-1);i++){                   
    a1=komplexNumbers[0].real;
    b1=komplexNumbers[0].imaginar;
    a2=komplexNumbers[counter].real;
    b2=komplexNumbers[counter].imaginar;
    calculation(a1, a2, b1, b2, komplexNumbers);         
    counter++;

    free(komplexNumbers[counter]);
}

Эта программа иногда падает, а иногда нет.Я тоже не смог увидеть шаблон, почему это происходит, но именно функция free() вызывает сбой (поскольку, когда я удаляю free и запускаю программу с теми же значениями, она не вылетает).Я тоже не смог увидеть шаблон, в котором происходит сбой.Он может обрабатывать отрицательные числа

Примечание: каждый элемент структуры называется complexNumber, а массив называется complexNumbers (с s :))

Answer 1

Есть как минимум несколько проблем:

• Вы всегда присваиваете komplexNumbers[0] и освобождаете komplexNumbers[count]

• komplexNumbers[0] = *k вероятно означает, что komplexNumbers это массив структур, а не массив указателей - вы присваиваете структуру, а не указатель

РЕДАКТИРОВАТЬ

В светеВ недавнем коде легче показать вам, что делать, чем объяснить, что вы делаете неправильно.Как и предполагалось, komplexNumbers - это массив структур.В вашей функции вычисления вам не нужны все эти вещи malloc (и, следовательно, вам не нужен бит free).Сделайте это вместо:

komplexNumbers->real = temp1;
komplexNumbers->imaginar = temp2;

Answer 2

Как распределяется komplexNumbers (массив)?Вы не можете позвонить free на что-либо, что не было возвращено malloc.И у вас течет память, память на k теряется при возврате с calculation.

Answer 3

Я нахожу эту часть кода немного грязной: - komplexNumbers[0]=*k; внутри отсчета. Так как он будет снова и снова устанавливать указатель на сложный тип только в индексе 0. Когда вы попытаетесь и освободите это, перезапуская цикл for с использованием counter, он потерпит неудачу для значений counter> 0. Я так думаю, но было бы лучше если вы правильно сформулируете вопрос и опубликуете его, чтобы мы могли разобраться в нем`

Answer 4

Поскольку в вычислениях вы всегда устанавливаете значение 0-го элемента (копируя значение, обозначенное k), вы хотите освободить нераспределенные элементы массива.