Erreur dans fread

**Strab** · 22/05/2006, 11h57

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** · 22/05/2006, 11h59

Pas de test du retour des malloc.

**jamalmoundir** · 22/05/2006, 12h02

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** · 22/05/2006, 12h08

C'est quoi fsize?

Envoyé par 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 .

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

**Strab** · 22/05/2006, 12h09

Envoyé par Skyrunner

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.

**Skyounet** · 22/05/2006, 12h09

Envoyé par 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 .

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);

**Skyounet** · 22/05/2006, 12h10

Envoyé par Gruik

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** · 22/05/2006, 12h11

Envoyé par Gruik

C'est quoi fsize?

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

**Gruik** · 22/05/2006, 12h14

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

**Skyounet** · 22/05/2006, 12h15

Envoyé par 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

qui a lieu dans le fread d'après gdb dès la première itération de la boucle

Pour le debugger c'est deja fais ^^
Decidemment.

**Strab** · 22/05/2006, 12h17

Envoyé par Gruik

si j'étais toi j'utiliserais un debugger, tu peux mieux savoir où ça plante

C'est ce que j'ai fait, gdb est un excellent debuggeur. C'est ce qui m'a permis de savoir que ça plantait dans fread. Ma question est pourquoi ?

**Trap D** · 22/05/2006, 12h17

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** · 22/05/2006, 12h19

+1 pour Gruik essaye des printf.

Moi j'en mettrai un apres le fsize pour afficher la taille.

Un avant fread et un apres.

**Strab** · 22/05/2006, 12h20

Envoyé par Trap D

De toute façon avec cette méthode (str += lu) tu auras des problèmes puisque tu perds l'adresse initialiement fournie par malloc.

Non, c'est l'utilité de save

Envoyé par Trap D

Pourquoi ne lis-tu pas tout d'un seul coup ?

Excellente question, pourquoi faire compliqué quand on peut faire simple

J'essaie de ce pas.

**Trap D** · 22/05/2006, 12h22

J'ai modifié plusieurs fois mon post parceque je n'avais pas tout lu et je m'étais mélangé dans les différentes variables

**Gruik** · 22/05/2006, 12h22

Désolé, j'avais pas lu

Est ce que le FILE * fourni est ouvert et valide?

**Strab** · 22/05/2006, 12h33

Envoyé par Gruik

Désolé, j'avais pas lu

Est ce que le FILE * fourni est ouvert et valide?

Oui, cf le code de la fonction valider.

J'ai essayé en une seule lecture, même problème au même endroit

Je n'ai pas essayé de printf, mais j'avais regardé toutes les variables dans le débuggeur, rien d'anormal.
Voici le code de calculerHash mis à jour :

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
 
hash calculerHash(FILE * f)
{
    hash h = malloc(sizeof(char) * 17);
    if (!h)
    {
        printf("Mémoire insuffisante !\n");
        return NULL;
    }
    long s = fsize(f);
    char * str = malloc(s+1);
    char * save = str;
    if (!str)
    {
        printf("Mémoire insuffisante !\n");
        return NULL;
    }
    int tlu  = fread(str, sizeof(char), s, f);
    if(tlu != s){
        fprintf(stderr, "Erreur lors de la lecture du programme.\n");
        return NULL;
    }
 
    save[s] = '\0';
    hasher(save, h);
    return h;
}