Warum verliert der Code Speicher?

Question

Ich wundere mich, warum dieser kleine Code-Frieden undichte Speicher verloren geht? Es dreht mein Bild horizontal und gibt das struct Image an meine Hauptdatei zurück.

void turnImg(Image *img) { 

    Image *tmp = (Image*)malloc(sizeof(Image)); 

    //swapping size between height and width 
    tmp->height = img->height; 
    img->height = img->width; 
    img->width = tmp->height; 

    //The loop is swapping every pixel from the "last" pixel to the "first" into a tmp 
    tmp->pixels = (Pixel**)malloc(sizeof(Pixel*)*img->height); 
    for (unsigned int i = 0; i < img->height; i++) { 
     tmp->pixels[i] = (Pixel*)malloc(sizeof(Pixel)*img->width); 
     for (unsigned int j = 0; j < img->width; j++) { 
      tmp->pixels[i][j] = img->pixels[img->width - 1 - j][img->height - 1 - i]; 
     } 
    } 

    for (unsigned i = 0; i < img->height; i++) { 
     free(img->pixels[i]); 
    } 
    free(img->pixels); 
    //tmp gives back the pixels to img, but they are now flipped 
    img->pixels = tmp->pixels; 

    free(tmp); 
} 

Nichts sollte in der Hauptdatei falsch sein, weil alle meine anderen Funktionen sind wohlauf ... Aber hier ist der Friede des Hauptes welches das Senden und Zurückführen die Struktur zur Funktion:

case 5: 
     //Flipping the image on its diagonal. 
     printf("Flipping the image on its diagonal..."); 
     turnImg(&plain); 
     printf("Image flipped.\n"); 
     break; 

und die Hauptdatei endet mit:

for (unsigned int i = 0; i < plain.height; i++) { 
    free(plain.pixels[i]); 
} 
free(plain.pixels); 

getchar(); 
return 0; 

doch was ich bemerkt habe ist, dass die Höhe und Breite Swap ein Teil des Problems ist, aber ich weiß, wie nicht ich in der Lage sein, dies zu tun, ohne ein Swa p.

Answer 1

Sie haben img geändert, indem Sie deren Höhe und Breite vertauscht haben.

Wenn Sie jedoch img sind die Freisetzung um Platz zu machen für die tmp Pixel, Sie sind zu befreien img im Anschluss an die tmp s Höhe. Das heißt, Sie verwenden img s neue Höhe statt seiner ursprünglichen Höhe.

Um zu beheben, ändern Sie Ihre Schleife zu frei mit img s Breite (die seine alte Höhe ist).

for (unsigned i = 0; i < img->width; i++) { 
     free(img->pixels[i]); 
    }