probleme de structure

Sujet résolu

23 novembre 2007 à 20:00:38

salut

je suis en train de coder un client qui doit entrer en contact avec un serveur via un thread, cependant lorsque je lance mon client il m'indique l'erreur : client.exe a rencontré une erreur et doit fermé.

Lorsque je lance le debug il m'indique une erreur de segmentation cependant je ne comprend pas pourquoi

Voici les codes

fonction thread :

DWORD WINAPI fonction_thread(void* x)
{
EnterCriticalSection(&Sync);
thread_param param;
bool continuer = false;
if( x!= NULL)
{
struct thread_param *params = reinterpret_cast<struct thread_param *>(x);
param.position = params->position;
param.activer = false;
}
else
{
continuer = false;
}
SOCKET sock_env;
SOCKADDR_IN sin_env;
sock_env = socket(AF_INET, SOCK_STREAM, 0);//creation de la socket
sin_env.sin_addr.s_addr = inet_addr("127.0.0.1");//initialisation des parametres
sin_env.sin_family = AF_INET;
sin_env.sin_port = htons(5000);
LeaveCriticalSection(&Sync);
int pre_message;
char message[5] ="";
EnterCriticalSection(&Sync);
if(connect(sock_env, (SOCKADDR *)&sin_env, sizeof(sin_env))!= SOCKET_ERROR)
{
send(sock_env,message, sizeof(message),0);
*param.activer = true;//le compilo m'indique une erreur de fragmentation ici
continuer = true;
}
LeaveCriticalSection(&Sync);
while(continuer)
{
EnterCriticalSection(&Sync);
creermessage(*param.position, message);
LeaveCriticalSection(&Sync);
send(sock_env,message, sizeof(message),0);
if(*param.activer == false)
{
continuer = false;
}
Sleep(50);
}
send(sock_env,"sp", sizeof("sp"),0);//envoie
closesocket(sock_env);
}

voici la création et la structure thread param

struct thread_param{
SDL_Rect *position;
bool *activer;
};
[...]
bool activer = false;
thread_param x;
x.activer =&activer;
x.position = &posobjet;

Voilà tout merci d'avance à ceux qui prendront le tmeps de lire et de m'aider

icare

P.S : si vous avez besoin de plus de code dite le moi

MatteX

23 novembre 2007 à 20:31:27

Citation : icare14

// ...

if( x!= NULL)

{

// ...

param.activer = false; <----

}

// ...

*param.activer = true;//le compilo m'indique une erreur de fragmentation ici

//...

voici donc ce que tu fais :

1) affectation de la valeur du pointeur param.activer à false (0)
2) accède à la valeur pointée par param qui vaut 0 (SEGFAULT!)

il te manque ton opérateur de déréférencement devant la première ligne

icare14

23 novembre 2007 à 20:33:28

if( x!= NULL)
{
struct thread_param *params = reinterpret_cast<struct thread_param *>(x);
param.position = params->position;
*param.activer = false;
}

J'ai modifié comme ci dessus (c'est bien comme ça qu'il faut modifier ?)

mais maintenant l'erreur de segmentation s'est déplacée à cet endroit

EDIT : j'ai trouvé en fait ça pointait sur rien donc j'ai mis :

param.activer = params->activer;

et ça a l'air de marcher

MatteX

23 novembre 2007 à 20:35:48

normal ton pointeur n'est pas initialisé non plus

param.activer = new bool( false );

icare14

23 novembre 2007 à 20:42:26

juste une dernière question : quelqu'un connaitrais un moyen d'attendre la fin d'execution d'un thread ?

MatteX

23 novembre 2007 à 20:50:22

Windows : WaitForSingleObject, WaitForSingleObjectEx, WaitForMultipuleObjects

sinon tu peux intégré ton thread dans une classe et avoir une méthode disant si le thread est vivant... il ne te reste qu'à faire un :

while( thread.isAlive() )
Sleep( 1000 );

icare14

23 novembre 2007 à 20:51:16

ok merci beaucoup pour ta précieuse aide