Discussion :

[captaindidou]

J'ai écrit un petit programme pour comprendre comment appeler correctement SuspendThread et GetThreadContext.
Je n'y ai pas réussi.

Le programme consiste à créer un thread 'StarThread' qui ecrit 0, 1 .., 10 à l'écran et se suspend (le thread principal ayant fourni en parametre l'handle).

Pourtant SuspendThread n'agit pas et ne relève pas d'erreur non plus.

Ne parlons pas de GetThreadContext qui donne EIP à 0.

Qu'est-ce qui cloche ?
Merci bcp.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
DWORD WINAPI StartThread(LPVOID argThreadHandle)
{
      char lszParam[3];
      for(int i=0;i<=10;i++)
      std::cout<<i<<std::endl;
      if (SuspendThread((HANDLE) argThreadHandle) == FALSE)
      {
          std::cerr<<"Je suis une conne de founction SuspendThread"<<std::endl;
          return 1;
      }
      std::cerr<<"ResumeThread a ete appele ou quoi ?"<<std::endl;
      return 0;
}
int main(int, char**)
{
      HANDLE hThread1;
      DWORD dwGenericThread;
      hThread1 = CreateThread(NULL,0,StartThread,&hThread1, 0 ,&dwGenericThread);
      if(hThread1 == NULL)
      {
            DWORD dwError = GetLastError();
            std::cerr<<"SCM:Error in Creating thread"<<dwError<<std::endl ;
            return 1;
      }

Sleep(2000);

CONTEXT context;

context.ContextFlags = CONTEXT_i386
                             | CONTEXT_INTEGER
                             | CONTEXT_FLOATING_POINT
                             | CONTEXT_EXTENDED_REGISTERS;

BOOLEAN succes = GetThreadContext(hThread1, &context);
.
.
.
}

[ram-0000]

Je n'aime pas la manière dont tu procédes car elle peut avoir des effets de bord et en plus j'ai l'impression que ce que tu fais n'est pas correct.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

HANDLE hThread1 = CreateThread(..., StartThread, &hThread1, ...);

Dans ce cas, tu passes à ton thread l'adresse d'un HANDLE qui n'est pas forcément fini d'initialiser. Si ton thread part avant que la fonction CreateThread() ne soit terminée (je ne sais pas si c'est possible), le handle de thread pointé par le paramètre risque d'être invalide (mais dans le cas présent, je ne pense pas que cela soit le problème car cette valeur est utilisée bien plus tard).

Dans ton cas, ton thread reçoit un paramètre qui est en fait un pointeur sur HANDLE et ton code de thread devrait ressembler à un truc comme cela :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
DWORD WINAPI StartThread(LPVOID argThreadHandle)
{
// nouveau code
HANDLE hthread = *((HANDLE *)argThreadHandle);
// ancien code
//HANDLE hthread = (HANDLE)argThreadHandle;

...
      if (SuspendThread(hthread) == FALSE)
...

Pour éviter le risque de variable pas tout à fait finie d'initialiser, il vaudrait mieux dans ce cas utiliser GetCurrentThread() comme cela, tu n'as plus besoin de passer un paramètre à ton thread (ou tu peux l'utiliser pour autre chose)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
DWORD WINAPI StartThread(LPVOID argThreadHandle)
{
...
      if (SuspendThread(GetCurrentThread()) == FALSE)
...

Pour GetThreadContext(), c'est peut être le même problème (je ne connais pas cette fonction, je n'ai jamais eu à l'utiliser.

[captaindidou]

On ne peut plus d'accord avec toi. C'est un peu osé.

SuspendThread fait à présent son oeuvre.

Donc, en toute logique je dois mettre [Résolu], bien que GetThreadContext reste une énigme.

Merci à toi.
Bonne journée.

[ram-0000]

Le bouton est en bas à gauche, ce n'est pas utile de modifier le titre du message