Discussion :

[dmichel]

Bonjour a tous,

voila, j'ai cree un petit parser de commande, modeste et pas parfait mais suffisant pour mes besoins....

il fonctionne correctement mais cree un bug assez bizarre : les allocations memoires dynamiques, que j'effectue apres, plantent.
Les variables utilisees pour ces allocations n'ont apparement aucun lien avec le parser. Cependant, si j'ote le parser, tous va bien.... bizarre

encore plus bizarre, le code fonctionne parfaitement sous linux. sous windows avec devcpp..bug.

voici un code minimal qui demontre le probleme

merci d'avance pour votre aides

Dav

[Thierry Chappuis]

Bonjour a tous,

voila, j'ai cree un petit parser de commande, modeste et pas parfait mais suffisant pour mes besoins....

il fonctionne correctement mais cree un bug assez bizarre : les allocations memoires dynamiques, que j'effectue apres, plantent.
Les variables utilisees pour ces allocations n'ont apparement aucun lien avec le parser. Cependant, si j'ote le parser, tous va bien.... bizarre

encore plus bizarre, le code fonctionne parfaitement sous linux. sous windows avec devcpp..bug.

voici un code minimal qui demontre le probleme

merci d'avance pour votre aides

Dav

Salut,

Je n'ai pas regardé en détail, mais c'est pas beau à voir. Outre l'inclusion de l'en-tête iostream qui n'existe pas en C et les casts de calloc() qui sont déconseillés, je vois:

• Des variables globales: en as-tu réellement besoin?
• Que fait ta fonction allocationmem(), à part générer des trous de mémoire, parce que là, je vois pas? ())

De manière générale, tu ne vérifies jamais la valeur de retour de tes fonctions d'allocation dynamique. C'est une grosse erreur, tu devrais toujours le faire...

Thierry

[souviron34]

et c'est passé à la compile ça ???? sous Linux ????

je te crois pas...

Au moins 2 énormités qui ne passent pas :

Code :

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

1
2
3
4
 
 Opt_name[0] = "-h";
 Opt_name[1] = "-c";
 Opt_name[2] = "-p";

et

Code :

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

1
2
 
           Opt_data[j] = cmd_v[i+k+1];

Les chaînes de caractères ne se copient pas comme ça, en C...

Il faut utiliser un strcpy, ou un strdup...

Comme tu as déjà alloué la mémoire, ça sera strcpy :

Code :

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

1
2
3
4
 
strcpy( Opt_name[0], "-h");
strcpy( Opt_name[1], "-c");
strcpy( Opt_name[2], "-p");

et

Code :

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

1
2
 
           strcpy (Opt_data[j], cmd_v[i+k+1]);

[dmichel]

oups...j'avais pa fais gaffe a l'en-tête iostream...

mais a part ca...oui le code est assez brouillon avec des variable globales, etc...
mais ca n'explique pas le probleme ! La fonction allocationmem() ne fait rien en effet....car c'est juste un example !

Pourquoi est-ce que ca plante avec le parser ? pourquoi ca passe sans le parser?

sinon, ca compile sans problemes sous linux (as-tu essaye au moins ?)
et je vois pas le probleme avec

Code :

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

Opt_name[0] = "-h"

Du moment que Opt_name est definie comme

Code :

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

 char **Opt_name

[Trap D]

Il n'y a aucun problème à faire

Code :

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

Opt_name[0] = "-h"

sauf que tu as une fuite de mémoire puisqu'avant tu as alloué de la mémoire à Opt_name[0].
J'ai regardé ton code, à quoi sert cette fonction

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
void allocationmem(int bla)
{
 
int *gb_type,*gb_line,*gb_fill,*gb_trans,*gb_orient;
int *gb_type_old,*gb_line_old,*gb_fill_old,*gb_trans_old,*gb_orient_old;
int *gb_type_left,*gb_type_right;
 
printf("allocation...\n");
 
       gb_type = (int*)calloc(bla,sizeof(int));
       gb_fill = (int*)calloc(bla,sizeof(int));
       gb_line = (int*)calloc(bla,sizeof(int));
       gb_trans = (int*)calloc(bla,sizeof(int));
       gb_orient = (int*)calloc(bla,sizeof(int));
       gb_type_old = (int*)calloc(bla,sizeof(int));
       gb_fill_old = (int*)calloc(bla,sizeof(int));
       gb_line_old = (int*)calloc(bla,sizeof(int));
       gb_trans_old = (int*)calloc(bla,sizeof(int));
       gb_orient_old = (int*)calloc(bla,sizeof(int));
       gb_type_left = (int*)calloc(bla,sizeof(int));
       gb_type_right = (int*)calloc(bla,sizeof(int));
 
printf("allocation terminated\n");
 
}/*end function*/

Puisque tu alloues des variables qui ne sont pas utilisées ensuite ?

[souviron34]

Il n'y a aucun problème à faire

Code :

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

Opt_name[0] = "-h"

EN C ?????????????

que je sache "-h" est une chaîne de caractères, pas un caractère..

Et en C, la copie/assignation de chaîne se fait par strcpy..