Erreur avec strcat

**nixonne** · 08/12/2006, 18h41

Bonjour à tous,

J'ai un problème avec mon strcat qui après son passage modifie le contenu de certaine variable (j'ai tester avant)

En effet une fois qu'il est executer 1 fois, la variable option est différent et je ne peux pas faire le strcmp()

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
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
//---------------------------------------------------------------------------
 
#pragma hdrstop
#include <stdio.h>
#include <string.h>
//---------------------------------------------------------------------------
#pragma argsused
 
long filesize(FILE *stream);
 
 
int main(int argc, char* argv[])
  {
  char* source=NULL;
  char* dest=NULL;
  char* option=NULL;
  char* optionCryptage;
  char* sourceFichier;
  char* destFichier;
  char* Chainefinal;
  char* temp=NULL;
  char ajoutFinal[2];
  unsigned char code=NULL;
  char resultat;
  FILE* fichierSource;
  FILE* fichierDest;
  int i,j,tabCaract[8];
  long taille_fichier;
 
  // on vérifie qu'il y ait les 4 arguments
  if(argc!=4)
    {
    printf("il manque des arguements");
    return(0);
    }
  else
    {
    // récupération des arguments
    optionCryptage = argv[1];
    sourceFichier = argv[2];
    destFichier = argv[3];
 
    option = optionCryptage;
    source = sourceFichier;
    dest = destFichier;
 
    // ouverture et lecture du fichier source
    if(fichierSource = fopen(source, "r+"))
      {
      taille_fichier = filesize (fichierSource);
      temp=(char *)malloc(sizeof(char) * taille_fichier);
      Chainefinal=(char *)malloc(sizeof(char) * taille_fichier);
      Chainefinal="";
      fread (temp,1,taille_fichier,fichierSource);
      fclose(fichierSource);
      }
    // traitement du contenu
        for (i=0;i<taille_fichier;i++)
        {
        code = temp[i];
        // on passe le caractère en tableau de 8bits
        for(j=0;j<8;j++)
          {
          tabCaract[j] = (code>127?1:0) ;
          code<<=1;
          }
        // on code ou décode en fonction de l'option
        for(j=0;j<4;j++)
          {
          if(strcmp(option,"-c")==0)
            {
            encodage(&tabCaract);
            printf("-c");
            }
          if(strcmp(option,"-d")==0)
            {
            decode(&tabCaract);
            printf("-d");
            }
          }
       // on repase le résultat en caractère et on le stock
      resultat = (128*tabCaract[0])+(64*tabCaract[1])+(32*tabCaract[2])+(16*tabCaract[3])+(8*tabCaract[4])+(4*tabCaract[5])+(2*tabCaract[6])+(1*tabCaract[7]);
      ajoutFinal[0] = (char)resultat;
      ajoutFinal[1] = '\0';
      strcat(Chainefinal, ajoutFinal);
      }
 
    //on enregistre dans le fichier de destination
    if(fichierDest=fopen(dest,"w+")!=NULL)
      {
      fwrite(Chainefinal,1,taille_fichier,fichierDest);
      fclose(fichierDest);
      }
    } // fin else
  return(0);
  }
 
//*************************************
//********** FONCTION ENCODE **********
//*************************************
int encodage(int * caractere)
  {
  int binaire1[4],binaire2[4],final[8],d0[4],d1[4],i;
 
  // on sépare en 2 blocque de 4
  for(i=0;i<4;i++)
    {
    binaire1[i]=caractere[i];
    binaire2[i]=caractere[i]=caractere[i+4];
    }
 
  // on inverse selon la clé
  d0[0]=binaire2[1];
  d0[1]=binaire2[0];
  d0[2]=binaire2[3];
  d0[3]=binaire2[2];
 
  // on fais le XOR
  d1[0] = binaire1[0]+d0[0];
  d1[1] = binaire1[1]+d0[1];
  d1[2] = binaire1[2]+d0[2];
  d1[3] = binaire1[3]+d0[3];
 
  for(i=0;i<4;i++)
    {
    if(d1[i]==2)
      d1[i]=0;
    }
 
  //on complete la fin
  caractere[4]=d1[0];
  caractere[5]=d1[1];
  caractere[6]=d1[2];
  caractere[7]=d1[3];
  //return (final);
  }
 
//*************************************
//********** FONCTION ENCODE **********
//*************************************
int decode(int * caractere)
  {
  int binaire1[4],binaire2[4],final[8],d0[4],g0[4],i;
 
  // on sépare en 2 blocque de 4
  for(i=0;i<4;i++)
    {
    binaire2[i]=caractere[i+4];
    binaire1[i]=caractere[i+4]=caractere[i];
    }
 
  // on inverse selon la clé
  d0[0]=binaire1[1];
  d0[1]=binaire1[0];
  d0[2]=binaire1[3];
  d0[3]=binaire1[2];
 
  // on fais le XOR
  caractere[0] = binaire2[0]+d0[0];
  caractere[1] = binaire2[1]+d0[1];
  caractere[2] = binaire2[2]+d0[2];
  caractere[3] = binaire2[3]+d0[3];
 
  for(i=0;i<4;i++)
    {
    if(caractere[i]==2)
      caractere[i]=0;
    }
  }
//---------------------------------------------------------------------------
long filesize(FILE *stream)
{
   long curpos, length;
 
   curpos = ftell(stream);
   fseek(stream, 0L, SEEK_END);
   length = ftell(stream);
   fseek(stream, curpos, SEEK_SET);
   return length;
 
}

Pouvez vous m'aider?

Merci d'avance

Nixonne

**thewho** · 08/12/2006, 18h50

Bonjour,

Sans aller très loin, une simple lecture en diagonale montre

Code :

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

  char* Chainefinal;

suivi de l'allocation

Code :

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

Chainefinal=(char *)malloc(sizeof(char) * taille_fichier);

mais surtout, juste après

Code :

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

Chainefinal="";

et Chainefinal étant un pointeur...

**nixonne** · 08/12/2006, 21h30

Merci bien mais si je ne met pas le Chainefinal=""; j'ai des caractères qui sont sont dedans et je ne sais pas comment les enlever.

Nixonne

**malves** · 08/12/2006, 21h40

Salut.

Ton ChaineFinal est un pointeur vers une chaine de caractère. Il pointe donc vers une adresse en mémoire qui est susceptible de contenir beaucoup de bazard après le malloc.

Ecrire: Chainefinal=""; est incorect car ChaineFinal représente une adresse...

Pour résoudre ton problème un simple

Code :

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