Bug de la fonction system en C :-(

Version imprimable

Bonjour tout le monde !

En cours, je suis sur des petits projets pour mon BTS... Or l'un d'eux va me rendre zinzin :aie:

Je pense que la commande system () en est la source, mais comment résoudre ce problème ? Je n'ai pas la réponse. C'est pourquoi je viens à vous :D

Mon programme permet de gérer les ordinateurs d'un réseau : il stock dans un fichier les données (adresse IP, nom du poste etc.) et l'une de ses fonctions consiste à effectuer un ping pour connaître l'état de la machine (en ligne / hors ligne).

Pour cela, j'utilise la fonction system... Mais quelque chose de pas cool se produit ; explications :

Voici mon code source :
Code:

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 void tester (char* fichierIP, char* fichierOK, char* fichierNOK) { Ordi* TabOrdi = chargerFichier(fichierIP); Ordi TabOrdiok[100]; Ordi TabOrdiNok[100]; int i = 0; int resultat = 0; char cmd[50]; for (i=0;i <= nbreElement;i++) // nbreElement contient le nombre d'enregistrement du fichierIP { printf("---------------\n"); // Je crée ma commande de ping et la stock dans cmd sprintf(cmd, "ping -n 1 %d.%d.%d.%d > NULL", TabOrdi[i].IPOrdi.ip1, TabOrdi[i].IPOrdi.ip2, TabOrdi[i].IPOrdi.ip3, TabOrdi[i].IPOrdi.ip4); // Je test printf("cmds : %s\n", cmd); // J'exécute ma commande et récupère dans "resultat" le code retourné par system() resultat = system (cmd); // J'affiche ce code résultat (pour voir si tout fonctionne) printf("resultat : %d\n", resultat); } }
Et là, voici le résultat ! Pas très jolie ...

Citation:

---------------
cmds : ping -n 1 123.12.123.10 > NULL
resultat : 0
---------------
cmds : ping -n 1 4980802.1999633846.1988581918.48 > NULL
resultat : 1
---------------
cmds : ping -n 1 0.0.2679004.1988581993 > NULL
resultat : 1
---------------
cmds : ping -n 1 0.0.0.0 > NULL
resultat : 1
---------------
cmds : ping -n 1 1988583728.88.1988583756.0 > NULL
resultat : 1
---------------
cmds : ping -n 1 0.2678220.0.0 > NULL
resultat : 1
---------------
cmds : ping -n 1 0.0.0.0 > NULL
resultat : 1

Comme on peut voir mes adresses IP sont pas très... Bref ! Y'a un souci !

Je commente la ligne où je fais mon system ()
Code:

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 void tester (char* fichierIP, char* fichierOK, char* fichierNOK) { Ordi* TabOrdi = chargerFichier(fichierIP); Ordi TabOrdiok[100]; Ordi TabOrdiNok[100]; int i = 0; int resultat = 0; char cmd[50]; for (i=0;i <= nbreElement;i++) // nbreElement contient le nombre d'enregistrement du fichierIP { printf("---------------\n"); // Je crée ma commande de ping et la stock dans cmd sprintf(cmd, "ping -n 1 %d.%d.%d.%d > NULL", TabOrdi[i].IPOrdi.ip1, TabOrdi[i].IPOrdi.ip2, TabOrdi[i].IPOrdi.ip3, TabOrdi[i].IPOrdi.ip4); // Je test printf("cmds : %s\n", cmd); // J'exécute ma commande et récupère dans "resultat" le code retourné par system() //resultat = system (cmd); // J'affiche ce code résultat (pour voir si tout fonctionne) printf("resultat : %d\n", resultat); } }
Et là, miracle tout s'affiche correctement ! Mais je n'ai pas de ping de fait !

Citation:

---------------
cmds : ping -n 1 123.12.123.10 > NULL
resultat : 0
---------------
cmds : ping -n 1 192.168.12.3 > NULL
resultat : 0
---------------
cmds : ping -n 1 192.168.13.4 > NULL
resultat : 0
---------------
cmds : ping -n 1 192.168.12.6 > NULL
resultat : 0
---------------
cmds : ping -n 1 192.168.14.5 > NULL
resultat : 0
---------------
cmds : ping -n 1 192.168.14.8 > NULL
resultat : 0
---------------
cmds : ping -n 1 192.168.1.96 > NULL
resultat : 0

Bizarre n'est-ce pas ?

Avez-vous une idée pour solutionner mon problème :cry: ?

Merci d'avance :ccool:

15/12/2010, 10h53
6-MarViN

Vérifie que les tableaux que tu utilise pour remplir cmd soit bien remplis.
15/12/2010, 10h55
jalekoo

Bien... Je dirais qu'ils sont bien remplis vu que lorsque je commente system (), on voit bien les valeurs =) (d'où mon expérience de mise en commentaire de system ())
15/12/2010, 11h14
6-MarViN

Essaye de remettre ta chaine à zéro après ton system:

Code:

bzero(cmd, 50);

Je ne sais pas si ca va changer quelque chose mais apparemment dans les deux cas le premier passage est bon. Or le point commun entre les deux cas c'est qu'au premier passage dans ta boucle, cmd est vide. Donc peut-être qu'en la réinitialisant à chaque fois ça résoudra ton problème.
15/12/2010, 11h32
jalekoo

Ben pas très concluant ...

Déjà avec bzero, j'ai une erreur de compil : undefined fonction (alors que j'ai bien string.h d'inclu !)

J'ai fait un :

Code:

strcpy (cmd, "")

Et le résultat :

Citation:

---------------
cmds : ping -n 1 123.12.123.10 > NULL
resultat : 0
---------------
cmds : ping -n 1 4980802.1999633846.1988581918.48 > NULL
resultat : 1
---------------
cmds : ping -n 1 0.0.2679004.1988581993 > NULL
resultat : 1
---------------
cmds : ping -n 1 0.0.0.0 > NULL
resultat : 1
---------------
cmds : ping -n 1 1988583728.88.1988583756.0 > NULL
resultat : 1
---------------
cmds : ping -n 1 0.2678220.0.0 > NULL
resultat : 1
---------------
cmds : ping -n 1 0.0.0.0 > NULL
resultat : 1

Visiblement, ça ne change rien :-(
J'ai fait un test entre temps (qui ne résoud rien mais qui peut faire réfléchir !)

j'ai légèrement modifier mon code pour vérifier si je pingais une ip prédéfini dans un tableau de char genre 192.168.10.1 : et là l'ip ne bouge pas.

Je me suis dis, bon bah je vais mettre mon ip dans un tableau de char à chaque fois ... donc "ip" reçoit TabOrdi[i].IPOrdi.ip1, TabOrdi[i].IPOrdi.ip2, TabOrdi[i].IPOrdi.ip3, TabOrdi[i].IPOrdi.ip4 (grossièrement) ; et là je fais un ping sur le tableau de char "ip"

Et bah même problème que précédemment =( l'ip est complêtement modifiée !

Je ne comprend pas la relation qu'il peut exister entre system () et les int de ma structure :'-(
15/12/2010, 11h52
jeroman

A tout hasard :
Quel compilateur utilises-tu ? Quelles sont tes options de compilation ? Diminue le niveau d'optimisation (s'il y en a).
15/12/2010, 11h55
6-MarViN

Et quelle plateforme utilise-tu? parce que le bzero pas reconnu ca me choque...
15/12/2010, 11h59
Malka1986

On peut voir la fonction chargerFichier ?
15/12/2010, 11h59
jalekoo

Ben j'utilise CodeBlocks avec minGw et les options de compil par défaut ^^

Voilà la fonction :

Code:

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
Ordi* chargerFichier (char* fichier)
{
    // Définition des variables
    FILE* fichierCharger = NULL;
    char cheminFichier[500];
    Ordi Tab[100];
    char vid;
    int i = 0;
 
    // Réinitialisation des variables
    strcpy(cheminFichier, "");
    nbreElement = 0;
 
    // initialisation du chemin complet cheminFichier du fichier mis en paramètre
    strcat(cheminFichier, _Path);
    strcat(cheminFichier, fichier);
 
    // Affichage du chemin complet
    printf("%s\n", cheminFichier);
 
    // Chargement du fichier et ouverture en lecture
    if((fichierCharger = fopen (cheminFichier, "r")) != NULL)
    {
        // Tant que le fichier n'est pas terminé
        while(feof(fichierCharger) == 0)
        {
            // Récupération des informations contenues dans le fichier
            fscanf(fichierCharger, "%d.%d.%d.%d;%[^;]s", &Tab[i].IPOrdi.ip1, &Tab[i].IPOrdi.ip2, &Tab[i].IPOrdi.ip3, &Tab[i].IPOrdi.ip4, &Tab[i].NomOrdi);
            fscanf(fichierCharger,"%c",&vid);
            fscanf(fichierCharger,"%s\n", &Tab[i].Telephone);
            i++;
        }
 
        // Définition du nombre d'élément contenu dans le tableau
        nbreElement = i - 1;
 
        // Fermeture du fichierCharger
        fclose(fichierCharger);
 
        // Retour du tableau Tab Ordi*
        return Tab;
    }
    else
    {
        // Sinon afficher une erreur
        printf("erreur !\n");
    }
}

15/12/2010, 12h04
Malka1986
Citation:
Envoyé par jalekoo

Voilà la fonction :

Code:

1 2 3 4 5 6 7 8 Ordi* chargerFichier (char* fichier) { // Définition des variables Ordi Tab[100]; [...] return Tab; } }
Il faut pas return une variable locale, son adresse est invalide hors de la fonction ;).
Fait plutot un malloc :
Code:

1 2 3 Ordi *Tab Tab = malloc(100 * sizeof(*Tab));
Et return quelque chose en cas d'erreur, null par exemple, sinon tu peux te retrouver avec tout et n'importe quoi dans ta variable qui recupere le retour de la fonction.
15/12/2010, 12h09
jeroman

Code:

while(feof(fichierCharger) == 0)

Ce n'est pas la bonne façon de lire un fichier. :aie:
15/12/2010, 13h13
jalekoo

Wawou ! Ca marche ! lol ^^

Alors merci à tous pour votre aide et vos conseils =)

Malka1986 a trouvé la solution ... En fait, c'était une question de mémoire ! En faisant le malloc, ça marche :ccool:

Mais j'ai du mal à comprendre le pourquoi du comment ? (on a jamais vu le malloc en cours, soit disant trop compliqué alors que le prof nous fait commencer la manipulation des classes en C++... J'ai du mal à comprendre sa logique... Bref !)

Sinon jeroman, ah bon ce n'est pas la bonne façon ? Que me conseil tu ? Tous conseils sont bon à prendre pour améliorer la qualité du programme, et de mes programmes futurs =)

Merci :D
15/12/2010, 13h24
jeroman

Citation:

Envoyé par jalekoo

Sinon jeroman, ah bon ce n'est pas la bonne façon ? Que me conseil tu ? Tous conseils sont bon à prendre pour améliorer la qualité du programme, et de mes programmes futurs =)

Merci :D

Extrait de la FAQ :

Citation:

Contrairement à une pratique trop souvent rencontrée, la fonction feof() ne sert pas à détecter la fin d'un fichier mais, lorsqu'une lecture a échoué, à savoir si cet échec est du à la rencontre de la fin de fichier.

La bonne pratique pour lire intégralement un fichier consiste donc à parcourir le fichier tant qu'aucune erreur de lecture n'est remontée et d'utiliser ferror et feof pour déterminer, suite à une erreur, la cause de cette erreur.

Le mieux, c'est d'utiliser la fonction fgets, de tester si elle ne renvoie pas NULL. Ensuite, tu peux troquer tes fscanf par des sscanf.

EDIT:

Citation:

Malka1986 a trouvé la solution ... En fait, c'était une question de mémoire ! En faisant le malloc, ça marche

Mais j'ai du mal à comprendre le pourquoi du comment ? (on a jamais vu le malloc en cours, soit disant trop compliqué alors que le prof nous fait commencer la manipulation des classes en C++... J'ai du mal à comprendre sa logique... Bref !)

Les variables (ou tableaux) locales à une fonction sont stockés sur la pile. Du coup, lorsqu'on quitte la fonction, les données peuvent à tout moment être écrasées par autre chose.
15/12/2010, 14h49
jalekoo

Ok ! Ca marche =)

Bonne journée à tous et encore merci !