MemleakErrormeldung

Du weist deinem pointer eine neue Adresse zu. Du wirst deinen buffer mit 
strncpy kopieren müssen.

Ich dachte ich erzeuge erstmal nur einen Pointer, der auf die gleiche 
Adresse von Buf zeigt... und ggf. kleiner ist, weil ich nur die strlen 
von buf haben will... oder nicht?

> char* RecPointer =
Dein Pointer ist eine stinknormale lokale Variable.

> free(RecPointer)
Diese lokale Variable musst/darfst/kannst du nicht freigeben!
Das passiert automatisch.
Der vorher definierte Pointer verfällt beim verlassen der Funktion.


Wenn du auch den Speicherbereich freigeben willst, auf den der Pointer 
zeigt, dann....
Aber das sehe ich hier nicht.

hmmm... ok, dann nur einen Pointer in der Funktion erzeugen, auf den 
übergebenen Speicherbereich zeigen lassen, damit arbeiten und am Ende 
der Funktion vergessen, hat mir schonmal sehr geholfen, Danke!


Habe noch eine weitere Frage, und zwar ein ähnlich gelagertes Problem, 
genutzt wird die Funktion realloc.

1

BufPtr = realloc(BufPtr, LENGTH);

hier erhalte ich Errormeldung: memleakOnRealloc,


Der Speicher kann und soll allerdings tatsächlich wachsen und auch 
schrumpfen können und die Errormeldung betrifft genau diejenigen Zeilen, 
die im Falle des Schrumpfens aufgerufen werden.
Bin mir daher nicht sicher, ob es wirklich ein Fehler ist, oder nur eine 
zu scharf eingestellte Warnung, die einen Error erzeugt, aber auch für 
diesen Fall würde ich dennoch gerne wissen, wie ich "Errorfrei" aus der 
Situation herauskomme...

Arduino F. schrieb:
>> free(RecPointer)
> Diese lokale Variable musst/darfst/kannst du nicht freigeben!
> Das passiert automatisch.

Unsinn. Jeder Pointer muss man freigeben. Zu jedem malloc gehört ein 
free.

Memleak schrieb:

> Bin mir daher nicht sicher, ob es wirklich ein Fehler ist

Ist es, denn wenn das Realloc scheitert, kriegst Du zwar einen 
Nullpointer zurück, aber damit überschreibst Du den alten Bufferzeiger, 
und wie willst Du den Buffer dann jemals wieder freigeben?

Daher macht man das besser nach dem Muster:

1

BUF_T *TmpBufPtr = realloc(BufPtr, LENGTH);

2

if (TmpBufPtr == NULL)

3

{

4

    // error handling

5

    // BufPtr STILL points to allocated memory but can be free'd

6

} else

7

{

8

    BufPtr = TmpBufPtr ;

9

    // more stuff

10

}

Peter II schrieb:
> Arduino F. schrieb:
>>> free(RecPointer)
>> Diese lokale Variable musst/darfst/kannst du nicht freigeben!
>> Das passiert automatisch.
>
> Unsinn. Jeder Pointer muss man freigeben. Zu jedem malloc gehört ein
> free.

Und warum kommt dann beim TO der Fehler?
Was hat er falsch gemacht und wie kann er es besser machen?

Adele schrieb:
> Und warum kommt dann beim TO der Fehler?
> Was hat er falsch gemacht und wie kann er es besser machen?

er überschreibt seinen Pointer, damit gibt er beim free den falschen 
Speicher frei.

1

char* RecPointer = (char*) malloc(sizeof(char) * strlen(buf));

so jetzt überschreibt er den Zeiger, wo er sich Speicher besorgt hat.

1

RecPointer = buf;

damit war das malloc sinnlos. Das Free gibt dann den Speicher von buf 
was so bestimmt nicht gewollt ist.

Nachtrag:

den Speicher den er mit malloc kann er gar nicht mehr freigeben, weil er 
seinen Pointer überschrieben hat. Damit hat er ein Memleak

Frank M. schrieb:
> Das liest sich so, als ob ich nach einem p1 = malloc(l1) und
> anschließendem erfolgreichen p2 = realloc(p1, l2) später auch noch p1
> freigeben müsste. Dem ist nicht so. Ich muss nur den zuletzt erhaltenen
> Pointer freigeben, also p2.

ich habe mich auf den 1. Code bezogen, da gab es kein realloc.