Discussion :

[ManusDei]

Bonjour,

j'ai un petit problème avec mon programme.
J'ai une fonction qui prend en entrée un int, à partir de cet int je créé une chaîne (t + l'int).
Jusque là pas de problème, j'affiche un résultat correct avec le printf.

Par contre, quand je veux rentrer cette chaîne dans le champ identificateur de ma structure Objet, ça ne marche pas, j'ai un seg fault.

J'imagine que c'est un problème de pointeur mal fait, mais je ne trouve pas la solution.

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
 
struct Objet{
	char identificateur[3];
	char objet_content[127];
};
typedef struct Objet Objet;
 
int threadProd(int num){
	int period=100; /* en millisecondes */
	char type_objet[3];
	char* trame;
	Objet* info; 
	char* num2;
 
	sprintf(type_objet,"t%d\0",num);
	printf("%s\n",type_objet);
	sprintf(info->identificateur),"%s",type_objet);
	printf("%s\n and pid = %d\n",info->identificateur,getpid());
	sprintf(info->objet_content,"25\0");
 
	sprintf(trame,"%s%s",info->identificateur,info->objet_content);
	printf("valeur copiee %s\n",trame);
//	write(fd[num][1],trame,4);
//	sleep(period);
 
	printf("valeur copiee %s\n",trame);
	return 0;
}

Merci pour votre aide.

Edit : Si je met

Code :

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

sprintf(info->identificateur,"%s",type_objet);

en commentaire, le printf suivant s'affiche et j'ai le seg fault après. Donc mon bug viendrait plus tard.

[mala92]

On va oublier la faute de frappe dans sprintf (sinon ça compilerait pas, je pense).
Tu ne nous as pas tout mis, j'espère !!!
Je ne vois pas les malloc !!!
Et num fait 1 chiffre ?

[ManusDei]

Oui, num fait un chiffre, et j'avais oublié le malloc (la honte).
Je met le code complet en dessous.
L'idée de base est simple, j'ai un thread "père" qui va créer des fils, ceux-ci vont envoyer une information via un pipe au père, num sert à différencier les fils (et je ne peux pas utiliser le pid pour ça).

Sauf que pour l'instant, j'ai du mal à créer ma chaîne de caractères à mettre dans le pipe (à terme le père envoie le tout sur le réseau, donc je commence à rassembler toute l'information sur un char*).
Pour l'instant la partie pipe est en commentaire, je m'en occuperais quand je construirais mon char* 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
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
struct Objet{
	char identificateur[3];
	char objet_content[127];
};
typedef struct Objet Objet;
 
#define PROCESS		10
 
int threadProd(int num){
	int period=100; /* en millisecondes */
	char type_objet[3];
	char* trame;
	Objet* info = malloc(sizeof(struct Objet)); 
	char* num2;
 
	sprintf(type_objet,"t%d\0",num);
	printf("%s\n",type_objet);
	sprintf(info->identificateur,"%s",type_objet);
//	strcpy(info->identificateur,type_objet);
 
	printf("%s and pid = %d\n",info->identificateur,getpid());
	sprintf(info->objet_content,"25\0");
 
	sprintf(trame,"%s%s",info->identificateur,info->objet_content);
	printf("valeur copiee %s\n",trame);
//	write(fd[num][1],trame,4);
//	sleep(period);
 
	printf("valeur copiee %s\n",trame);
	return 0;
}
 
int main()
{
    pid_t pid;
 
    for(int i=0;i<PROCESS;i++)
    	pipe(fd[PROCESS]);
 
	//printf("coucou\n");	
	while (process < PROCESS) {
 		pid = fork();
 
 		// Erreur à la creation du fils
		if (pid < 0) {
			perror("fork");
			exit(EXIT_FAILURE); 
		}
		// Execution du Producteur
		else if (!pid) {
			pid = getpid();
			//printf("fils cree\n");
			close(fd[process][0]);
			threadProd(process);
			exit(EXIT_SUCCESS);
		}
 
		++process;
 
	}
 
	// execution du Producteur couche MAC
	for(int i=0;i<PROCESS;i++){
//		close(fd[PROCESS][1]);	
//		threadMACProd();
	}
 
 
 
	return EXIT_SUCCESS;
}

[mala92]

Ton programme plante à quelle ligne ? La 18 ?

Edit : et le malloc de trame, il est où ? (phi1981 m'a grillé )

[phi1981]

alors je suis pas vraiment reveillé, mais je ne vois pas l'allocation d'un buffer pour la trame avant la ligne 24 :
sprintf(trame,"%s%s",info->identificateur,info->objet_content);

du coup, sprintf va écrire n'importe où.

et sinon, l'idée de mettre des \0 à la fin de tes chaines de caractères, c'est quoi le but ??? parce que si c'est juste pour insérer un 0 à la fin, le compilateur le fait très bien.

[ManusDei]

Je me suis un peu emballé (c'est mal de s'énerver tout seul devant sa machine ).

Alors, là je n'ai plus de seg fault, mais

Code :

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

printf("valeur copiee %s\n",trame);

ne s'affiche pas (ligne 24-25 sur le post précédent), donc j'ignore ce que j'ai dans mon char* trame.

Merci pour votre aide.

Edit : je teste avec un malloc pour le char* trame, mais pourquoi un malloc pour ça ? le \0 n'était pas là au début, mais à force de tenter de trouver des solutions... il est arrivé

Edit2: avec le malloc sur trame ça fonctionne, merci. Pourquoi ?

[mala92]

Edit : je teste avec un malloc pour le char* trame, mais pourquoi un malloc pour ça ? l

Tu déclares un char*, donc il faut l'allouer.

Au fait, quelle est la cible de ton programme ? Il me semble que j'avais vu dans le forum emploi que tu fais des études dans l'embarqué. Il me semble que les allocations dynamiques ne sont pas conseillées dans ce domaine.

[phi1981]

La grande question c'est plutôt "pourquoi ça marcherait sans malloc ?".
Le pauvre sprintf ne fait rien de très intelligent : il compose dans un buffer une chaine de caractère suivant un format que tu lui as passé en paramètre.
Le problème, c'est que lorsque tu déclares juste char *trame sur la pile comme tu l'as fait, trame vaut n'importe quoi (je suis étonné que ton compilateur ne t'ai pas grondé d'ailleurs).
du coup sprintf, bête mais discipliné, va composer ta chaine n'importe où en mémoire.
quand tu as de la chance, ça plante. quand tu n'as pas de chance, ça modifie qqchose d'autre et ça cause un bug très difficile à repérer.

Edit : Grillé par mala92

[ManusDei]

Au fait, quelle est la cible de ton programme ? Il me semble que j'avais vu dans le forum emploi que tu fais des études dans l'embarqué. Il me semble que les allocations dynamiques ne sont pas conseillées dans ce domaine.

C'est pour le cours de réseau, avec le protocole WorldFIP (donc pas directement de l'embarqué).

Pourtant, je n'alloue par toujours les char*. Je ne dirais pas que c'est la première fois, mais probablement pas loin.
Et il me semble que ça marchait (ça fait un moment que je n'avais pas fait de manipulation de char* en C).