problème sur l'affectation d'un path

Bonsoir,

j'ai un problème qui me prend la tête, je vous le soumets en espérant que vos cerveaux seront mieux équipés que le mien...

voilà, j'ai une structure dans laquelle j'ai un objet path :

Code:

---

1 2 3 4 5

typedef struct fichiers { boost::filesystem::path path; /* ... */ } fichier;

---

et j'ai la fonction suivante :

Code:

---

1 2 3 4 5 6 7

fichier * push(const boost::filesystem::path& _path) { fichier *f = (fichier*)malloc(sizeof(fichier)); if(!f) exit(1); // Si l'allocation a échoué f->path = _path; return f; }

---

Cette fonction est appelée plusieurs fois, et à un certain moment (alors que le path semble valide, je peux l'afficher avec std::cout), ça plante à l'exécution sur ça :

Citation:

---

Erreur de segmentation (core dumped)

---

A noter deux choses quand même :
1) le répertoire sur lequel ça bloque ne change pas d'une fois sur l'autre
2) en revanche, lorsque je modifie la fonction et que je lui passe en argument le path au format std::string, puis que je fais

Code:

---

f->path = boost::filesystem::path(_path);

---

alors là, ça plante quelques répertoires plus loin, et l'erreur est la suivante :

Citation:

---

*** glibc detected *** ./Affichage: free(): invalid pointer: 0x080d647c ***
======= Backtrace: =========
/* ici la trace */
======= Memory map: ========
/* les infos mémoire */
Abandon (core dumped)

---

Et là je ne vois pas le pourquoi du comment... Quelqu'un a une idée de ce que j'ai mal fait ?


Merci d'avance !!

Citation:

---

Envoyé par zarbiman

Code:

---

1 2 3 4 5

typedef struct fichiers { boost::filesystem::path path; /* ... */ } fichier;

---

---

Pourquoi le typedef en C++?

Citation:

---

Envoyé par zarbiman

Code:

---

1 2 3

fichier * push(const boost::filesystem::path& _path) { fichier *f = (fichier*)malloc(sizeof(fichier));

---

Et là je ne vois pas le pourquoi du comment... Quelqu'un a une idée de ce que j'ai mal fait ?

---

Est-ce que tu as quelque part une garantie que boost::filesystem::path est un type POD?

Pourquoi un malloc? Pourquoi pas new?

Surtout : est-ce du C ou du C++?

Ce qu'il fait c'est du C compilé avec un compilateur C++.
Donc soit tu passe au C et tu oubli boost, soit tu garde boost mais du fait du vrai C++.

Effectivement il y a une confusion de ma part : le projet est en C++, mais comme je débute avec ce langage, j'ai utilisé à certains endroits des morceaux de code C que je maîtrise mieux...

J'ai donc utilisé un typedef, car je voulais une liste chaînée, et je pensais que ce qui était faisable en C l'était également en C++...
Mais du coup, je ne sais pas comment adapter ce que je faisais en C pour que ça fonctionne en C++.

Pour information, j'ai utilisé une liste chaînée plutôt qu'une classe car j'ai eu des problèmes lorsque j'ai essayé la classe. De mémoire ça devait ressembler à quelque chose dans ce genre :

Code:

---

1 2 3 4 5 6 7

class fichier { private: boost::filesystem::path path; fichier suivant; /* ... */ }

---

au lieu de :

Code:

---

1 2 3 4 5 6

typedef struct fichiers { boost::filesystem::path path; struct fichiers *suivant; /* ... */ } fichier;

---

Et ça ne fonctionnait pas car le compilateur ne reconnaissait pas le type fichier utilisé dans la classe fichier. Et moi je voulais un pointeur vers l'élément suivant...

Citation:

---

Envoyé par zarbiman

J'ai donc utilisé un typedef, car je voulais une liste chaînée, et je pensais que ce qui était faisable en C l'était également en C++...

---

C'est généralement faisable, mais c'est souvent une solution sous optimale. Si tu veux une liste chaînée de fichiers en C++, c'est déjà prêt à l'emploi :

Code:

---

std::list<fichier>

---

(il faut par contre dans ce cas que les objets fichiers soient copiables correctement).

Citation:

---

Envoyé par zarbiman

Code:

---

1 2 3 4 5 6 7

class fichier { private: boost::filesystem::path path; fichier suivant; /* ... */ }

---

Et moi je voulais un pointeur vers l'élément suivant...

---

Là, tu n'a pas mis un pointeur, mais directement un objet du type en question, ce qui ne peut pas marcher.