Discussion :

[ManusDei]

Après les HANDLE, CreateProcess.

tabtemp est initialisé ailleurs, mais vaut "dir" au moment du lancement de CreateProcess (merci cout).

Code C++ :

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
	int error = 0;
	Process *process;
	PROCESS_INFORMATION *pi = new PROCESS_INFORMATION;
	STARTUPINFO si = {sizeof(si)};
 
	if (!error) {
		// Build arguments
		StringBuffer tab;
		for(int i = 0; i < args.count() ; i++)
			tab << args[i];
		char tabtemp[tab.length() +1];
		strcpy(tabtemp,tab.toString().chars());
		cout << tabtemp << GetLastError() << io::endl;
		// Launch process
		if(CreateProcess(tabtemp,NULL,0,0,TRUE,0,0,0,&si,pi) == 0)
			error = GetLastError();
		cout << "error = " << error << io::endl;
		process = new Process(pi);
 
	}
 
	// Return the result
	if(error)
		throw new SystemException(error, "process building");
	else
		return process;

En sortie du terminal, j'ai

Code :

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

1
2
3
4
5
6
dir12
error = 2
terminate called after throwing an instance of 'elm::system::SystemException*'

This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.

Donc le dernier code d'erreur était 12 ERROR_INVALID_ACCESS
avant le lancement de CreateProcess, et avec le lancement de CreateProcess il passe à 2 ERROR_FILE_NOT_FOUND, bref l'appel à CreateProcess échoue.

Mais je vois pas trop pourquoi ni comment :s

[ram-0000]

Peut être ajouter un espace pour séparer les paramètres

Code :

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

1
2
3
4
5
for(int i = 0; i < args.count() ; i++)
{
			tab << args[i];
			tab << " ";
}

Cette instruction ne fait rien :

Code :

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

		char tabtemp[tab.length() +1];

Au fait, la commande "dir" est une commande builtin du shell, il ne peut pas être appelé par CreateProcess(). CreateProcess ne peut appeler qu'un binaire (et peut un script bat ou cmd)

[ManusDei]

Au fait, la commande "dir" est une commande builtin du shell, il ne peut pas être appelé par CreateProcess(). CreateProcess ne peut appeler qu'un binaire (et peut un script bat ou cmd)

Ah, ça doit être ça. J'essayerais demain, merci.

[ManusDei]

C'était ça :'(

Merci !

[ManusDei]

En fait pas tout à fait résolu.

Je lance "ProcessBuilder builder("C:\\Win16App\\MinGW\\bin\\gcc.exe");"
J'ai bien l'affichage à l'écran "gcc.exe : no input files"

Sauf que j'ai un code d'erreur 12 (ERROR_INVALID_ACCESS) qui traine .

Code C++ :

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

1
2
3
4
5
if(CreateProcess(tabtemp,NULL,0,0,TRUE,0,0,0,&si,pi) == 0)
	error = GetLastError();
cout << "error = " << GetLastError() << io::endl; /*là, le code d'erreur est 0 */
process = new Process(pi);
cout << "error = " << GetLastError() << io::endl; /* la c'est 12*/

Code C++ :

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

1
2
3
4
5
Process::Process(void* _pi){
	printf("init process %d\n",GetLastError()); /* là le code d'erreur est 12 */
	this->pi=_pi;
	printf("saving process %d\n",GetLastError()); /* toujours 12 */
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
C:\Win16App\MinGW\bin\gcc.exe  error = 0
error = 0
init process 12
saving process 12
error = 12
terminate called after throwing an instance of 'elm::system::SystemException'

This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.

D:\Lecourt\workspace\elm\test>gcc.exe : no input files

Si je suis les codes d'erreur, j'ai un problème sur la ligne "process = new Process(pi);"n mais avant d'entrer dans Process(void*).

Erreur de type entre void* et PROCESS_INFORMATION, ou autre ?

[Médinoc]

Peut-on voir ce que ta classe Process fait de sa variable membre?

Edit: J'ai pigé! Tu dois copier la structure PROCESS_INFORMATION, car elle ne sera pas valide quand la fonction retournera.

[ManusDei]

La classe Process l'utilise dans ce genre de cas (partie pas encore déboguée):

Code C++ :

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

1
2
3
4
void Process::kill(void) {
	if(::TerminateProcess(reinterpret_cast<PROCESS_INFORMATION&>(pi).hProcess, 0) == 0)
		throw SystemException(errno, "process kill");
}

Je débute en C++ (c'était pas prévu le C++), il me semblait que "new Class(bidule)" faisait systématiquement appel au constructeur de copie.

J'ai essayé ça (qui ressemble beaucoup à ta signature )

Code :

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

1
2
3
4
5
6
7
8
Process::Process(void* _pi){
	printf("init process %d\n",GetLastError());
	reinterpret_cast<PROCESS_INFORMATION&>(pi).hProcess=reinterpret_cast<PROCESS_INFORMATION&>(_pi).hProcess;
	reinterpret_cast<PROCESS_INFORMATION&>(pi).hThread=reinterpret_cast<PROCESS_INFORMATION&>(_pi).hThread;
	reinterpret_cast<PROCESS_INFORMATION&>(pi).dwProcessId=reinterpret_cast<PROCESS_INFORMATION&>(_pi).dwProcessId;
	reinterpret_cast<PROCESS_INFORMATION&>(pi).dwThreadId=reinterpret_cast<PROCESS_INFORMATION&>(_pi).dwThreadId;
	printf("saving process %d\n",GetLastError());
}

qui plante à l'exécution (je tente d'accéder à l'adresse mémoire 0x00000000, pas de bol), je ne dois pas avoir copié.

Je ne sais pas trop comment faire.

Edit : d'après le lien de ce post, j'initialise mal mes structures windows. Mais je vois pas très bien comment initialiser tout ça correctement, j'ai un peu du mal avec la doc msdn, que je trouve un peu légère.

[Médinoc]

On peut faire simple:

Code C++ :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
class Process
{
private:
	PROCESS_INFORMATION m_pi;
public:
	Process(PROCESS_INFORMATION const * pcpi);
};
 
 
Process::Process(PROCESS_INFORMATION const* pcpi)
{
	m_pi = *pcpi;
}

Ou si tu tiens à passer par des void*:

Code C++ :

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
class Process
{
private:
	void* m_pvpi;
public:
	Process(void const* pcvpi);
	~Process();
};
 
 
Process::Process(void const* pcvpi)
{
	//Création de la structure membre
	PROCESS_INFORMATION * ppi = new PROCESS_INFORMATION;
	m_pvpi = ppi;
 
	//Copie du contenu
	PROCESS_INFORMATION const *pcpi = static_cast< PROCESS_INFORMATION const * >(pcvpi);
	*ppi = *pcpi;
}
 
Process::~Process()
{
	PROCESS_INFORMATION *ppi = static_cast< PROCESS_INFORMATION * >(m_pvpi);
	delete ppi;
	m_pvpi = 0;
}

[ManusDei]

Un peu tard, mais ça semble fonctionner (je dis semble parce que j'ai un autre problème, mais je pense que c'est lié à autre chose).

Et je vais me pencher dessus avant de demander de l'aide.

Merci.