Discussion :

[Vantin19]

Bonsoir !

Voici mon problème: on se donne un fichier que je vous écris ci dessous :

1
7
1 3 5 8 6 7 4 3 2 1 9 6 4 3 6 2
+*++**-///*-/-+-*-*++++-
2
20

J'aimerais transformer ce fichier texte en une unique chaîne de caractère et pour cela j'ai écris cette fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void all_in_one(int max_line, char *buffer,FILE* f){
    char *chaine=malloc(sizeof(char)*max_line);
    for(int i=0,n=0;i<6;i++){
        fgets(chaine,max_line,f);
        sprintf(buffer+n,"%s\0",chaine);
        n=n+strlen(chaine);
    }
    fseek(f, 0, SEEK_SET);
    free(chaine);
    return;
}

Sauf que lorsque je lance valgrind, j'obtiens ==103== Invalid write of size 1, j'avais essayé avec la fonction strcat mais j'ai une erreur du même registre...

Quand j'essaye de cibler mon problème en ne récupérant qu'une ligne particulière j'ai exactement le même soucis Invalid write of size 1 :

Code :

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

1
2
3
4
5
6
7
8
9
void give_me_operator(FILE* f,int max_line,char *p){
    char chaine[max_line];
    for(int i=0;i<4;i++){
        fgets(chaine, max_line, f);
    }
    fseek(f, 0, SEEK_SET);
    strcpy(p, chaine);
    return;
}

Je m'en remets à vous, merci d'avoir pris le temps de me lire !

[WhiteCrow]

Bonjour,
plusieurs points :

• valgrind t'indique quel ligne fait le débordement … ça devrait fortement aider le debug (il faut impérativement compiler en mode debug soit avec l'option `-g` avec gcc et clang);
• relis la doc de fgets … surtout la partie concernant le paramètre size et ce qui se passe avec le caractère '\0' :
  

  fgets() reads in at most one less than size characters from stream and stores them into the buffer pointed to by s.
  Reading stops after an EOF or a newline. If a newline is read, it is stored into the buffer. A terminating null byte
  ('\0') is stored after the last character in the buffer.

• attention à l'utilisation des VLA qui peuvent rapidement exploser la pile;
• je ne peux que te conseiller d'implémenter un string buffer qui croît dynamiquement (avec un facteur de croissance entre 1.5 et 2) avec toutes les fonctions qui vont bien. Cela simplifiera non seulement ton code mais également le debug.

Sinon si tu es sous *nix, tu as mmap qui permet d'accéder au contenu d'un fichier directement en mémoire …

[Sve@r]

Bonjour

J'aimerais transformer ce fichier texte en une unique chaîne de caractère et pour cela j'ai écris cette fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void all_in_one(int max_line, char *buffer,FILE* f){
    char *chaine=malloc(sizeof(char)*max_line);
    for(int i=0,n=0;i<6;i++){
        fgets(chaine,max_line,f);
        sprintf(buffer+n,"%s\0",chaine);
        n=n+strlen(chaine);
    }
    fseek(f, 0, SEEK_SET);
    free(chaine);
    return;
}

Déjà pas besoin de mettre le "\0' dans sprintf(). Toute fonction de création de chaine s'occupe elle-même de mettre le '\0' qui va bien. Accessoirement tu peux remettre strcat(), le souci ne vient pas de là. En fait d'ailleurs il n'y a pas de réel souci avec cette fonction. Pour peu que "buffer" ait une taille suffisante quand on le lui passe et que le fichier ait été bien ouvert avec possibilité de lecture elle "fonctionne" parfaitement. Je ne dis pas qu'elle est parfaite, comme par exemple cette boucle de 0 à 6 limitant un peu sa "généricité" (elle ne marchera que pour les fichiers de 6 lignes, pas ceux de 5 ni ceux de 7) mais son code est sans erreur, y compris avec ce "return" inutile.

PS: dans un but de correspondre à la réalité, je travaillerais avec max_line + 1 dans la fonction, afin que "max_line" représente le nb de caractères "réels" de la ligne et non pas le nombre de caractères "alloués" (auquel cas on en perd un pour laisser la place au '\0').

[Vantin19]

Bonsoir, je vous remercie d'avoir pris le temps de me répondre, en effet le problème c'est que je générais un tableau de taille insuffisante à cause du \0 ça m'apprendra à utiliser des fonctions sans savoir réellement ce qu'elle font, encore merci !!