Discussion :

[Strab]

Bonjour,
Je suis confronté à une erreur qui me laisse perplexe.

J'ai le code suivant :

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
 
 int valider(char * f, cle p)
{
    hash h;
    signature s;
    FILE * stream = fopen(f, "r");
    if(!stream){
        fprintf(stderr, "Erreur : impossible d'ouvrir %s en lecture.\n", f);
        return -1;
    }
 
    h = calculerHash(stream);
 
    ...
}
 
hash calculerHash(FILE * f)
{
    hash h = malloc(sizeof(char)*17);
    long s = fsize(f);
    char * str = malloc(sizeof(char) * (s+1));
    char * save = str;
    int lu = 1024, lire = 1024, tlu = 0;
    while(lu == lire){
        lu = fread(str, sizeof(char), lire, f);
        tlu += lu;
        str += lu;
    }
    if(tlu != s){
        fprintf(stderr, "Erreur lors de la lecture du programme.\n");
        return NULL;
    }
    save[s] = '\0';
    hasher(save, h);
    return h;
}

En exécutant ce programme, j'obtient un segmentation fault, qui a lieu dans le fread d'après gdb dès la première itération de la boucle (testé sur un fichier de 10180 octets). De quoi cela peut-il venir ? La fonction fsize est celle donnée dans la FAQ pour connaitre la taille d'un fichier.

Merci de votre aide

[Skyounet]

Pas de test du retour des malloc.

[jamalmoundir]

au lieu
char * str = malloc(sizeof(char) * (s+1)); il faut mettre
char * str = malloc(sizeof(char *) * (s+1)); je suis pas sur ,essaye on sait jamais .

[Gruik]

C'est quoi fsize?

au lieu
char * str = malloc(sizeof(char) * (s+1)); il faut mettre
char * str = malloc(sizeof(char *) * (s+1)); je suis pas sur ,essaye on sait jamais .

Nonnon, là ca serait pour allouer un tableau de char *, c'est pas ce qu'il veut faire

[Skyounet]

C'est quoi fsize?

Voir le premier post

La fonction fsize est celle donnée dans la FAQ pour connaitre la taille d'un fichier.

[Strab]

C'est quoi fsize?

Ca : http://c.developpez.com/faq/c/?page=...HIERS_filesize

[Gruik]

Ok, désolé

Euh je vois pas trop non plus, si j'étais toi j'utiliserais un debugger, tu peux mieux savoir où ça plante
Sinon, ya la technique du printf

[Trap D]

Le sizeof(char) est inutile, par definition en C c'est 1;
Pourquoi ne lis-tu pas tout d'un seul coup ?

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
hash calculerHash(FILE * f)
{
    hash h = malloc(17);
    long s = fsize(f);
    char * str = malloc(s+1);
    char * save = str;
     int lu = 1024;
 
    if (h == NULL)
    {
        free(str);
        return NULL;
    }
 
    if (str == NULL)
    {
       free(h);
       return NULL;
    }
 
     lu = fread(str, 1, s, f);
    if(lu != s){
        fprintf(stderr, "Erreur lors de la lecture du programme.\n");
        return NULL;
    }
 
    save[s] = '\0';
    hasher(save, h);
    // il faut peut-être libérer la mémoire ??
    return h;
}

[Skyounet]

au lieu
char * str = malloc(sizeof(char) * (s+1)); il faut mettre
char * str = malloc(sizeof(char *) * (s+1)); je suis pas sur ,essaye on sait jamais .

Non !!!

Code :

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

1
2
3
4
5
6
7
8
9
 
char *str = malloc(s+1); // sizeof(char) vaut 1 par définition
if(str == NULL)
{
    printf("Allocation memoire echouee !!!\n");
    return EXIT_FAILURE; // ou meme exit(EXIT_FAILURE) selon le degre de gravite
}
[...]
free(str);

[Emmanuel Delahaye]

au lieu
char * str = malloc(sizeof(char) * (s+1)); il faut mettre
char * str = malloc(sizeof(char *) * (s+1)); je suis pas sur ,essaye on sait jamais .

Il faut arréter de programmer au hasard.

"engage your brain"...

[Jean-Marc.Bourguet]

Il faut arréter de programmer au hasard.

C'est pas ca qu'on appelle la programmation genetique ?

[Strab]

Pas de test du retour des malloc.

Ouep, merci du rappel, je viens de les rajouter.
Néanmoins, l'erreur ne vient pas de ça, la mémoire est suffisante.