Verständnisfrage C-Code in VDR

und Dein code ?
Frithjof

m
original VDR:

char *cRecording::StripEpisodeName(char *s)
{
  char *t = s, *s1 = NULL, *s2 = NULL;
  while (*t) {
        if (*t == '/') {
           if (s1) {
              if (s2)
                 s1 = s2;
              s2 = t;
              }
           else
              s1 = t;
           }
        t++;
        }
  if (s1 && s2)
     memmove(s1 + 1, s2, t - s2 + 1);
  return s;
}

s wird nur durchgereicht und nirgenwo verändert und somit sollte free(s) auch ok sein.
verstehs erstmal auch nicht
Frithjof

Hi Brougs78,

der VDR-Code sollte funktionieren. Dort wird ein Pointer
auf einen Teil des Strings FileName() zurückgeliefert.
Nur auf diese Kopie wird free() angewendet. Die Kopie
hat mit dem Original nichts mehr zu tun.

Mich wundert nur, dass dein Aufruf von

free(s - strlen(VideoDirectory) - 1)

funktioniert, denn genau das müsste einen SEGFAULT liefern.

Schau nochmal genau nach.

Viele Grüße
KdF

Zitat

Original von Brougs78
Da müssten sich ja jede Menge Speicherlecks rumtreiben da ich free nicht aufrufe ...

Stimmt, aber Du greifst ja nicht auf Speicher zu der nicht Dir gehört, und löscht auch keinen Speicher der nicht Dir gehört. Deshalb gibt's hier verständlicherweise auch keine Segfaults

Ein gutes Betriebssystem merkt sich auch den Speicher, den eine Task sich geholt hat und gibt ihn wieder frei, sobald die Task endet, kann aber nicht sagen, ob das bei Windows mit mingw auch so ist.
In Deinem Beispiel passiert also erstmal nichts schlimmes.
Falls das Problem noch besteht, dann zeig mal einen größeren Ausschnitt von Deinem Programm.
Frithjof

Zitat

Original von frithjof
Ein gutes Betriebssystem merkt sich auch den Speicher, den eine Task sich geholt hat und gibt ihn wieder frei, sobald die Task endet, kann aber nicht sagen, ob das bei Windows mit mingw auch so ist.

Das dürfte grundsätzlich wohl für jedes Protected-Mode Betriebssystem heutzutage gelten. Dennoch ist es eine schlechte Praxis, sich darauf zu verlassen - Denn insbesondere Server und nicht zuletzt ja auch unser VDR werden nicht sehr oft beendet (im Fall VDR jedenfalls nicht bei jedem).

Zitat

Original von Brougs78
??? also da komme ich jetzt nicht mit.
Lt. der Beschreibung des Befehls "strdup()" wird dabei der Speicher für den kopierten String reserviert und man muss ihn selber wieder mit "free()" freigeben. Ist dem nicht so? Oder ist das nur ein Programmierfehler den der Compiler (oder wer auch immer) für mich ausmerzt und mich nicht davor warnt dass ich eigentlich was vergessen habe?

Ausmerzen tut der Compiler in dem Bereich nichts für Dich. Das Speicherleck entsteht weiterhin. Speicherleck heisst ja, dass Du irgendwo Speicher allokiert hast (hier tut das strdup für Dich), auf den kein Zeiger mehr zeigt (weil der bei jedem Schleifendurchlauf überschrieben wird). Du kommst also später nicht mehr dran, um zu löschen. Nichtsdestotrotz existiert er noch. Aber Speicherlecks verursachen keine Segfaults, verschwenden nur Speicher

Zitat

BTW, wenn ich die Kopie als const char* definiere, dann kann ich free() garnicht darauf anwenden (kompiliert nicht). Gibts da dann nen anderen Befehl oder muss ich da den Speicher nicht freigeben?

Wie das bei free ist kann ich nicht sagen, mit new/delete kann man aber auch konstante Speicherbereiche holen und freigeben. Macht aber imho hier sowieso keinen Sinn, da ja herade der Zweck von strdup ist, Dir eine Kopie zu beschaffen, die Dir gehört, die Du also auch verändern darfst.

Zitat

In der SortName()-Routine von VDR wird mit strdup() Speicher von n Bytes reserviert und der Pointer zurückgegeben für den Start dieses Speichers. Jetzt wird allerdings dieser Pointer gleich um einige Byte verschoben und der Pointer zeigt nicht mehr an den Start des Speicherbereiches. Wenn jetzt free den Speicher wieder freigibt bei übergabe dieses Pointers, dann - nehme ich an - macht es das von der aktuellen Pointeradresse bis '\0' richtig? Das heißt die paar Byte vor der Pointeradresse die zuerst mitreserviert wurden liegen da immer noch so im Speicher rum (im Beispiel von VDR sowas wie "\video\", oder wie werden die wieder freigegeben?

Speicher wird immer blockweise freigegeben, d.h. das System weiss anhand der Adresse, wie groß ein Block ist. Ein Freigeben einer allokierten Adresse + X ist schlichtweg nicht definiert. Allerdings sehe ich in dem angesprochenen Programmteil nicht, wo der Pointer vor dem Freigeben verschoben wird. Dem strdup wird ein Zeiger auf einen Teilstring übergeben (die Addition mit strlen(VideoDirectory) passiert in den Klammern) und er liefert einen neuen Speicherbereich mit einer Kopie dieses Teilstrings zurück. Und der Zeiger, der auf diese Kopie zeigt, wird m.E. in den angegebenen Codes nirgendwo verschoben.

Zitat

Original von Brougs78
D.h. solange ich mich mit meinem Pointer in dem allokierten Speicher befinde habe ich kein Problem. Wenn ich dann den Pointer an free() übergebe ist es egal ob er gerade am Anfang oder am Ende des Speicherbereichs steht.
Dann liegt hier wohl mein Denkfehler. Ich dachte free() muss den Anfangsbereich des Speicherbereichs erhalten.

free muss einen Zeiger auf den Anfang erhalten. Ein free in der Mitte ist undefiniert.

Zitat

Wegen dem Beispiel aus VDR: strdup() erstellt IMHO eine vollständige Kopie des Strings und erst dann wird der Zeiger verschoben und "s" verweist auf einen Teilstring.

strdup erstellt eine vollständige Kopie desjenigen Strings, den es übergeben bekommt. Und das ist in diesem Fall ein Teilstring, da die Berechnung in der Klammer schon lange abgeschlossen ist, wenn der Aufruf tatsächlich passiert. Beim Aufruf erhält strdup doch sowieso nur einen Zeiger, es kann also garnicht wissen, um wieviel Stellen Du diesen verschoben haben möchtest.

Nicht ganz, der Lord hat schon recht, die Klammern sind wichtig

StripEpisodeName(strdup(       FileName() + strlen(VideoDirectory) + 1       ) );


  /video/Atlantis.......\0

s Zeigt auf das A von Atlantis und ab da macht strdup() eine Kopie.
bei free(s) muss s auch noch da stehen.
man free:
free() frees the memory space pointed to by ptr, which must have been returned by a previous call to malloc(), calloc() or realloc(). Otherwise, or if free(ptr) has already been called before, undefined behaviour occurs. If ptr is NULL, no operation is performed.
Frithjof