Discussion :

[bambam978]

Bonjour,

je suis entrain de réaliser un projet informatique pour mon dut geii qui porte sur la cryptographie. Le but de ce projet est d'insérer dans une image au format .BMP un message caché grâce à la transparence des couleurs.

la première partie du projet consiste à déterminer si l'image est bien au format .BMP . voici comment se présente ma fonction d'analyse :

Fichier Header tete.h :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
//définition des types de base.
typedef unsigned char byte;
typedef unsigned short int word;
typedef unsigned long int dword;
//structure de l'entete fichier.
typedef struct tete_f {
        byte signature[2];
        dword taille;
        dword inconnu;
        dword offset;
        }tete_f;

Fonction Main :

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "tete.h"
 
//initialisation des fonctions du programme.
int LitHeader(FILE *out);
 
int main(int argc, char *argv[])
{
    FILE *out;
 
    out=fopen("picto.bmp","rb");
 
    if (LitHeader(&*out)==1){
                    ...
       }
    else {
         printf("L'image choisit n'est pas valide.\n");
         system("PAUSE");
         return 0;
         }
 
  fclose(out);
  system("PAUSE");	
  return 0;
}

Voici la fonction LitHeader appelé par main :

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
int LitHeader(FILE *out)
{
    tete_f BMP_Header;    
 
    fread(BMP_Header.signature,2,1,out);
    printf ("signature de fichier 1. : (%c) [%c]\n",BMP_Header.signature[0],BMP_Header.signature[1]);
 
    if (fread(&BMP_Header.signature[0],2,1,out)=='B'&& fread(&BMP_Header.signature[1],2,1,out)=='M'){
       printf("je passe par le valide\n");
       return 1;
       }
    else{
       printf ("signature de fichier 2. : (%c) [%c]\n",BMP_Header.signature[0],BMP_Header.signature[1]);
       printf("je passe par le non valide\n");
       return 0;
       }
}

Son rôle est de déterminer si les deux premiers octets contenus dans le fichier "Picto.bmp" sont bien 'B' et 'M', si c'est le cas, la fonction renvoie '1' à la fonction main. Sinon la fonction renvoie '0'.

J'ai mis en évidence mon problème en programmant deux fonctions "printf", une située en début de fonction, numérotée "1." , la seconde, située après la condition du "if", dans "else", numérotée "2.".

Le programme détermine bien les lettres "B" et "M" au niveau des deux premiers octets lors de la première vérification (numérotée 1.), mais l'image n'est pas validée comme étant de format .bmp et passe alors par "else" avec pour nouvelles valeurs de signature "ê" et "N".

Je n'ai fait que lire le fichier .bmp et poser des conditions. pourquoi les valeurs sont-elles modifiées ?

[diogene]

Code :

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

1
2
3
4
5
6
7
8
9
 ....
   fread(BMP_Header.signature,2,1,out);
    printf ("signature de fichier 1. : (%c) [%c]\n",BMP_Header.signature[0],BMP_Header.signature[1]);
 
    if (fread(&BMP_Header.signature[0],2,1,out)=='B'&& fread(&BMP_Header.signature[1],2,1,out)=='M'){
       printf("je passe par le valide\n");
       return 1;
       }
....

Pourquoi deux fread ? Le premier charge la signature dans BMP_Header.signature , il suffit ensuite de tester :

Code :

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

1
2
3
4
5
6
7
8
9
 ....
   fread(BMP_Header.signature,2,1,out);
    printf ("signature de fichier 1. : (%c) [%c]\n",BMP_Header.signature[0],BMP_Header.signature[1]);
 
    if (BMP_Header.signature[0]=='B'&& BMP_Header.signature[1]=='M'){
       printf("je passe par le valide\n");
       return 1;
       }
....

[bambam978]

Ah oui. En effet, c'était bien l'appel successifs de la fonction fread qui, apparemment, modifiait la valeur de ma signature... pourquoi ?

Ma fonction LitHeader ne detecte pas les fichiers .bmp . la condition du "if" est-elle comprise par le programme ?

[diogene]

Ah oui. En effet, c'était bien l'appel successifs de la fonction fread qui, apparemment, modifiait la valeur de ma signature... pourquoi ?

Ma fonction LitHeader ne detecte pas les fichiers .bmp . la condition du "if" est-elle comprise par le programme ?

Parce que tu lis avec le deuxième fread la suite du fichier qui naturellement n'est pas "BM" puisque qu'il a déjà été lu par le premier fread !

Je ne comprend pas la question sur le if

[tonton fred]

Salut

Ah oui. En effet, c'était bien l'appel successifs de la fonction fread qui, apparemment, modifiait la valeur de ma signature... pourquoi ?

Lors du premier appel tu lis 2 octets en partant de l'offset du fichier qui vaut 0. Lorsque fread retourne, l'offset du fichier est de +2 octets.
Lors de l'appel suivant, tu lis 2 octets en partant de l'offset du fichier qui vaut +2.

[bambam978]

très bien merci pour ces renseignements.

voici mon code actuel :

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
int LitHeader(FILE *out)
{
    tete_f BMP_Header;    
 
    fread(BMP_Header.signature,2,1,out);
 
    if ((&BMP_Header.signature[0],2,1,out)=='B' && (&BMP_Header.signature[1],2,1,out)=='M'){
       printf("Le fichier est au format .BMP\n");
       return 1;
       }
    else{
       printf("Le fichier n'est pas au format .BMP\n");
       return 0;
       }
}

ma seconde question portait sur la condition de mon "if". Lors de la lecture d'un fichier .BMP, le message renvoyé par le programme est "Le fichier n'est pas au format .BMP", par contre, la signature est bien 'B' 'M'.
Ma condition ne doit donc pas être valable. Comment dois-je l'améliorer ?

[bredelet]

Je crois qu'il y a mauvaise utilisation des pointeurs.

D'abord tu veux lire le header dans la structure BMP_Header que tu as déclarée. Il faudrait indiquer à fread l'adresse de cette structure. On utilise & pour obtenir l'adresse d'une variable:

Code :

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

fread(&BMP_Header,2,1,out);

Quand on a ça tu peux commencer à regarder les données stockées dans la structure. Maintenant on n'a plus besoin de &:

Code :

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

if (BMP_Header.signature[0]=='B' && BMP_Header.signature[1]=='M'){

[tonton fred]

Code :

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

1
2
 
    if ((&BMP_Header.signature[0],2,1,out)=='B' && (&BMP_Header.signature[1],2,1,out)=='M'){

Ma condition ne doit donc pas être valable. Comment dois-je l'améliorer ?

Effectivement. Tu compares out et 'B'...

Code :

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

if ((&BMP_Header.signature[0],2,1,out)=='B')

Tu as oublie de corriger ta condition en enlevant fread

[bambam978]

oO ... eurf. merci ^^

Mon dernier problème en plus de celui-ci était que j'utilisai l'adresse du tableaux au lieu de la valeur qu'il possédait.

Ma fonction marche maintenant. la voici :

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
int LitHeader(FILE *out)
{
    tete_f BMP_Header;    
 
    fread(BMP_Header.signature,2,1,out);
 
    if ((BMP_Header.signature[0])=='B' && (BMP_Header.signature[1])=='M'){
       printf("Le fichier est au format .BMP\n");
       return 1;
       }
    else{
       printf("Le fichier n'est pas au format .BMP\n");
       return 0;
       }
}

Merci pour votre aide.