Discussion :

[Tomystique]

Bonjour,
Ça fait deux jours que je me casse la tête là dessus et je ne parviens pas à trouver mon erreur
Surtout qu'il m'a semblé que mon programme fonctionnait à la première utilisation, puis le lendemain... plus rien

Enfin quand je dis "plus rien", voici ce qu'il me donne quand je l'exécute et que je rentre "Thomas" sur l'entrée standard (de Windows) :



Voici le programme :

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
92
93
94
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <string.h>
 
char * formatChaine (const char * chaine);
int estIP(const char * chaine);
 
int main(void)
{
    char * chaine;
    char result[30];
    chaine = (char*) malloc(20);
    strcpy(result,"ping -t ");
    printf("Ping sur ?\n");
    gets(chaine);
    chaine = formatChaine(chaine);
    if (strcmp(chaine,"thomas")==0)
    {
       strcat(result,"192.168.1.11");
    }
    else if (strcmp(chaine,"cecile")==0)
    {
         strcat(result,"192.168.1.12");
    }
    else if (estIP(chaine))
    {
         strcat(result,chaine);
    }
    else
    {
        printf("Adresse incorrecte !\n");
        system("PAUSE");
        exit(0);
    }
    system(result);
    return 0;
}
 
char * formatChaine (const char * chaine)
{
     char * s;
     s=(char*)malloc(strlen(chaine));
     strcpy(s,chaine);
     int i;
     for(i = 0; i < strlen(chaine); i++)
     {
           s[i] = tolower(s[i]);
           switch(s[i])
           { 
            case 0x82:
            case 0x88:
            case 0x89:
            case 0x8a: s[i]='e'; break;
            case 0xa0:
            case 0x83:
            case 0x84:
            case 0x85:
            case 0x86: s[i]='a'; break;
            case 0x87: s[i]='c'; break;
            case 0x93:
            case 0x94:
            case 0x95:
            case 0xa2: s[i]='o'; break;
            case 0x96:
            case 0x97:
            case 0xa3:
            case 0x81: s[i]='u'; break;
            default: break;
           }
     }
     return(s);
}
 
int estIP (const char * chaine)
{
    int n1;
    int n2;
    int n3;
    int n4;
    int compteurPoint = 0;
    int i;
    for(i = 0; i < strlen(chaine); i++)
    {
          if (chaine[i] == '.')
          {
             compteurPoint++;
          }
    }
    if (compteurPoint !=3)
       return 0;
    sscanf(chaine,"%d%*[.]%d%*[.]%d%*[.]%d",&n1,&n2,&n3,&n4);
    return(192 <= n1 && n1 < 224 && 0 < n2 && n2 < 255 && 0 < n3 && n3 < 255 && 0 < n4 && n4 < 255);
}

Je ne comprends même pas pourquoi cela me donne un tel résultat...
Merci d'avance à ceux qui m'aideront à avancer

[groscask]

Félicitations, tu fais de l'overflow !

En tout cas, ça y ressemble (mais c'est alors très fort).

Code :

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

1
2
3
4
char * formatChaine (const char * chaine)
{
     char * s;
     s=(char*)malloc(strlen(chaine));

strlen(chaine) renvoie le nombre de caractères dans la chaîne... SANS prendre en compte le caractère NULL de fin de chaine, vital en C !

Code :

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

1
2
3
4
5
 
s=(char*)malloc(strlen(chaine) + 1);
 
/* Ou meme, tout initialiser a zero */
s=(char*)calloc(strlen(chaine) + 1);

C'est déjà mieux comme ça

Ensuite tu souffres de DEUX fuites mémoire : tu fais deux malloc() et aucun free n'apparait dans ton programme...

[Tomystique]

Tout d'abord, merci de m'avoir répondu !
Ensuite, apparemment le problème du strlen n'en est pas vraiment un puisque j'arrive tout de même à faire la bonne comparaison (le result à la fin de mon main a la bonne valeur quand je l'affiche).
Pour ce qui est de la libération de l'espace mémoire, je l'ai rajouté dans le main, mais j'avoue ne pas savoir comment le faire dans ma fonction formatChaine...
Comment puis je retourner mon char* si je l'efface de la mémoire ou comment l'effacer de la mémoire une fois retourné ?

Donc j'en suis là, j'ai corrigé le strlen, et j'ai libéré ma chaine dans le main... et j'ai toujours le même problème...

[souviron34]

Ensuite, apparemment le problème du strlen n'en est pas vraiment un puisque j'arrive tout de même à faire la bonne comparaison (le result à la fin de mon main a la bonne valeur quand je l'affiche).

hasard total !! malchance...

Cela veut dire simplement que tu ecrases quelque chose qui ne sers pas a grand chose derriere..

Sur une autre machine, ou sur la meme mais avec une instruction ou un programme en plus, ca peut crasher...

[Tomystique]

Oui ça je le sais bien, ce que je voulais dire, c'est que malgré mon erreur, le problème ici présenté ne vient pas de là.
Je précise que j'ai corrigé l'erreur, et que ça ne fonctionne malheureusement toujours pas

Mais quand même, je me pose une question, comment le programme peut-il ré afficher "Ping sur ?" après que j'ai récupéré la chaine sur l'entrée standard alors que je ne boucle pas ?
J'ai tracé son exécution et tout va bien jusqu'au system(result); ou il ne fait pas ce qu'il devrait faire...

[Tusbar]

Félicitations, tu fais de l'overflow !

Code :

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

1
2
3
4
5
 
s=(char*)malloc(strlen(chaine) + 1);
 
/* Ou meme, tout initialiser a zero */
s=(char*)calloc(strlen(chaine) + 1);

C'est déjà mieux comme ça

Et si un char est sur deux octets, tu affiches quoi ? Segfault ?

PS : malloc() / calloc() renvoient un void *, donc pointeur génerique, donc cast inutile.

PS2 : En general, on verifie aussi le retour de l'allocation, histoire de ne pas utiliser strcat() sur un pointeur nul...

[Tomystique]

Et si j'utilise sizeof(chaine) au lieu de strlen(chaine) + 1, ça serait mieux ?

Par ailleurs, mon compilateur foire quand je ne cast pas le retour du malloc...
Pour la vérif sur le retour d'allocation, je l'ai rajouté, j'avais complètement zappé ^^, merci

Est ce que mon problème pourrait venir de mon compilateur (Dev-C++) ?

[Médinoc]

Et si j'utilise sizeof(chaine) au lieu de strlen(chaine) + 1, ça serait mieux ?

Non, car chaîne est un char* et pas un tableau.

Par ailleurs, mon compilateur foire quand je ne cast pas le retour du malloc...
Pour la vérif sur le retour d'allocation, je l'ai rajouté, j'avais complètement zappé ^^, merci

Est ce que mon problème pourrait venir de mon compilateur (Dev-C++) ?

Vérifie que tu compiles bien en C et non en C++ : http://www.developpez.net/forums/sho...37&postcount=5

[Tomystique]

Oui Médinoc je compile bien en C (j'avais moi même vérifié avant, ayant déjà fait cette bourde dans un autre programme ^^).

Bref, quelqu'un aurait il d'autres idées ?

[Médinoc]

Tu peux nous montrer l'erreur que tu obtiens sans le cast ?

[souviron34]

Et si un char est sur deux octets, tu affiches quoi ? Segfault ?

sizeof(char) == 1 par definition...

[Tomystique]

Tu peux nous montrer l'erreur que tu obtiens sans le cast ?

Bon alors là je suis vraiment un gros boulet... ça compile sans le cast... je dois avoir des hallucinations...

Mais ça ne résout toujours pas mon problème... et je ne comprends toujours pas comment mon programme peut boucler comme il le fait... ça m'enneeeerve