realloc in Funktion auslagern

[sommerfee]

Zu dem eigentlichen Problem hat ja quinte17 schon was geschrieben.

Hierzu habe ich aber noch eine zusätzliche Anmerkung:

data = (Data*) realloc(data, rows * sizeof(Data));
if(data == NULL){
fprintf(stderr, "out of memory\n");
// return 0;
}

So bastelt man sich ein Speicherleck, denn wenn realloc() fehlschlägt, ist der ursprüngliche Speicher immer noch belegt. Also sollte man den Rückgabewert von realloc() in eine temp. Variable speichern, und erst dann "data" neu zuweisen, wenn es geklappt hat. Im Fehlerfalle kann man so noch den alten Speicher mit free( *data ) freigeben.

int i;
for(i = cnt.value_rows; i < cnt.value_rows+rows; i++)

So knallt es natürlich, wenn man data[i] verwendet und cnt.value_rows != 0 ist.

data[i].value = (double*) malloc(cnt.value_colums * sizeof(double));
data[i].value = i;

Ja, welcher Wert soll denn nun in data[i].value rein, da mußt du dich schon entscheiden

if(data[i].value == NULL){
fprintf(stderr, "out of memory\n");
// return 0;
}

Hier vermisse ich in der Fehlerbehandlung wieder das Freigeben des bisher belegten Speichers.

[dml]

Hi,
vielen Dank für Eure Unterstützung. Das Speicherleck wäre mir für eine längere Zeit, wenn überhaupt, gar nicht erst, aufgefallen. Ich habe noch eine Variante ausprobiert:

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


void mymalloc(int *data,int rows){
        data = malloc(rows*sizeof(*data));
}

void myrealloc(int *data,int rows){
        data = realloc(data,rows*sizeof(*data));
}

void myrealloc2(int **data,int rows){
        *data = realloc(*data,rows*sizeof(**data));
}


int main()
{
        int *a;
        printf("&a=%p, a=%p\n",&a,a);
        a = malloc(sizeof(int));
//         mymalloc(a, 1);
        printf("&a=%p, a=%p\n",&a,a);
        myrealloc(a, 6);
        printf("&a=%p, a=%p\n",&a,a);
	
        int *b;
        b = malloc(sizeof(int));
        myrealloc2(&b, 6);

	int i;
        for(i = 0; i < 6; i++){
                a[i] = 1;
                b[i] = 1;
	}

        for(i = 0; i < 6; i++){
                printf("%d\n",b[i]);		
	}

	
        printf("\n");

        free(a);
        free(b);
	
        return 0;
}

Was zeigen würde, das ein Doppelpointer gar nicht erst benötigt wird.
Vielen Dank!

[sommerfee]

Was zeigen würde, das ein Doppelpointer gar nicht erst benötigt wird.

Natürlich wird der Doppelpointer benötigt, zumindest in C, und wenn man es so macht wie du. mymalloc() und myrealloc() verändern lediglich die lokale Variable "data", deren Gültigkeitsbereich nur bis zum Funktionsende reicht.

Dementsprechend solltest du hier

printf("&a=%p, a=%p\n",&a,a);
myrealloc(a, 6);
printf("&a=%p, a=%p\n",&a,a);

beide Male den gleichen Wert für "a=" ausgegeben bekommen.

Und daß die Zuweisungen hinterher funktionieren, hat überhaupt nichts zu sagen, du pinselst halt über deinen zugewiesenen Speicherbereich hinüber, das kann später wildeste Auswirkungen haben. (C ist halt nicht Pascal oder Java. BTW: Nimm mir es nicht übel, aber kaufe dir ein Buch, welches solche elementaren Grundlagen besser als dein vorhandenes Lehrbuch vermittelt.)