Erreur dans fread

Version imprimable

Voir 40 message(s) de cette discussion en une page

Erreur dans fread

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

J'ai le code suivant :

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

22/05/2006, 11h59
Skyounet

Pas de test du retour des malloc.
22/05/2006, 12h02
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 .
22/05/2006, 12h08
Gruik

C'est quoi fsize?

Citation:

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
22/05/2006, 12h09
Strab

Citation:

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.

:?

Citation:

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:

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

22/05/2006, 12h10
Skyounet

Citation:

Envoyé par Gruik

C'est quoi fsize?

Voir le premier post

Citation:

La fonction fsize est celle donnée dans la FAQ pour connaitre la taille d'un fichier.
22/05/2006, 12h11
Strab

Citation:

Envoyé par Gruik

C'est quoi fsize?

Ca : http://c.developpez.com/faq/c/?page=...HIERS_filesize
22/05/2006, 12h14
Gruik

Ok, désolé :oops:

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
22/05/2006, 12h15
Skyounet

Citation:

Envoyé par Gruik

Ok, désolé :oops:

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

Citation:

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.
22/05/2006, 12h17
Strab

Citation:

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 ?

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

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

22/05/2006, 12h19
Skyounet

+1 pour Gruik essaye des printf.

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

Un avant fread et un apres.
22/05/2006, 12h20
Strab

Citation:

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

Citation:

Envoyé par Trap D

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

Excellente question, pourquoi faire compliqué quand on peut faire simple :oops:
J'essaie de ce pas.
22/05/2006, 12h22
Trap D

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
22/05/2006, 12h22
Gruik

Désolé, j'avais pas lu

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

Citation:

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:

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

22/05/2006, 12h41
Gruik

Il ya peut être un depassement de mémoire antérieur à la fonction.
Et là, bonne chance pour le trouver :mrgreen:
22/05/2006, 12h44
Gruik

Ptet que fsize() fait des operations qu'il devrait pas faire sur le FILE *, ptet essayer en remplaçant l'appel à fsize() par la longueur du fichier
22/05/2006, 12h55
Trap D

Pour plus de précaution, fait un fseek(f, 0l, SEEK_SET); qui te remet en début de fichier.
Tu fais du C ou du C++ ?

Voir 40 message(s) de cette discussion en une page