Discussion :

[tazimovski773]

Bonjour,

j'essaie depuis quelque temps de créer un petit programme de cryptage XOR (ou exclusif), avec la conception suivante :
On a un fichier qui contient du texte quand veux crypter avec un algorithme de cryptage XOR à l'aide d'une clé numérique contenue dans un 2ème fichier. Le programme doit créer un 3ème fichier qui est le fichier crypté.

J'ai crée beaucoup de versions de ce programme avec des résultats plus ou moins mitigés, mais la version la plus aboutie est ci-jointe avec ce post.
Le code se compile sans problème mais pour une raison obscure le programme n'arrive pas à lire correctement la clé à partir du fichier qui la contient. Et svp avant de faire la compilation le fichier qu'on veux crypter et le fichier clé doivent déjà être créés au préalable.

Merci de me répondre dans les plus brefs délais et n'oubliez pas que c'est mon premier post dans ce forum !

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
void CryptageFichiersOuverts(FILE *pfIn, FILE *pfOut, int* clef)
{
	int carac;
	clef = malloc (8 * sizeof(int));
	while((carac = fgetc(pfIn)) != EOF)
	{
		int i;
		for (i=0;i<8;i++)
		{
			int xored = carac ^ clef[i];
			fputc(xored, pfOut);
		}
	}
}
 
void Cryptage(const char *nomFichSrc, const char *nomFichDest, int* clef)
{
	FILE *pfIn;
	pfIn = fopen(nomFichSrc, "r");
	if(pfIn == NULL)
	{
		puts("Echec d'ouverture du fichier source.");
	}
	else
	{
		FILE *pfOut =NULL;
		pfOut = fopen(nomFichDest, "w");
		if(pfOut == NULL)
		{
			puts("Echec d'ouverture du fichier destination.");
		}
		else
		{
			CryptageFichiersOuverts(pfIn, pfOut, clef);
			fclose(pfOut);
		}
 
		fclose(pfIn);
	}
 
}
 
void DemandeEtCrypte()
{
	char nomFich[300] = "";
	char *pRetour = NULL;
	int clef [8];
 
	printf("Nom du fichier a crypter : \n");
	fflush(stdout); // Nécessaire si une ligne ne se termine pas par \n - fflush(stdout) : fonction qui vide le tampon de sortie.
 
	/* On lit le nom du fichier, 
	   puis on vérifie qu'il rentre en entier dans la zone de 300 caractères.
	   S'il rentre, le \n sera dedans, et il faudra le supprimer.
	   Si ça ne rentre pas, le \n ne sera pas dedans, et on dira Erreur. */
 
       fgets (nomFich, 300, stdin);
	pRetour = strchr(nomFich, '\n');
	if(pRetour == NULL)
	{
		// Le \n final n'est pas dans le nom de fichier.
		puts("Nom de fichier trop grand.\n");
	}
	else
	{
		// Le \n est présent : OK, on tronque la chaîne ici.
		*pRetour = '\0';
		FILE *pfclef;
		pfclef = fopen("clef.txt", "r");
		if (pfclef != NULL)
		{
			/*int j;
			for (j=0;j<8;j++)
			{	
				scanf ("%d", &clef[j]);
			}*/
			fscanf (pfclef, "%s" , clef);
			fclose (pfclef);
		}
		 else
		{
		 printf ("Erreur d'ouverture du fichier\n");
		}
		// On lit la clef.
 
		/* On lance le cryptage du fichier, et on met le résultat dans "fichier_crypte.txt".
		   Si on veut, on peut demander le nom du fichier destination à la place... */
		Cryptage(nomFich, "crypte.txt", clef);	
		printf ("fichier crypte\n");
	}
}
 
int main(int argc, char *argv[]) {
	DemandeEtCrypte();
	return 0; 
}

[CodeurPlusPlus]

Je vois une incohérence. clef est déclaré tableau d'entiers... donc pourquoi scanf ("%s", clef) ?

[Kannagi]

Je me pose juste la question pourquoi tu ne fais tout simplement pas un xor directement pendant la lecture/ecriture , en gros tu lis , tu applique le XOR et tu écris ensuite (sur le même fichier), il existe "r+" pour pouvoir lire et écrire sur le même fichier .

[tazimovski773]

Bonjour,

merci de vos vaillants efforts, mais j'ai un peu amélioré le code en introduisant la fonction "atoi" pour faciliter la lecture de la clef numérique. J'ai joint ci-dessous ma dernière version du code.

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
void CryptageFichiersOuverts(FILE *pfIn, FILE *pfOut, int clef)
{
	int carac;
	while((carac = fgetc(pfIn)) != EOF)
	{
		int xored = carac ^ clef;
		fputc(xored, pfOut);
	}
 
}
 
void Cryptage(const char *nomFichSrc, const char *nomFichDest, int clef)
{
	FILE *pfIn = fopen(nomFichSrc, "r");
	if(pfIn == NULL)
	{
		puts("Echec d'ouverture du fichier source.");
	}
	else
	{
		FILE *pfOut =NULL;
		pfOut= fopen(nomFichDest, "w");
		if(pfOut == NULL)
		{
			puts("Echec d'ouverture du fichier destination.");
		}
		else
		{
			CryptageFichiersOuverts(pfIn, pfOut, clef);
			fclose(pfOut);
		}
 
		fclose(pfIn);
	}
 
}
 
void DemandeEtCrypte()
{
	char nomFich[300] = "";
	char *pRetour = NULL;
	int clef;
 
	printf("Nom du fichier a crypter : \n");
	fflush(stdout); // Nécessaire si une ligne ne se termine pas par \n - fflush(stdout) : fonction qui vide le tampon de sortie.
 
	/* On lit le nom du fichier, 
	   puis on vérifie qu'il rentre en entier dans la zone de 300 caractères.
	   S'il rentre, le \n sera dedans, et il faudra le supprimer.
	   Si ça ne rentre pas, le \n ne sera pas dedans, et on dira Erreur. */
 
	fgets(nomFich, 300, stdin);
	pRetour = strchr(nomFich, '\n');
	if(pRetour == NULL)
	{
		// Le \n final n'est pas dans le nom de fichier.
		puts("Nom de fichier trop grand.\n");
	}
	else
	{
		// Le \n est présent : OK, on tronque la chaîne ici.
		*pRetour = '\0';
 
		FILE *pfclef;
		pfclef = fopen("clef.txt", "r");
		if (pfclef != NULL){
			char nomClef[8]="";
 
			fgets (nomClef,8,pfclef);
			clef=atoi(nomClef);
			fclose (pfclef);
		}
		 else{
		 	printf ("Erreur d'ouverture du fichier\n");
		}
		Cryptage(nomFich, "crypte.txt", clef);
		printf ("fichier crypté\n");	
		}	
}
 
int main(int argc, char *argv[]) {
	DemandeEtCrypte();
	return 0; 
}

A +.

[Médinoc]

Il faudrait savoir ce que tu manipules. Tu ouvres tes fichiers en mode texte, mais la sortie d'un cryptage XOR ordinaire n'est pas du texte, mais un tableau d'octets (peut contenir des caractères nuls, des caractères non-imprimables, et des séquences d'octets interdites dans l'encodage utilisé). C'est pourquoi généralement, quand on bosse en XOR, on bosse sur des fichiers binaires et non texte.