Problème extraction d'un fichier binaire

**rayon-gama** · 01/04/2013, 11h54

Bonjour, bonsoir je viens ici car je rencontre un problème assez déroutant et inexpliqué ... Mon but étais de créer un code java permettant de récupérer une image stocké dans un binaire, pour cela nous devions renseignez au code l'adresse de départ de l'image et l'adresse de fin de l'image dans le binaire, et bien sur le nom du binaire. Oui mais voila lors de cette récupération la plus part des caractère sont remplacé par des 00 (en hexa). Même l'entête de l'image avez disparus. Ne comprenant pas j'ai essayé de copier la totalité de mon binaire et là à ma grande surprise, certain caractère était remplacé par des 00... et oui encore.

Voici mon 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
import java.io.File;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.RandomAccessFile;
import java.util.Scanner;
 
 
public class Main {
    public static void main(String[] args) throws IOException {
        Scanner scan = new Scanner(System.in);
        RandomAccessFile bFile = null;
        FileOutputStream pFile = null;
        System.out.print("Enter your binary file : ");
        try {
            bFile = new RandomAccessFile(new File(scan.nextLine()), "r");
        } catch (FileNotFoundException e) {
            System.err.println("File not found !");
            e.printStackTrace();
        }
        System.out.print("Enter your output file (with extention) : ");
        try {
            pFile = new FileOutputStream(scan.nextLine());
        } catch (FileNotFoundException e1) {
            // TODO Auto-generated catch block
            e1.printStackTrace();
        }
        System.out.print("Enter start address : ");
        long startAddress = Long.parseLong(scan.nextLine(),16);
        System.out.println(startAddress);
        System.out.print("Enter stop address :");
        long stopAddress = Long.parseLong(scan.nextLine(),16);
 
        bFile.seek(startAddress);
        byte[] buf = new byte[(int) (stopAddress - startAddress)];
        bFile.read(buf);
        pFile.write(buf);
        try {
            bFile.close();
            pFile.close();
        } catch (IOException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
 
    }
}

**thelvin** · 01/04/2013, 13h00

Bonjour,

La méthode read() ne remplit pas le buffer. Elle ne peut pas d'ailleurs : si tu étais près de la fin du fichier, le fichier ne contiendrait pas assez d'octets pour remplir le buffer.
Vu que le buffer n'est pas rempli, les octets qui n'ont pas été touchés sont restés à 0.

Quand on appelle read(), on ne sait pas combien d'octets seront lus. Ca peut être 1, ça peut être 24643, on sait pas. Il lira au moins 1 octet, et il ne lira pas plus que la taille du buffer. Mais ça peut être n'importe quoi entre les deux. Pour savoir combien d'octets ont été lus, il faut regarder l'int renvoyé par read().

Donc, solution :
- toujours regarder combien d'octets ont été lus
- appeler read() en boucle jusqu'à ce que tous les octets aient été lus.

**rayon-gama** · 01/04/2013, 15h11

Merci de ta réponse mais malgré tes conseils ça ne marche toujours pas voici ce que j'ai changé dans mon 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
 
import java.io.File;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.RandomAccessFile;
import java.util.Scanner;
 
 
public class Main {
	public static void main(String[] args) throws IOException {
		Scanner scan = new Scanner(System.in);
		RandomAccessFile bFile = null;
		FileOutputStream pFile = null;
		System.out.print("Enter your binary file : ");
		try {
			bFile = new RandomAccessFile(new File(scan.nextLine()), "r");
		} catch (FileNotFoundException e) {
			System.err.println("File not found !");
			e.printStackTrace();
		}
		System.out.print("Enter your output file (with extention) : ");
		try {
			pFile = new FileOutputStream(scan.nextLine());
		} catch (FileNotFoundException e1) {
			// TODO Auto-generated catch block
			e1.printStackTrace();
		}
		System.out.print("Enter start address : ");
		long startAddress = Long.parseLong(scan.nextLine(),16);
		System.out.println(startAddress);
		System.out.print("Enter stop address :");
		long stopAddress = Long.parseLong(scan.nextLine(),16);
 
		bFile.seek(startAddress);
		int taille = (int) (stopAddress - startAddress);
		byte[] buf = new byte[taille];
		int oLu = 0; 
		do{
			oLu = bFile.read(buf);
			System.out.println("taille : "+taille);
			System.out.println("oLu : "+oLu);
		}while(oLu != taille);
		pFile.write(buf);
		try {
			bFile.close();
			pFile.close();
		} catch (IOException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
 
	}
}

J'ai comme tu as dis boucler sur read() tant que celui ci n'avais pas remplie le buffer entièrement .. mais ça n'a rien changé toujours des 00.

**jdevbe** · 01/04/2013, 15h52

Tu oublie d’écrire les octets lus !
Ton code devrait être modifié comme suit

Code :

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

1
2
3
4
5
6
7
do{
  oLu = bFile.read(buf);
  System.out.println("taille : "+taille);
  System.out.println("oLu : "+oLu);
   pFile.write(buf,0,oLu);
}while(oLu  >0);
//pFile.write(buf); A supprimer

**rayon-gama** · 01/04/2013, 16h49

A votre code j'ai une jolie erreur :

Code :

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

1
2
3
4
5
 
Exception in thread "main" java.lang.IndexOutOfBoundsException
at java.io.FileOutputStream.writeBytes(Native Method)
at java.io.FileOutputStream.write(Unknown Source)
at Main.main(Main.java:43)

**jdevbe** · 01/04/2013, 18h13

Effectivement j'ai été trop vite.
En principe le code ci-dessous est plus juste

Code :

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

1
2
3
4
5
6
 
  while ((oLu = bFile.read(buf)) > 0) {
  System.out.println("taille : "+taille);
  System.out.println("oLu : "+oLu);
   pFile.write(buf,0,oLu);
 }

**rayon-gama** · 01/04/2013, 18h29

Toujours même que précédemment des 00 et certains caractères qui sont copier ... mais certain copier et qu'ils sont pas vraiment pas dans le binaire d'origine.

**jdevbe** · 01/04/2013, 19h09

Ma proposition de code permet d'utiliser un buffer plus petit et de contrôler que tous les bytes lus sont écrits. Dans votre code comme le buffer est de la taille des données à lire, sauf erreur de lecture ou taille de buffer, les données peuvent être lues et écrite en une fois sans boucles.
Puisque le résultat soit identique, c'est que les données lues ne sont pas structurées comme vous le supposez...

**rayon-gama** · 01/04/2013, 19h43

Je ne vois pas ou peut être un erreur de structure du fichier... car même lorsque je copie le binaire en entier et que je les compare avec un éditeur héxa les fichiers sont différents : certains caractères sont les mêmes, d'autres n'y sont pas, etc
C'est assez étrange je trouve ..

**jdevbe** · 01/04/2013, 20h19

Effectivement c'est étrange ! j'ai déjà utilisé ce type de copie entre un inputStream et un outpStream sans problème.

**thelvin** · 01/04/2013, 21h19

Moi je dis qu'il faut qu'on voir le code source complet comme il est maintenant. Il y a sûrement une erreur.

**rayon-gama** · 01/04/2013, 21h56

C'est bon tout marche normalement .. l'erreur ne venait pas de mon code, enfin si quand même aussi, mais il venais surtout de l’exécutable qui est impossible a copier sans altérer ses données et je ne comprend pas pourquoi ^^ Même en utilisant le copie du cmd le binaire était altérée .. en tout cas j'ai essayé sur un autre exécutable et ça marche. Donc merci pour toutes vôtres aides.

**rayon-gama** · 01/04/2013, 22h13

Après vérification il y a toujours un problème après avoir testé sur un gif contenue dans un binaire, (ça a fonctionné) j'ai essayer avec un png contenue dans un autre binaire, mais la copie c'est mal passé et le fichier sortant et illisible voici un extrait des deux codes hexa pour que vous voyiez bien :
Original (le png contenue dans le binaire) :

Puis le contenue après extraction et copie (celui du fichier sortant donc):

**thelvin** · 01/04/2013, 23h16

Rebonjour,

Je répète qu'il faudrait qu'on voie le code en entier tel qu'il est maintenant.

**rayon-gama** · 02/04/2013, 18h40

Je l'avais déjà donné un peux plus haut mais le revoici :

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
 
import java.io.File;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.RandomAccessFile;
import java.util.Scanner;
 
 
public class Main {
	public static void main(String[] args) throws IOException {
		Scanner scan = new Scanner(System.in);
		RandomAccessFile bFile = null;
		FileOutputStream pFile = null;
		System.out.print("Enter your binary file : ");
		try {
			bFile = new RandomAccessFile(new File(scan.nextLine()), "r");
		} catch (FileNotFoundException e) {
			System.err.println("File not found !");
			e.printStackTrace();
		}
		System.out.print("Enter your output file (with extention) : ");
		try {
			pFile = new FileOutputStream(scan.nextLine());
		} catch (FileNotFoundException e1) {
			// TODO Auto-generated catch block
			e1.printStackTrace();
		}
		System.out.print("Enter start address : ");
		long startAddress = Long.parseLong(scan.nextLine(),16);
		System.out.print("Enter stop address :");
		long stopAddress = Long.parseLong(scan.nextLine(),16);
 
		bFile.seek(startAddress);
		int taille = (int) (stopAddress - startAddress+1);
		byte[] buf = new byte[taille];
		int oLu = 0; 
		do{
			oLu = bFile.read(buf);
		}while(oLu != taille);
		pFile.write(buf);
		try {
			bFile.close();
			pFile.close();
		} catch (IOException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
 
	}
}

**thelvin** · 02/04/2013, 22h57

Envoyé par rayon-gama

Je l'avais déjà donné un peux plus haut

Voilà. Or, entretemps, jdevbe t'avais donné la solution.

Dont on ne voit aucune trace dans le code que tu viens fièrement de poster. A nouveau, oui, sans changement.

**rayon-gama** · 03/04/2013, 07h55

Si tu avais mieux lu tu verrais qu'un peux haut il ma dit que son code était pour un buffer plus petit étant donné que j'utilise un buffer beaucoup plus grand(de la taille de ce que je veux récupérer) je n'ai pas besoin de faire comme il avait codé et mon code est bon. De toute façon dans les deux cas j'obtenais le même résultat.

**thelvin** · 03/04/2013, 10h28

Envoyé par rayon-gama

Si tu avais mieux lu tu verrais qu'un peux haut il ma dit que son code était pour un buffer plus petit étant donné que j'utilise un buffer beaucoup plus grand(de la taille de ce que je veux récupérer) je n'ai pas besoin de faire comme il avait codé et mon code est bon.

J'ai très bien lu et ce raisonnement est faux, tout simplement. Avec un buffer de la bonne taille tu n'es pas obligé de coder comme il a fait, mais en tout cas sa manière à lui marche, et pas la tienne.

Envoyé par rayon-gama

De toute façon dans les deux cas j'obtenais le même résultat.

Pour la troisième fois, je demande à voir.

**hwoarang** · 03/04/2013, 10h52

Envoyé par rayon-gama

Code :

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

1
2
3
4
 		do{
			oLu = bFile.read(buf);
		}while(oLu != taille);
		pFile.write(buf);

La, il y a clairement un probleme de logique. D'apres la javadoc, read ne garantit pas que l'ensemble du tableau sera remplit. C'est d'ailleurs pour ca qu'il renvoit le nombre de byte lus.
Bref, sans rentrer dans les détails, quelque soit la taille du buffer, il faut plutot faire :

Code :

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

1
2
3
while (oLu < taille) {
oLu += bFile.read(buf, oLu, buf.length-oLu));
 }

Ceci dit, ca n'explique pas les problemes que tu observes. J'ai testé le code avec un fichier test et pour moi, ca marche (à la remarque que j'ai faite près). Est ce que tu peux poster les parametres que tu utilises pour le fichier (le string que tu comme debut et fin de fichier) ?

**rayon-gama** · 03/04/2013, 11h50

Pour que vous voyez exactement ce que ça fais je vais vous présenter un exemple que vous pouvez essayer de faire très facilement si vous êtes sur Windows ..
Vous lancer donc le code pour le binaire vous mettez : C:\Windows\System32\calc.exe (la calculatrice quoi..)
L'output : Vous mettez ce que vous voulez.
Le start address : 736C8
Et le stop address : 748E8
Et l'a maintenant vous ouvrez calc.exe avec un editeur hexa, vous allez à l'adresse 736C8 et vous verrez l'entête d'une entête PNG (prouvé avec ouverture de la calculatrice avec ResHacker ou j'ai trouvé les mêmes adresses pour cette image).
Puis vous lancer donc l'extraction de l'image avec mon code.
Et vous ouvrez le fichier extrait avec un éditeur hexa et vous comparez les deux hexa et vous verrez tout de suite mieux mon problème.
De plus je l'ai testé avec un logiciel ou j'avais mis moi même un PNG et ça à fait exactement le même problème.
P.S : Si vous voulez vous rendre compte encore plus du problème modifier un peux mon code pour qu'il copie l'intégralité du binaire et vous verrez que les hexas des deux binaires (l’orignal et la copie) sont différents.

Problème extraction d'un fichier binaire

Java

Discussions similaires

Partager

Partager