Discussion :

[trobin]

Bonjour,

je vais tâcher d'être suffisemment clair afin d'exposer mon problème en vue de trouver uen solution.

Voilà, j'ai un souci avec l'execution multiple de code depuis des applications (EXE) dffférents sous windows.

1/ une libraire statique (SYSTEM.LIB) contient une routine "message.c" pouvant afficher un message à l'écran selon les conditions d'executions et de résultats antérieurs.

2/ une seconde libriarie statique (ROUTINE.LIB) contient une une routine "appel.c" qui dans son exécution fait appel à la routine "message.c".

3/ l'appel à la routine "message.c". se fait dans un thread de la façon suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
			// creation du thread
			hthread_rcv=CreateThread
			(
				NULL,
				0,
				(LPTHREAD_START_ROUTINE)message,
				(LPVOID)pointer,
				0,
				&threadID_rcv
			);	
 
			// ferme le handle
			CloseHandle(hthread_rcv);

4/ Quatre EXE différents (A.exe, B.exe, C.exe et D.exe), linkés avec ces 2 libraires sont susceptibles de faire afficher ce message, donc d'appeler la routine "appel.c". les 4 EXE tournent en même temps et effectuent périodiquement différents calculs.

5/ une fois les conditions remplies pour faire afficher le message par l'un des 4 EXE, le message est affiché (exemple A.exe va être à l'origine des conditions), un thread est créé dans l'EXE appelant (A.exe), ainsi que des USER objects et des GDI objects.

6/ dans la routine "message.c", je vais vérifier que si le message est affiché, je n'en affiche pas un autre, en utilisant le code suivant :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
	LPCSTR Wname="message";	
	HANDLE hMutex;
 
	// si l'instance existe déjà
	hMutex = CreateMutex(NULL, FALSE,Wname);
 
	if ( GetLastError() == ( ERROR_ALREADY_EXISTS || ERROR_ACCESS_DENIED ) ) 
	{
		// ferme le handle créé
		if (hMutex) 
			CloseHandle(hMutex);
		// on s'en va
		return; 
	}

7/ maintenat si les conditions d'affichage du message pour une autre EXE sont remplies elles aussi (exemple pour C.exe), un nouveau message est affiché, avec un un thread est créé dans l'EXE appelant (C.exe), ainsi que des USER objects et des GDI objects.

==> le problème se situe là : je pensais que le code du point 6/ allait empêcher ce nouveau message d'être affiché, mais non.

d'où ma question : comment faire pour n'avoir plus qu'un seul message affiché, et avoir la possibilité, pour n'importe quel EXE de faire fermer ce message, qui aurait pu être ouvert pour un autre EXE ?

Merci beaucoup par avance de votre aide,
Thierry

[Médinoc]

Inclus l'ID du processus (ou du thread) appelant dans le nom du mutex.

Aussi, retire ce cast en (LPTHREAD_STARD_ROUTINE). Les casts de pointeurs de fonction sont à bannir. Si ça ne compile pas, regarde ma signature.

[trobin]

Merci Médinoc de ta réponse rapide ;

avant de revenir sur ce que tu écris, j'aimerai faire un rajout d'une modif sur le check fait sur le GetLastError supplémentaire dans mon code qui semble - en partie - avoir résolue une partie de mon souci :

Code :

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

1
2
3
4
5
6
7
	DWORD _CrMuError;	// 
 
	// si l'instance existe déjà
	hMutex = CreateMutex(NULL, FALSE,Wname);
	_CrMuError = GetLastError();
	if ( (_CrMuError /*GetLastError()*/ == (DWORD)ERROR_ALREADY_EXISTS ) 
		|| (_CrMuError /*GetLastError()*/ == (DWORD)ERROR_ACCESS_DENIED ) )

je dis "semble", car plus loin dans mon code de "message.c" j'ai d'autre vérifications qui ne vont pas non plus ; en gros, une variable globale définie dans cette routine qui semble bien n'être mise à jour qu'à travers l'appel d'exécution du message ; en gros, "message.c" est construit de la sorte :

Code :

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

1
2
3
4
5
6
7
8
9
 
// variable globale
static BOOL enroute;	// Flags
 
// procédure pour gérer le message et son contenu (enroute sera mis à jour dans cette procédure)
LRESULT CALLBACK WndPro(HWND hwnd,UINT msg,WPARAM wParam, LPARAM lParam)
 
// Mon main (enroute sera vérifié dans cette fonction qui contient aussi le check avec le CreateMutex)
void shsys_counter(LPVOID param)

Je pense donc qu'il faut que je définisse enroute autrement et/ou ailleurs.... mais comment ?

Sinon, pour revenir sur ta remarque : je vais faire ce que tu me dit concernant l'ID du processus/thread... faut juste que je trouve comment
Pour ce qui est le cast, je vais le retirer... merci de l'info.

Thierry