Discussion :

[Looney]

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 :

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
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

[LittleWhite]

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.

[Looney]

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 :

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
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

[loufoque]

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).

[LittleWhite]

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

Code :

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

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.

[Looney]

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

Je marque le sujet en résolu.