Kommentare große Funktion

imageInput.c

@@ -81,7 +81,7 @@ GrayScaleImageSeries *readImages(const char *path)
         return NULL;
     }
 
-    //Speicher für die gesamte Bildserie reservieren (aber nur für die Pointer und das count-Feld)
+    //Speicher für die gesamte Bildserie reservieren (aber nur für die Struktur)
     series = malloc(sizeof(GrayScaleImageSeries));
     if(!series) {
         free(series);
@@ -91,21 +91,12 @@ GrayScaleImageSeries *readImages(const char *path)
 
     series->count = numImages;
     //Speicher für die Pointer an sich belegen
-    series->images = malloc(numImages * sizeof(GrayScaleImage));
+    series->images = malloc(numImages * sizeof(GrayScaleImage)); //Speicher für alle Bild-Strukturen, nicht aber für die Pixel selbst (denn image ist nochmal ein Array selbst)
-    series->labels = malloc(numImages * sizeof(unsigned char));
+    series->labels = malloc(numImages * sizeof(unsigned char)); //Speicher für alle Labels
 
-    //Prüfen, ob der Speicher reserviert werden konnte
+    //Prüfen, ob Speicher für die Arrays reserviert werden konnte, wenn nicht -> Freigabe
-    for(unsigned int i = 0; i < numImages; i++){
+    if(series->images == NULL || series->labels == NULL){
 
-        series -> images[i].buffer = malloc(width * height);
-
-        if(series->images[i].buffer == NULL){
-
-            for(unsigned int j = 0; j < i; j++){
-                free(series->images[j].buffer);
-            }
-
-        
         free(series->images); //wenn trotzdem was reingeschrieben wurde, dann freigeben
         free(series->labels);
         free(series);
@@ -113,17 +104,39 @@ GrayScaleImageSeries *readImages(const char *path)
         fclose(fp);
         return NULL;
 
+    }
+
+    //Speicher für die Pixeldaten selbst reserviern und Prüfen, ob er reserviert werden konnte 
+    for(unsigned int i = 0; i < numImages; i++){
+
+        series -> images[i].buffer = malloc(width * height); //Speicher für Pixeldaten selbst
+
+        if(series->images[i].buffer == NULL){ 
+
+            for(unsigned int j = 0; j < i; j++){
+                free(series->images[j].buffer); //wenn Speicher nicht reserviert werden konnte, dann zuerst das Array mit Pixeldaten freigeben
+            }
+
+        
+        free(series->images); //anschließend auch die Speicher für Bildstrukturen und Label-Arrays freigeben, weil zuvor für diese schon Speicher reserviert wurde
+        free(series->labels);
+        free(series);
+    
+        fclose(fp);
+        return NULL;
+
         
         }
     }
 
    for(int i = 0; i < numImages; i++)
    {
-        
+        //Breite und Höhe werden gesetzt (GrayScaleImage)
         series -> images[i].width = width;
         series -> images[i].height = height;
 
-
+        
+        //Pixeldaten einlesen und Prüfen ob alle essentiellen Pixel eingelesen wurden
         if(fread(series -> images[i].buffer, 1, width * height, fp) != (width * height))
         {
             clearSeries(series);
@@ -131,7 +144,12 @@ GrayScaleImageSeries *readImages(const char *path)
             return NULL;
         }
 
-        fread(&series -> labels[i], 1, 1, fp);
+        //Jedes Bild hat ein Label, was direkt danach eingelesen wird und auf Größe geprüft wird
+        if(fread(&series -> labels[i], 1, 1, fp) != 1){
+            clearSeries(series);
+            fclose(fp);
+            return NULL;
+        }
     }