Discussion :

[kolodz]

Bonjour,

Je viens de débugé une DLL écrite en C pour une application Java. Cette dll provoquait des crash (non systématique) de l'application Java lors de l'initialisation :

#
# An unexpected error has been detected by HotSpot Virtual Machine:
#
# EXCEPTION_ACCESS_VIOLATION (0xc0000005) at pc=0x7706e3be, pid=476, tid=4880
#
# Java VM: Java HotSpot(TM) Client VM (1.5.0_13-b05 mixed mode)
# Problematic frame:
# C [ntdll.dll+0x2e3be]
#

Après un peu de temps, la cause du crash s'est rapprochée des lignes suivantes :

Code :

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

1
2
3
4
char * sChemin = NULL;
char nomIni[250]="\0"; 
sChemin = sC_GEN_AddSepToDirectory(nomFichier);
strcat(sChemin,"MONFICHIER.INI");

En sachant que :

Code :

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

1
2
3
FCT_EXP char * sC_GEN_AddSepToDirectory(char * _sChemin) {
// L'implémentation de la méthode ...
}

En comparant avec le reste du code C à ma disposition dans cette DLL, j'ai mis le code suivant :

Code :

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

1
2
3
4
char sChemin[250] = "\0";
strcpy(sChemin,sC_GEN_AddSepToDirectory(nomFichier));
printf(sChemin);
strcat(sChemin,"MONFICHIER.INI");

Après une série d'essai, je n'ai plus de crash de l'application avec la DLL corrigé.

Ma question est pourquoi la DLL provoquait un crash ?

Cordialement,
Patrick Kolodziejczyk.

[gerald3d]

Bonjour.

La fonction FCT_EXP char * sC_GEN_AddSepToDirectory(char * _sChemin) allouait-elle correctement l'espace nécessaire pour la chaîne retournée ?

[kolodz]

Que veux tu dire par "correctement " ?

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
FCT_EXP char * sC_GEN_AddSepToDirectory(char * _sChemin) 
{
	char * sChemin;
	int taille = 0;
 
	if(_sChemin == NULL) 	
	{
		_sChemin = malloc(sizeof(char));	 
		_sChemin="\0";
	}
 
	sChemin = strrchr(  _sChemin, XGN_ANTISLASH); 
	/* Il n'y a pas de sép à la fin du chemin donc on le met */
	if(!sChemin || strlen(sChemin) > 1)
	{
		taille = strlen(_sChemin);
		sChemin = malloc((taille+2)*sizeof(char));	 
		strncpy(sChemin, _sChemin, taille);
		sChemin[taille] = XGN_ANTISLASH;
		sChemin[taille+1] = '\0';
	}
	/* Il y a un sép à la fin du chemin c'est bon */
	else
	{
		sChemin =  _sChemin;
	}
 
 	return sChemin;
 }

J'ai bien le malloc pour la chaine de retour, mais je ne pense pas que c'est cela dont tu parle...

Cordialement,
Patrick Kolodziejczyk.

[Obsidian]

Bonsoir,

Que veux tu dire par "correctement " ?

Il y a une énorme erreur dans le passage suivant :

Code :

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

1
2
3
4
5
	if(_sChemin == NULL) 	
	{
		_sChemin = malloc(sizeof(char));	 
		_sChemin="\0";
	}

Tu ne déposes pas un caractère nul dans l'espace fraîchement alloué mais tu détournes le pointeur pour le faire pointer une autre chaîne, en lecture seule. Ce faisant, tu perds la valeur de retour renvoyée par malloc() et tu provoques une fuite de mémoire.

Par la suite, si ta condition est fausse, tu renvoies directement cette adresse à l'utilisateur qui, lui, pense qu'elle a été allouée. S'il essaie d'y écrire, cela plantera. S'il essaie de la libérer avec free(), cela plantera aussi.

[kolodz]

Je n'ai pas bien compris ton explication. Mais globalement, tout les appels à cette méthodes peuvent provoquer un crash.
La modification que j'ai fait n'est donc qu'une rustine...

Si je reprends ton explication :

Code :

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

_sChemin = malloc(sizeof(char));

On réalisé une allocation mémoire et place un mon pointeur de char dans la variable.

Code :

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

_sChemin="\0";

On place le pointeur du caractère null dans la variable.
Ce qui fait qu'on perd la référence de ce qu'on vient d'alloué en mémoire. (Fuite mémoire de la taille d'un char)

Cependant, pour cette partie :

Par la suite, si ta condition est fausse, tu renvoies directement cette adresse à l'utilisateur qui, lui, pense qu'elle a été allouée.

Si _sChemin a été initialé avec une caractère nul, alors sChemin sera forcément un pointeur nul. Car strrchr, il ne trouvera pas occurrence cherché.

Code :

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

sChemin = strrchr(  _sChemin, XGN_ANTISLASH);

Ce qui fait que je passe dans ce morceau de code :

Code :

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

1
2
3
4
5
taille = strlen(_sChemin);
sChemin = malloc((taille+2)*sizeof(char));	 
strncpy(sChemin, _sChemin, taille);
sChemin[taille] = XGN_ANTISLASH;
sChemin[taille+1] = '\0';

Dans ce cas là, strlen(_sChemin) retournera 0.
On aura donc sChemin ayant le contenu suivant {["\"],["\0"]} (ma synthase ressemble plus à du json que du c non ?)

Par contre, si on donne un pointeur de char non null. qui finit par un XGN_ANTISLASH et qui est en plus en read only. Là ça va crashé, non ?

Cordialement,
Patrick Kolodziejczyk.

[gerald3d]

Je ferais plutôt :

Code :

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

*_sChemin='\0';

Enfin je crois...