Discussion :

[zais_ethael]

Salut à tous.
Voila, je dois faire des lectures binaires sur un flux de la stl. Pour cela, je commence par l'ouvrir en binaire puis j'utilise une fonction pour lire les données au fur et à mesure. J'ai mis au point deux versions de cette fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
	int yaread(struct SDL_RWops *context, void *ptr, int size, int maxnum) {
		std::istream* s = (std::istream*)context->hidden.unknown.data1;
		int taille=0;
		while(!s->eof() && taille<size*maxnum) {
			((char*)ptr)[taille] = s->get();
			taille++;
		}
		return taille/size;
	}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
	int yaread(struct SDL_RWops *context, void *ptr, int size, int maxnum) {
		std::istream* s = (std::istream*)context->hidden.unknown.data1;
		int beg = s->tellg();
		s->seekg(0,ios::end);
		int end = s->tellg();
		s->seekg(beg,ios::beg);
		int taille = size*maxnum<=end-beg?size*maxnum:end-beg;
		s->read((char*)ptr,taille);
		return taille/size;
	}

Ne vous préoccupez pas trop des deux premières lignes ni de la valeur de retour, cette fonction est appellée par une bibliothèque qui n'est pas de moi.
Il faut savoir que ces deux versions fonctionnent, j'arrive à charger correctement mon image (parceque c'est une image) et à l'afficher. Ensuite, l'objet contenant cette fonction est détruit, le flux est fermé et mon programme continue son exécution (c'est un jeu de société, ca peut prendre longtemps).
Le problème est que si j'utilise la version numéro 2 de cette fonction, le destructeur du flux provoque un plantage en accédant à une portion de mémoire invalide (je rapelle que ce destructeur est appelé longtemps après l'exécution de ma fonction.
Je pourrais bien sur n'utiliser que la première version de cette fonction, mais elle est incroyablement lente (forcément, si on lit les octets un à un...).

Quelqu'un verrait-il ou est l'erreur? J'utilise visual studio 2005.

[Médinoc]

Le stream est bien en binaire? Je ne sais plus trop sur quelle plate-forme ou avec quelle version, mais il me semble que certaines classes de stream ont des problèmes avec seekg()/tellg() quand elles sont ouvertes en mode texte...

Et aussi, je ne suis pas certain de la priorité du ?: et du >=, tu devrais peut-être t'en assurer avec des parenthèses...

[zais_ethael]

Non non, rien à voir avec ca.
Je dis bien que ces deux implémentations fonctionnent parfaitement. D'ailleurs la portion de code servant à vérifier si je ne suis pas arrivé à la fin de mon fichier ne sert quasiment à rien puisque je peux constater avec un peu de logging que la bibliothèque appelant ces fonctions ne demande jamais plus que la taille du fichier (c'est une biblio de lecture de JPG, je suppose qu'il est marqué quelquepart au début du fichier la taille qu'il fait).
Le seul problème est que la seconde implémentation semble "déteriorer" le flux. Ce qui fait qu'à sa destruction (qui se produit longtemps après sa fermeture) son destructeur va causer un plantage.

Edit: ca vient justement de me donner une petite idée. J'ai viré cette fameuse partie du code et visiblement le résultat est exactement le même. On dirait que c'est la fonction read() qui pose problème.

[zais_ethael]

Ca va, je crois avoir trouvé une solution. Ma

[zais_ethael]

Ca va, je crois avoir trouvé une solution. Ma fonction est devenue:

Code :

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

1
2
3
4
5
	int yaread(struct SDL_RWops *context, void *ptr, int size, int maxnum) {
		std::istream* s = (std::istream*)context->hidden.unknown.data1;
 
		return s->rdbuf()->sgetn((char*)ptr,size*maxnum)/size;
	}

En plus, contrairement à istream.read() je recois directement le nombre de bytes lus. Par contre il me semble que j'avais déja eu des problèmes avec certaines implémentations de la stl qui ne permettaient pas de récupérer un pointeur le buffer. Enfin, je verrais bien quand je testerais sous gcc.

[zais_ethael]

Et m***, ca cesse de fonctionner dés que je compile en mode release. Je commence à en vouloir à visual studio la

[Luc Hermitte]

Plus simple que ton dernier truc, tu as std::istream::readsome(), non ?
Je suis sceptique sur ta première fonction, à cause du eof qui intervient avant le get(), soit trop tard.

Sinon, il n'y a pas de problèmes de mélanges de reinterpret_cast alors que tu es en plein héritage, possiblement multiple (si std::fstream, sans o ni i) ? Utilises-tu la bonne vtable ? -> si tu affiches la valeur du pointeur sur std::istream avant et après obsfuscation via un void* (si je comprends bien).

Evidemment, il faut s'assurer que le type du pointeur officiellement stocké soit réutilisé avant utilisation, et pas un autre.