Discussion :

[Korben-Dallas]

Bonjour,

Mon code ci-dessous echoue toujours et je ne comprends pas pourquoi.
Qu'en pensez-vous ?
Merci

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
 
char* DateDernierReboot=malloc(sizeof(char*));
char* DateDernierRebootTemoin=malloc(sizeof(char*));
 
 
...
...
...
...
...
 
 
while(fgets(s_line, BUFSIZ, p_file))
{
	/* Suppression du caractère de fin de ligne s'il est present. */
	char * p = strchr(s_line, '\n');
	if(p != NULL)
	{
	 *p = 0;
	}
 
	/* Comparaison */
	if(sscanf(s_line , "Statistiques depuis %s %s %s", Date , Heure, TypeHeure)==2)
	{
		/* La ligne a ete trouvee, on sort. */
		done = 1;
		break;
	}
 
	memset (s_line, 0, sizeof (s_line));
}
 
 
...
...
...
...
...
 
 
while(fgets(s_lineTemoin, BUFSIZ, p_fileTemoin))
{
	/* Suppression du caractère de fin de ligne s'il est present. */
	char * p = strchr(s_lineTemoin, '\n');
	if(p != NULL)
	{
	 *p = 0;
	}
 
	/* Comparaison */
	if(sscanf(s_lineTemoin , "Statistiques depuis %s %s %s", DateTemoin , HeureTemoin, TypeHeureTemoin)==2)
	{
		/* La ligne a ete trouvee, on sort. */
		done = 1;
		break;
	}
 
	memset (s_lineTemoin, 0, sizeof (s_lineTemoin));
}
 
 
...
...
...
...
...
 
 
sprintf(DateDernierReboot, "%s %s %s", Date, Heure, TypeHeure);
sprintf(DateDernierRebootTemoin, "%s %s %s", DateTemoin, HeureTemoin, TypeHeureTemoin);
 
if(strcmp(DateDernierReboot, DateDernierRebootTemoin)==0)
{
	// Les 2 fichiers sont identiques
	// Le PC n'a pas redémarré
	INFO1("------------------------------------------------------");
	INFO1("|     Redemarrage du poste de travail necessaire     |");
	INFO1("------------------------------------------------------");
}
else
{
	// Les 2 fichiers sont différents
	// Le PC a donc redémarré
	Traitement();
}
 
...
...
...
...
...

Le pb est que si mes variables ainsi récupérées valent toutes les 2 "5/19/2011 1:32 PM" (par exemple)
La commande if(strcmp(Date.....)==0) est toujours différent de 0 et donc fait le traitement alors que ça ne devrait pas être le cas.
Je ne comprends pas ce qui se passe.
Merci pour votre aide.

[diogene]

Code :

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

1
2
char* DateDernierReboot=malloc(sizeof(char*));
char* DateDernierRebootTemoin=malloc(sizeof(char*));

Ces deux lignes sont fausses : typiquement on doit avoir

Code :

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

type * p = malloc(sizeof(type)) // et pas sizeof(type*))

De plus, et surtout elles n'allouent de la mémoire que pour stocker un pointeur ce qui sera insuffisant pour stocker les dates/heures/typeheures
La taille minimum à allouer (dans le premier cas) est

Code :

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

char* DateDernierReboot=malloc(strlen(Date)+strlen(Heure)+strlen(TypeHeure)+2+1);

Code :

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

1
2
if(sscanf(s_line , "Statistiques depuis %s %s %s", Date , Heure, TypeHeure)==2)
if(sscanf(s_lineTemoin , "Statistiques depuis %s %s %s", DateTemoin , HeureTemoin, TypeHeureTemoin)==2)

Pourquoi == 2 alors qu'on demande 3 conversions ?

[Korben-Dallas]

OK Merci
J'ai corrigé mon programme au niveau des malloc ainsi que pour le sscanf()
Ca fonctionne.

Par contre je ne comprends pas le "+2+1" pour le malloc()
Qd au "==2" au lieu de "==3" c'est maintenant corrigé, c'était un oubli de ma part car au début je n'utilisait que 2 variables et j'en ai rajoutée une par la suite.

[Korben-Dallas]

Après différents tests, ceci fonctionne si les 2 variables sont identiques, le programme affiche bien qu'il faut redémarrer le PC mais si elles sont différentes, là ça plante. Le pb s'arrête sans lancer le traitement.

[jeroman]

Par contre je ne comprends pas le "+2+1" pour le malloc()

Deux espaces + le caractère '\0' de fin de chaine :

"5/19/2011 1:32 PM"

Code :

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

sprintf(DateDernierReboot, "%s %s %s", Date, Heure, TypeHeure);

[Korben-Dallas]

ha oui ok ok
j'avais pas pensé à ça

[Korben-Dallas]

Pour mon pb de plantage, j'ai trouvé d'où ça vient mais je n'arrive pas à résoudre le pb.
En fait, si

Code :

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

1
2
 
DateDernierReboot="5/19/2011 1:32 PM"

et

Code :

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

1
2
 
DateDernierRebootTemoin="5/19/2011 11:32 AM"

donc avec minimum 1 caractère supplémentaire, là ça plante royalement.
Même si je met le malloc des 2 variables à 20 (nbre maximum de caractères), là ça continu de planter.
La comparaison ne peut se faire que sur 2 variables ayant le même nombre de caractères mais ici ce n'est pas forcément le cas.
Que faire docteur ?

Je penses qu'il est donc nécessaire d'écrire des caractères "bidons" dans les variables afin que celles-ci fassent exactement le même nombre de caractères fixes (par exemple 25 caractères) comme ça la comparaison se fera sur les 25 caractères sans pb (il me semble) mais je ne vois pas comment injecters ces fameux caractères bidon pour arriver à un total de 25 carcatères (25 étant un nombre pris au hasard)
Qu'en pensez-vous ?