pb realloc char**

**le mage tophinus** · 10/05/2006, 11h53

salut à tous,

Je souhaite utiliser (sous linux) l'appel execl définit comme suit :

Code :

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

1
2
 
int execv(const char *application, const char * argv[]);

Comme je dois faire cet appel un nombre important de fois, je le place dans une procédure "iptables_call" à laquelle je transmet les paramètres :

Code :

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

1
2
3
4
5
6
7
8
9
 
int iptables_call(const char *);
 
int main()
{
...
if (iptables_call("-t filter -F")) return(1);
...
}

Donc le problème c'est que j'ai en entrée une chaîne de caractères séparées par des espaces alors qu'il faut que je transmette une série de chaîne de caractères sans espace. Donc voilà mon idée :
1) comme la chaîne en entrée est constante, je la copie dans une zone mémoire allouée dynamiquement (variable copied_chain)
2) Je boucle maintenant sur la longueur de copied_chain et à chaque fois que je trouve un espace :
a) je remplace l'espace par 0 pour simuler la fin de la chaîne
b) j'utilise un char ** list_parameters pour le passage de la liste à execl (le 2ème paramètre) et donc quand je tombe sur un espace dans copied_chain, je realloc le char** pour lui dire qu'il y a un pointeur en plus et faire pointer vers le caractère suivant l'espace
3) je finis par un pointeur NULL pour clotûrer la liste comme préciser dans mes références pour execl
4) je fais en gros mon appel execl.

Maintenant voilà le code :

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
int iptables_call(const char *iptables_parameters)
{
  pid_t pid_son;
  char *copied_chain;
  int long_param, nb_param,i;
  char **list_pointers, **new_list_pointers;
  char *iptables_chain="/sbin/iptables";
  
 // copie de la chaine
  long_param=strlen(iptables_parameters);
  copied_chain=(char *)(malloc(long_param*sizeof(char)));
  if (copied_chain==NULL) {
    fprintf(stderr,"Error : Not enough memory !\n");
    return(1);
  }
  strcpy(copied_chain,iptables_parameters);
 // il faut initialiser la chaine avec "/sbin/iptables" d'abord en sus, et le début de copied_chain
  nb_param=2;
  list_pointers=(char **)(malloc(nb_param*sizeof(char *)));
  if (list_pointers==NULL) {
    fprintf(stderr,"Error : Not enough memory !\n");
    return(1);
  }
  list_pointers[0]=iptables_chain;
  list_pointers[1]=copied_chain;
  i=0;
 //boucle sur la longueur
  while (i<long_param) {
 // test si c'est un espace
    if (copied_chain[i]==' ') {
      copied_chain[i]=0;
      nb_param++;
 // affichage de debogage
      fprintf(stdout,"%s\n",iptables_chain);
 // realloc
      new_list_pointers=(char **)(realloc(list_pointers,nb_param*sizeof(char *)));
 // affichage de debogage
      fprintf(stdout,"%s\n",list_pointers[1]);
      if (new_list_pointers==NULL) {
    fprintf(stderr,"Error : Not enough memory !\n");
    return(1);
      }
      list_pointers=new_list_pointers;
 // initialiser le nouveau pointeur
      list_pointers[nb_param-1]=copied_chain+i+1;
    }
    i++;
  }
 // sortie temporaire en attendant que la chaîne soit bien initialisée
  fprintf(stdout,"ici\n");
  return(1);
   // add NULL at the end
  nb_param++;
  new_list_pointers=(char **)(realloc(list_pointers,nb_param*sizeof(char *)));
  if (new_list_pointers==NULL) {
    fprintf(stderr,"Error : Not enough memory !\n");
    return(1);
  }
  list_pointers=new_list_pointers;
  list_pointers[nb_param-1]=NULL;
 // mise en place de l'appel execl
  pid_son=fork();
  if (pid_son==-1) {
    fprintf(stderr,"Error : New process cannot be created !\n");
    return(1);
  }
  if (!pid_son) {
     /* prototype : int execv(const char *application, const char * argv[]); */
    execv("/sbin/iptables",list_pointers);
  }
  else wait(NULL);
  free(list_pointers);
  free(copied_chain);
  return(0);
}

Mon compilateur ne me donne aucune erreur ni warning mais quand je lance le programme, j'ai ça comme affichage :

/sbin/iptables
-t
/sbin/iptables
*** glibc detected *** free(): invalid next size (normal): 0x0804d030 ***
Abandon

Donc sur le premier espace, ça fontionne vu qu'il affiche "-t" mais il bloque sur le deuxième realloc.
Là j'avoue que je sèche complètement, est-ce que quelqu'un a une idée ? Une simple explication de mon erreur me suffirait.
Merci d'avance

**je®ome** · 10/05/2006, 12h36

Je ne suis pas sûr de ce que j'avance, mais je pense qu0il bloque sur ton free.
Sinon, tu n'aurais ce message d'erreur.

Code :

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

1
2
3
 
char **list_pointers; 
free(list_pointers);

essaies un peu de faire

Code :

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

1
2
 
 free(*list_pointers);

ou mets le en commentaire pour tester.

Comme dis, je ne suis pas sûr ...

**fearyourself** · 10/05/2006, 13h49

Quelques remarques:

Code :

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

copied_chain=(char *)(malloc(long_param*sizeof(char)));

en C devrait être écrit :

Code :

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

copied_chain= malloc(long_param);

Pas de cast et sizeof(char) vaut 1...

Pareil pour le realloc...

Code :

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

copied_chain[i] = 0;

Pourquoi utiliser la valeur directement, le code ascii existe :

Code :

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

copied_chain[i] = '\0';

Sinon, es-tu sûr d'avoir mis les bonnes inclusions pour chaque fonction que tu
utilises parce ton code fonctionne chez moi...

[EDIT]
J'ai mis:

Code :

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

1
2
3
4
5
6
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include <sys/types.h>
#include <unistd.h>
#include <sys/wait.h>

[/EDIT]

Jc

**le mage tophinus** · 10/05/2006, 15h07

Je ne suis pas sûr de ce que j'avance, mais je pense qu0il bloque sur ton free

Il ne bloque pas au niveau du free, il bloque bien avant. D'ailleurs le free dans cette version du programme n'est même pas atteint car il y a un return avant. Merci quand-même

Sinon, es-tu sûr d'avoir mis les bonnes inclusions pour chaque fonction que tu utilises parce ton code fonctionne chez moi...

J'ai vérifié mes inclusions et il n'y a que sys/types.h que je n'utilisais pas mais même avec ça ne fonctionne toujours pas...

Je rajoute quelques précisions au cas où : c'est une partie du programme, il y a une autre partie qui s'occupe au préalable de lire un fichier de confiuration et qui fait donc quelques malloc et realloc. Existerait-il une limite ??? Par contre, ce que je suis sûr c'est que j'ai encore suffisamment de mémoire...
Mais effectivement, je viens de tester cette partie du programme seule et ça fonctionne... je comprends pas pourquoi j'ai pas un segfault dans ce cas ?

Bon, à priori mon code est bon donc je réitère ma question : existe-t-il une limite du nombre de malloc/realloc ???

**fearyourself** · 10/05/2006, 15h24

Envoyé par le mage tophinus

Bon, à priori mon code est bon donc je réitère ma question : existe-t-il une limite du nombre de malloc/realloc ???

Non, pas que je sache, pour tester:

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
#include <stdio.h>
 
#include <stdlib.h>
 
int main()
{
        int i,j;
        char *ptr,*ptr2;
 
        for(i=0;i<10000;i++)
                {
                ptr = malloc(2);
 
                if(ptr==NULL)
                        {
                        printf("Erreur de malloc\n");
                        return 1;
                        }
 
                for(j=0;j<10000;j++)
                        {
                        ptr2 = realloc(ptr,3+j);
                        if(ptr2==NULL)
                                {
                                printf("Erreur de realloc\n");
                                return 1;
                                }
                        ptr = ptr2;
                        }
 
                free(ptr);
                }
 
        return 0;
}

Et ça fonctionne très bien...

Jc

**crocodilex** · 10/05/2006, 15h38

Envoyé par fearyourself

Non, pas que je sache, pour tester:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <stdio.h>

#include <stdlib.h>

int main()
{
        int i,j;
        char *ptr,*ptr2;
        [...]
                         ptr = ptr;
        [...]

        return 0;
}

Et ça fonctionne très bien...

Jc

Petite erreur de frappe je suppose....

Code :

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

ptr = ptr2;

**le mage tophinus** · 10/05/2006, 15h39

effectivement, je n'ai pas d'erreur avec ton nouveau code, donc ce n'est pas un problème de nombre de malloc/realloc.

Dernière question et après je délesterais : Est-ce que j'initialise bien ma chaîne en faisant ça :

Code :

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

1
2
 
char *iptables_chain="/sbin/iptables";

Y'a pas besoin d'un malloc ici ??? Parce que je vois que ça à priori...

[edit]
même avec la modification de crocodilex, le nouveau code fonctionne toujours
[/edit]

**fearyourself** · 10/05/2006, 15h42

Envoyé par crocodilex

Petite erreur de frappe je suppose....

Code :

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

ptr = ptr2;

Exact, j'ai modifié les indentations lors de la copie, le 2 est passé à la trappe on dirait... Merci

Jc

**le mage tophinus** · 10/05/2006, 16h06

Bon, je crois qu'il y a tout de même un bug dans la partie du char**. Si on essaye uniquement cette partie de code comme on l'a fait tout à l'heure, ça fonctionne à priori au premier passage, mais pas aux suivants. Il suffit de remplacer main par ça :

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
 
int main()
{
  if (iptables_call("-t filter -F")) return(1);
  if (iptables_call("/sbin/iptables -t filter -X")) return(1);
  if (iptables_call("/sbin/iptables -t filter -P INPUT DROP")) return(1);
  if (iptables_call("/sbin/iptables -t filter -P FORWARD DROP")) return(1);
  if (iptables_call("/sbin/iptables -t filter -P OUTPUT DROP")) return(1);
  if (iptables_call("/sbin/iptables -t nat -F")) return(1);
  if (iptables_call("/sbin/iptables -t nat -X")) return(1);
  if (iptables_call("/sbin/iptables -t nat -P PREROUTING ACCEPT")) return(1);
  if (iptables_call("/sbin/iptables -t nat -P POSTROUTING ACCEPT")) return(1);
  if (iptables_call("/sbin/iptables -t nat -P OUTPUT ACCEPT")) return(1);
  if (iptables_call("/sbin/iptables -t mangle -F")) return(1);
  if (iptables_call("/sbin/iptables -t mangle -X")) return(1);
  if (iptables_call("/sbin/iptables -t mangle -P PREROUTING ACCEPT")) return(1);
  if (iptables_call("/sbin/iptables -t mangle -P INPUT ACCEPT")) return(1);
  if (iptables_call("/sbin/iptables -t mangle -P OUTPUT ACCEPT")) return(1);
  if (iptables_call("/sbin/iptables -t mangle -P FORWARD ACCEPT")) return(1);
  if (iptables_call("/sbin/iptables -t mangle -P POSTROUTING ACCEPT")) return(1);
  return(0);
}

Et là le résultat est flagrant (j'évite de tout poster car ça va polluer pour rien). En gros, au premier passage, il m'affiche bien :

/sbin/iptables
-t
/sbin/iptables
-t

Mais aux passages suivants, on obtient ça :

/sbin/iptables
/sbin/iptables
/sbin/iptables
/sbin/iptables
/sbin/iptables
/sbin/iptables

Avec tout de même un nombre proportionnel de lignes en fonction du nombre d'espace dans la chaîne en entrée...
Là

**le mage tophinus** · 10/05/2006, 16h08

Arf, je viens de me rendre compte que je viens de dire une grosse bétise, c'est normal qu'il m'affiche /sbin/iptables .... Désolé

**le mage tophinus** · 10/05/2006, 16h14

Bon finalement, j'ai pas dit tant de bétises que ça. En remplaçant donc mon main par ça :

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
 
int main()
{
  if (iptables_call("-t filter -F")) return(1);
  if (iptables_call("-t filter -X")) return(1);
  if (iptables_call("-t filter -P INPUT DROP")) return(1);
  if (iptables_call("-t filter -P FORWARD DROP")) return(1);
  if (iptables_call("-t filter -P OUTPUT DROP")) return(1);
  if (iptables_call("-t nat -F")) return(1);
  if (iptables_call("-t nat -X")) return(1);
  if (iptables_call("-t nat -P PREROUTING ACCEPT")) return(1);
  if (iptables_call("-t nat -P POSTROUTING ACCEPT")) return(1);
  if (iptables_call("-t nat -P OUTPUT ACCEPT")) return(1);
  if (iptables_call("-t mangle -F")) return(1);
  if (iptables_call("-t mangle -X")) return(1);
  if (iptables_call("-t mangle -P PREROUTING ACCEPT")) return(1);
  if (iptables_call("-t mangle -P INPUT ACCEPT")) return(1);
  if (iptables_call("-t mangle -P OUTPUT ACCEPT")) return(1);
  if (iptables_call("-t mangle -P FORWARD ACCEPT")) return(1);
  if (iptables_call("-t mangle -P POSTROUTING ACCEPT")) return(1);
  return(0);
}

j'obtiens mon erreur classique :

toph@pcfixe:~/prog/firewall$ ./essai
/sbin/iptables
-t
/sbin/iptables
-t
FATAL: Module ip_tables not found.
iptables v1.3.3: can't initialize iptables table `filter': Permission denied (you must be root)
Perhaps iptables or your kernel needs to be upgraded.
/sbin/iptables
-t
/sbin/iptables
-t
FATAL: Module ip_tables not found.
iptables v1.3.3: can't initialize iptables table `filter': Permission denied (you must be root)
Perhaps iptables or your kernel needs to be upgraded.
*** glibc detected *** free(): invalid next size (fast): 0x0804a008 ***
Abandon

Donc il y a bien finalement un problème avec cette partie de code.... Une idée ???

**crocodilex** · 10/05/2006, 16h19

Envoyé par le mage tophinus

iptables v1.3.3: can't initialize iptables table `filter': Permission denied (you must be root)

J'espère que tu as tenu compte de ce message !!!!

**le mage tophinus** · 10/05/2006, 16h46

Envoyé par Crocodilex

J'espère que tu as tenu compte de ce message !!!!

ça n'a rien à voir du tout avec le problème, enfin je pense. Si je le lance en root, j'ai ça comme erreur :

/sbin/iptables
-t
/sbin/iptables
-t
iptables: No chain/target/match by that name
/sbin/iptables
-t
/sbin/iptables
-t
iptables: Too many links
*** glibc detected *** free(): invalid next size (fast): 0x0804a008 ***
Abandon

Je ne le lance pas en root simplement pour éviter des dégâts en cas de mauvaises manipulations.

[edit]
Pour plus de précisions, je préfère redonner la procédure que j'utilise actuellement :

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
 
int iptables_call(const char *iptables_parameters)
{
  pid_t pid_son;
  char *copied_chain;
  int long_param, nb_param,i;
  char **list_pointers, **new_list_pointers;
  char *iptables_chain="/sbin/iptables";
 
  long_param=strlen(iptables_parameters);
  copied_chain=malloc(long_param);
  if (copied_chain==NULL) {
    fprintf(stderr,"Error : Not enough memory !\n");
    return(1);
  }
  strcpy(copied_chain,iptables_parameters);
  nb_param=2;
  list_pointers=malloc(nb_param*sizeof(char *));
  if (list_pointers==NULL) {
    fprintf(stderr,"Error : Not enough memory !\n");
    return(1);
  }
  list_pointers[0]=iptables_chain;
  list_pointers[1]=copied_chain;
  i=0;
  while (i<long_param) {
    if (copied_chain[i]==' ') {
      copied_chain[i]='\0';
      nb_param++;
      fprintf(stdout,"%s\n",iptables_chain);
      new_list_pointers=realloc(list_pointers,nb_param*sizeof(char *));
      fprintf(stdout,"%s\n",list_pointers[1]);
      if (new_list_pointers==NULL) {
    fprintf(stderr,"Error : Not enough memory !\n");
    return(1);
      }
      list_pointers=new_list_pointers;
      list_pointers[nb_param-1]=copied_chain+i+1;
    }
    i++;
  }
  // add NULL at the end
  nb_param++;
  new_list_pointers=realloc(list_pointers,nb_param*sizeof(char *));
  if (new_list_pointers==NULL) {
    fprintf(stderr,"Error : Not enough memory !\n");
    return(1);
  }
  list_pointers=new_list_pointers;
  list_pointers[nb_param-1]=NULL;
  pid_son=fork();
  if (pid_son==-1) {
    fprintf(stderr,"Error : New process cannot be created !\n");
    return(1);
  }
  if (!pid_son) {
    /* prototype : int execv(const char *application, const char * argv[]); */
    execv("/sbin/iptables",list_pointers);
  }
  else wait(NULL);
  free(list_pointers);
  free(copied_chain);
  return(0);
}

[/edit]

**fearyourself** · 10/05/2006, 17h12

Aller, soyons gentils, après avoir planté ma machine (merci d'ailleurs pour ton exemple), j'ai changé le code du main pour iptables ne soit pas content...

Comme on le dit toujours, donnez un exemple simple compilable qui montre le problème (donc ce que tu as fait ici, à part que t'as planté ma machine

) et on pourra trouver la solution (si le temps le permet...)

Ton problème se situe ici:

Code :

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

1
2
3
4
5
6
7
  long_param=strlen(iptables_parameters);
  copied_chain=malloc(long_param);
  if (copied_chain==NULL) {
    fprintf(stderr,"Error : Not enough memory !\n");
    return(1);
  }
  strcpy(copied_chain,iptables_parameters);

Combien de caractères est-ce que tu alloues pour copied_chain?
Combien de caractères est-ce que tu utilises pour copied_chain?

Jc

**le mage tophinus** · 10/05/2006, 17h19

Nom d'un chien, une bétise pareille et je ne l'avais même pas vu... J'ai honte de chez honte

Merci beaucoup du coup de main

Et désolé pour ta machine, je n'aurais jamais pensé qu'un petit code aussi simple pourrait planter une bécane... t'aurais pas des lecteurs en réseaux par hasard ?

Et merci encore

**fearyourself** · 10/05/2006, 17h20

Je bosse en NIS donc je pense que toute modif des tables ip pose un problème, j'aurais dû y penser avant de lancer ton code lol

Jc

pb realloc char**

C

Discussions similaires

Partager

Partager