Traiter une erreur "Segmentation Fault"

Bonjour,

Je vous explique mon problème :

Je porte une classe que j'avais développée en vb.net pour gérer des fichiers logs en Linux C++. Jusque la pas de soucis. Je suis +- débutant on peut dire en C++.

Mon problème vient lorsque je tente d'initier mon fichier log, si le chemin du fichier se trouve dans un répertoire ou j'ai les droits, pas de soucis, mais si je mets comme chemin un répertoire ou je n'ai pas les droits, je reçois "Segmentation Fault". Qq part logique, mais j'aimerai pouvoir traiter cette erreur avec un try catch, mais il ne la prends pas.

Si je lance avec sudo ./main la ca marche sans problème.

Qqn a une idée ou a déjà expérimenté ce problème ?

voici un bout de code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

bool CLogfile::Initlogfile() { try { if(!FileExist(m_filepath)) { FILE *myfile; myfile=fopen(m_filepath, "w+t"); fprintf(myfile, "Initiation of log file- %s\n", m_filepath); // c'est ici que le Segmentation fault apparait fclose(myfile); return true; } } catch(...) { printf("An error has occured!\n"); return false; } }

---

Merci d'avance,

Looney

Bonjour,

En fait, c'est juste que tu ne verifie pas si fopen ouvre correctement le fichier.
Comme pour les fichiers dont tu n'as pas le droit d'ouverture, le fopen() va echouer, donc te renvoyer un pointeur NULL. ( Que tu va essayais d'utiliser ) => donc segmentation fault.

Pour cela se produit, c'est que la fonction FileExist te renverra true, même si tu n'as pas le droit d'ouvrir, car le fichier existe.

Donc pour eviter le segmentation fault , il faut que tu verifie après le fopen() si myfile == NULL.

Sinon, le try catch ne marcherai pas , car fopen ne renvoie pas d'exception. D'ailleurs, si tu fais du C++ ( try catch ce n'est qu'en C++ pas en C ) tu devrais utiliser les flux. ( fstream ). Car fopen est une fonction C, et donc comme les exceptions sont C++ , fopen ne renverra jamais d'exception.

Bonjour Monsieur Littlewhite,

Merci pour ta réponse rapide, effectivement en testant myfile, je vois bien qu'il est ==NULL donc ca marche.

Avant de mettre resolved, j'avais qd mm une derniere petite question, suivant tes conseils pr écrire en c++ j'ai remplacé mon fopen par un ofstream

voici un petit bout de code pr test :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

try { const char* m_filepath; m_filepath="/usr/test.log";   ofstream myfile(m_filepath, ios::app);   if(myfile!=NULL) myfile << "Initiation of log file - " << m_filepath << "\n"; else printf("Could not create file\n"); } catch(...) { printf("An error has occured while opening or writing to file\n"); }

---

Je m'attendais avant de mettre mon test sur myfile!=NULL a sortir une exception, mais même pas. Alors y a-t-il moyen comme en vb.net de voir la raison pr laquelle le fichier n'a pas pu etre ouvert/écrit ou je pense encore trop vb.net ?

D'avance je vous remercie,

Looney

Le code dans ton try ne peut lèver aucun exception.
En C++, une exception ne peut être levée qu'avec throw.

Invoquer des comportements indéfinis ne lève pas des exceptions, ça fait des comportements indéfinis (ça peut marcher, corrompre ton disque, ou planter).

Pour tester si le fichier est ouvert, en C++, on fait:

Code:

---

1 2 3

if ( ! myfile.isOpen() ) // Fichier pas ouvert ...

---

Je n'utilise pas de try catch, pour ces commandes, peut être que je devrai, mais la verification avec isOpen() me semble suffire.

Ok ben ca marche, et un grand merci pour vos lumières en tous cas.

Je marque le sujet en résolu.