Discussion :

[bouritosse]

Bonjour,

Je suis actuellement en train de faire un programme qui copie un fichier
le stocke dans un tableau puis le recopie dans un fichier de sortie.

Le problème est que je me trouve devant un segfault, dés que je rajoute
la fonction CopyFile() (c'est pour ça qu'il est en commentaire)
et je ne sais pas du tout comment régler ce problème j'ai voulu tester en
mettant des printf() partout mais ça aide pas.

Je vous remercie pour votre aide, si vous avez une idée !

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
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
 
#include <stdio.h>
#include <string.h>
//#include "Prototype.h"
 
#define TAILLE_MAX 20
 
typedef struct{
	 char InpName[TAILLE_MAX];
	 char OutName[TAILLE_MAX];
	 int cpt;
	char tabImage[255];
}Image;
//Cette structure contient le nom du fichier d'entrée et sortie, elle compte le nombre de
//caractère (cpt), et garde en mémoire le fichier récupéré
void InpOutFile(Image *Im1);
char * saisir_chaine(char * lpBuffer, int buf_size);
void cptChar(Image *Im2);
void CopyFile(Image *Im3);
 
int main()
{
Image Im;
 
InpOutFile(&Im);
cptChar(&Im);
//CopyFile(&Im);
 
return 0;
}
 
void InpOutFile(Image *Im1)
{
 
    FILE *pictureI=NULL;
    FILE *pictureO=NULL;
 
    printf("Ce programme permet de copier un fichier.\n");
    printf("source : ");
    saisir_chaine(Im1->InpName, sizeof(Im1->InpName));
 
    pictureI= fopen(Im1->InpName,"rb");
    		if(pictureI== NULL)
    		perror(Im1->InpName);
 
        printf("dest : ");
        saisir_chaine(Im1->OutName, sizeof(Im1->OutName));
 
        if (strcmp(Im1->InpName, Im1->OutName) == 0)
            printf("La source ne peut pas etre en meme temps la destination.\n");
        else
        {
        pictureO = fopen(Im1->OutName,"wb");
                if (pictureO == NULL)
                perror(Im1->OutName);
        }
        printf("fin code 1");
        close(pictureI);
        close(pictureO);
}
 
char * saisir_chaine(char * lpBuffer, int buf_size)
{
    char * ret = fgets(lpBuffer, buf_size, stdin);
 
    if (ret != NULL)
    {
        char * p = strchr(lpBuffer, '\n');
        if (p != NULL)
            *p = '\0';
        else
        {
            int c;
 
            do
                c = getchar();
            while (c != EOF && c != '\n');
        }
    }
    return ret;
}
 
void cptChar(Image *Im2)
{
	FILE *pictureI=NULL;
 
      pictureI= fopen(Im2->InpName,"rb");
      if(pictureI== NULL)
    		perror(Im2->InpName);
 
      while(fgetc(pictureI) != EOF)
      Im2->cpt++;
      //On compte le nombre de caractère
 
	close(pictureI);
	printf("fin code 2");
}
 
void CopyFile(Image *Im3)
{
    FILE *pictureI=NULL;
    FILE *pictureO=NULL;
 
    char tab[Im3->cpt+1];
    printf("%c",Im3->cpt);
    int i=0;
 
     pictureI= fopen(Im3->InpName,"rb");
    		if(pictureI== NULL)
    		perror(Im3->InpName);
 
      pictureO = fopen(Im3->OutName,"wb");
                if (pictureO == NULL)
                perror(Im3->OutName);
 
    while( fgets(tab,Im3->cpt,pictureI) != NULL)
    {
    strcat(Im3->tabImage,tab);
    //Dés qu'on atteint un '\n' dans le fichier on stocke dans
    //un tableau qu'on concaténe avec le tableau complet
    }
 
    for(i=0;i<Im3->cpt;i++)
    {
    fputc(Im3->tabImage[i],pictureO);
    //On écrit dans le fichier de sortie ce qu'on a dans le tableau
    //en mémoire
    }
 
	close(pictureI);
	close(pictureO);
}

[LinuxUser]

Grâce à gdb on peut voir que le problème se situe à la ligne 118 :

Code :

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

    strcat(Im3->tabImage,tab);

Vérifie que la taille de ton tableau n'est pas trop petite (augmente significativement la taille de char tabImage[255]).

[bouritosse]

Ah oui c'est bien ça, je connaissais pas gdb je crois que je vais m'y mettre !

Je te remercie

[LinuxUser]

OK très bien, voilà pour commencer (juste une introduction)
http://perso.ens-lyon.fr/daniel.hirs...cs/doc_gdb.pdf

Par contre il faut que tu testes desormais qu'une fois concaténée, ta chaîne ne dépassera pas la taille que tu as fixée.