[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Lost] [Patch] LostIO - io_split_dirname



Toni Kaufmann schrieb:
Dieser Patch ergänzt unsere Pfadfunktionen um eine Funktion, die das
letzte Glied vom Pfad abtrennt und nur den Anfang zurückgibt.

Ist es Absicht, daß du keine relativen Pfade auflöst und nicht ./..-bereinigst? Das hätte ich in so einer Funktion schon erwartet.

+    size_t size = calc_path_length(element_stack);
+    if ((result = malloc(size + 1))) {
+        // Wenn das geklappt hat, wird der Pfad jetzt wieder in einen String
+        // umgewandelt
+        create_path_string(element_stack, result, TRUE);
+    } else {
+        // Falls ein Fehler beim anlegen des Puffers aufgetreten ist, wurden
+        // die Elemente nich freigegeben. Dann geschieht das jetzt manuell.
+        struct path_element* element;
+        while ((element = list_pop(element_stack))) {
+            free(element);
+        }
+    }

Diese Überprüfung und den else-Zweig gibt es in io_get_absolute_path() nicht. Eins von beidem ist also falsch, vermutlich die ältere Funktion. Kommt dazu noch ein Patch?