_beginthreadex - Problem mit Parameter übergabe an Thread

Weiß keiner eine Lösung?
Ist echt dringend!

Am Sinnvollsten ist es, den vorhandenen Code zu nehmen und Stück für Stück in eine neue Sourcecode-Datei zu kopieren, so dass er möglichst kurz bleibt und dennoch den Fehler beinhaltet. Häufig stolpert man beim Erstellen dieses minimalen Codebeispiels selbst über die Fehlerursache.

Naja das hilft mir auch nicht wirklich weiter,da ich nicht weiß wie ich den Fehler beheben kann.
Es liegt ja nur an einer Zeile.

zeig einfach ein minimalen, kompletten, komplierbaren code der zum abstürz führt. dann kann man dir auch helfen.

Leider stürtzt mein Programm immer ab wenn ich folgendes mache:

unsigned __stdcall cmd(void* message) {
//irgendwas

free(message) //<----------Hier stürtzt alles ab....
}

Also bitte mal die Augen aufmachen beim Lesen.

das lässt sich so nicht kompilieren!

for(;;)
{
//irgendwelcher code

char *p = calloc(strlen(buffer),sizeof(char));
strcpy(p, buffer);
_beginthreadex(NULL,0,cmd,p,0,NULL);
}

unsigned __stdcall cmd(void* message) {
//irgendwas

free(message) //<----------Hier stürtzt alles ab....
}

Ich denke jeder dürfte jetzt das Problem erkennen.

calloc(strlen(buffer) + 1, 1);

Leider stürtzt des Programm immer noch bei free() ab.

bei mir nicht

#include <windows.h>
#include <process.h>
#include <iostream>

unsigned __stdcall cmd(void* message)
{
    free(message);
    return 0;
}

int main()
{
    const char* buffer = "Hallo";
    char *p = (char*)calloc(strlen(buffer) + 1, 1);
    strcpy(p, buffer);
    _beginthreadex(NULL,0,cmd,p,0,NULL);
    std::cin.get();
}

Es lag tatsächlich an folgenden:

p = (char*)calloc(strlen(puffer)+1,1);

Wenn ich es jetzt so wieder mache:

p = calloc(strlen(puffer)+1,1);

Dann stürtzt des Programm doch wieder ab.
Kannste mir des mal kurz erklären warum es mit einem cast (char*) klappt?

keine ahnung! den cast braucht man nur wenn man c++ benutzt. unter c ist der cast nicht nötig und nicht empfehlenswert.

hier steht's: http://www.c-plusplus.net/forum/viewtopic.php?t=206606

ich habs nochmal in c und ohne cast probiert. stürzt auch nicht ab. der fehler liegt bestimmt woanders.

Ok das ganze Programm läuft jetzt eigentlich bis auf eine kleinigkeit.

//main Thread
char *b = NULL;

 if(b == NULL)
 //bedingung

x = _beginthreadex(NULL,0,split_cmd,b,0,NULL); //thread aufruf

//thread
unsigned __stdcall split_cmd(void* message) {

free(message);
message = NULL;

Doch leider zeigt b im "Mainthread" nach ausführung des "Nebenthreads" nicht auf NULL.

lass die finger von threads wenn du solche grundlagen nicht verstanden hast.

Ich denke das Problem hat weniger mit Threads zu tun.

hustbaer

Martin Schmid schrieb:

Ich denke das Problem hat weniger mit Threads zu tun.

Ich denke du hast vollkommen Recht

hustbaer schrieb:

Ich denke du hast vollkommen Recht

Schön das mir mal jemand recht gibt.

Aber woran liegt es dann?

hustbaer

Du übergibst den Wert der in "p" steht by value an die Thread Funktion.
Das heisst du übergibst eine Kopie des Wertes.

Dadurch kann aber der Wert der "in" der originalen Variable "p" steht niemals vom Thread verändert werden, da dieser ja nur eine Kopie des Wertes übergeben bekommen hat, und daher keinen Zugriff mehr auf die originale Variable "p".

Wenn du "p" ändern willst, musst du die Adresse von "p" übergeben.

//main Thread
volatile char *b = NULL; // NOTE: volatile ist ein GANZ ÜBLER Hack, bloss
                         // die "Korrekte" Variante zu erklären ist mir
                         // grad viel zu umständlich, sorry.
                         // Auf jeden Fall gehört hier volatile hin, wenn
                         // es überhaupt irgendwie funktionieren soll.

if(b == NULL)
 //bedingung

x = _beginthreadex(NULL,0,split_cmd,&b,0,NULL); //thread aufruf

// --------------------------------------------------------------------
// --------------------------------------------------------------------

unsigned __stdcall split_cmd(void* param) {

char** ppMessage = reinterpret_cast<char**>(param); // Zeiger auf "b" ("b" ist selbst ein Zeiger auf char, daher Typ char**)
char* pMessage = *ppMessage; // Wert von "b" (die Kopie des Wertes)

free(pMessage); // zum freigeben können wir den Wert (die Kopie des Wertes) verwenden
*ppMessage = NULL; // zum Null-Setzen müssen wir auf die originale Variable "b" zugreifen, also über den Zeiger auf "b" gehen

Das ist aber trotzdem furchtbares gefrickel.

In dieser Zeile:

char** ppMessage = reinterpret_cast<char**>(param);

Erhalte ich diese Fehler:

error C2065: 'reinterpret_cast' : undeclared identifier
error C2059: syntax error : 'type'