Grundlagenfrage zu char * und asprintf

Hallo,

also wenn ich mich recht erinnere, gibt's im ersten Fall ein Speicherleck.
Die Initialisierung mit NULL ist meiner Meinung nach nicht notwendig.
Der free-Aufruf vom dritten Fall ist mir auch unklar, ich denke nicht, dass der was bewirkt.

Achim

Hi,

Zitat

Original von HFlor
in der Funktionsbeschreibung von asprintf(&buffer, ...) ist leider nichts davon enthalten, ob ein bereits reservierter Speicherbereich wie hier

Code

char *timestr = NULL;
  asprintf(&timestr, "%s", IndexToHMSF(Current, DisplayFrames));
  asprintf(&timestr, "%s  /  %s ", timestr, IndexToHMSF(Total));
  Text(width - cBitmap::Width(timestr) - 14, ty + 3, timestr, clrWhite, clrBackground);
  free(timestr);

in Zeile 2 in der Zeile 3 wieder freigeben wird?

Wenn nicht noch wichtige Zeilen fehlen, dürfte dort ein klassisches Speicherleak sein, da der Speicher der mit asprinf() zugeteilt wird, auch mit free() wieder freigeben werden muss, sonst ist keine Langzeitstabilität des Programmes gewährleistet.
Bei selten durchlaufenen Funktionen fällt sowas nicht mal auf, bei der heutigen RAM Bestückung.

Und da in der man-Page nichts von einer automatischen Freigabe steht, wird dies auch nicht stattfinden.

Zitat

Original von HFlor
* Wenn ja warum wird wie hier

Code

char *Prompt;
           asprintf(&Prompt, tr("Press key for '%s'"), tr(cKey::ToString(NewKey)));

nicht Prompt mit NULL initalisiert?

Die Initialisierung mit NULL ist im Prinzip nicht notwendig, entspricht aber einem sauberen Programmierstil, keine uninitalisierten Variablen einzusetzen.
Allerdings ist mit der Kontrolle der Variable auf NULL überprüfbar ob das interne malloc der Funktion fehlgeschlagen ist.
Zumal mit dem Setzen auf NULL und dem prüfen darauf kann auch eine doppelte und damit sinnlose Freigabe verhindert werden.

Zitat

Original von HFlor

oder wofür steht dann die 2 in diesem Code

Code

static char *buffer = NULL;
  free(buffer);
  asprintf(&buffer, "%s/plugins%s%s", configDirectory, PluginName ? "/" : "", PluginName ? PluginName : "");

So jetzt bin ich mit nicht mehr sicher, da ich persönlich keine
statischen Variablen innerhalb von Funktionen mehr einsetze, weil
der damit resultierende Code nicht mehr Threadsafe ist,

Wenn static char * ausserhalb der eigentlich Funktion liegt, erfolgt
die initialisierung wärend des Startup, bei nachschlagen in der Literatur
konnte aber auf die Schnelle nichts zu diesen Fall finden ob dies
auch so ist, ansonsten wäre es technischer Unfug.

Ich habe auch schon mal versucht valgrind auf den vdr zuhetzen,
aber der danach vollausgelasstet und es ging nichts mehr, aber in
showpic von glcd-plugin hat es diesbezüglich wertvolle informationen geliefert.

HTH,
Andreas

Hallo,

Zitat

Ist aber Orginal aus 'theme.c'

Deswegen darf's doch trotzdem falsch sein

Zitat

Es ist als static definiert und läuft in einer Schleife, so das der Aufruf vor free durchaus Sinn macht, wenn es nicht Bestandteil von asprintf ist.

Ups, das static hab ich übersehen. Dann machts vermutlich schon Sinn, obwohl mir das Verhalten beim *ersten* Aufruf bzw. Durchlauf der Schleife nicht klar ist.

Achim

Hallo,

free(NULL) macht einfach nichts. Also ist:

ststic char *bla = NULL;
free(bla);
asprintf(&bla, ...);

eine Abkürzung für:

statis char *bla = NULL;
if (bla) free(bla);
asprintf(&bla, ...);

Die Stelle in theme.c ist tatsächlich falsch, da müsste für die zweite Zeile ein zweites char* genommen werden, und hinterher müssten beide gefree'd werden.

Ich verstehe zwar nur Bahnhof mit Bratkartoffeln, aber es ist immer wieder schön sowas zu lesen (Fragt mich nicht warum ich sowas lese wenn ich gar keine Ahnung davon habe.)

Gruss,

Jörg

P.S.: Ich sollte wirklich mal wieder damit anfangen. Vielleicht kann ich dann irgendwann mal wieder mitreden

Meine C- Kentnisse sind 12 Jahre alt. Aber über ein etwas grösseres "Hello" bin ich damals nicht gekommen. Also mach dir keine Sorgen wenn ich hier nix verstehe

Wenn ich mich in das Thema wieder einarbeite und soweit kommen will wie Ihr
brauch ich noch mindestens 1 Jahr oder mehr.

Gruss,

Jörg

Hat diese Änderung jemand probiert?

asprintf(&timestr, "%s  /  %s ", IndexToHMSF(Current, DisplayFrames), IndexToHMSF(Total));

Bei mir funktioniert es nämlich nicht.
Statt der hinteren Totaldauer, wird dann auch die Current angezeigt.

Ich weiß nicht warum, aber es geht nicht.

lg, Gerhard