Discussion :

[Alga007]

Bonjour,
Je travaille actuellement sur une ouverture de liaison série sous qt, j'ouvre avec un "open", et je voudrais donc fermer avec un "close". Mais voilà, le compilateur me dit: segmentation fault.
Voici le morceau de code concerné:

Code :

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

1
2
3
4
5
6
void dialMeteo::initLiaison()
{
 int ret;
 if ( fd > 0) close(fd);  // si déjà ouverte
 ret = dialMeteo :: ouvrir liaison();
}

fd est déclaré dans le .h en public, l'include du .h est fait. Je ne vois pas trop ou peut ete le problème, donc si qqun peut m'aider.
Merci, Alga

[yan]

Bonjour.
IL manque des expliquations...
Qu'es ce que fd???
Comment est il initialisé???

[Alga007]

Oui désolé....
Fd est le descripteur de fichier qui est retourné par la fonction open

J'utilise la ligne suivante dans une autre fonction du programme:

fd = open ("/dev/ttyS0", O_RDWR | O_NONBLOCK);

En debug, il est bien égal à 0 avant que j'ai ouvert la liaison, puis passe à une valeur positive. On rentre donc dans le if, mais il y a cette erreur.

[yan]

Sans plus de code.
Je ne vois pas comment l'on peut t'aider.
Puis cela ne semble pas etre un problème Qt.

[Alga007]

Ok ben merci quand meme. Mais pour mettre tout le code, se serait trop long. Et puis à mon avis ce c'est pas non plus un probleme qt, je pense que cela vient soit du kdevelop, soit du code en c.
Je verrais bien

[yan]

Pour commencer, verifie qu tu initialise bien fd.
Ca, ça ne compile sûrement pas :

Code :

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

ret = dialMeteo :: ouvrir liaison();

Si tu veut je peu déplacer ton poste, mais il faudrait que tu dise quel lib tu utilise et être un peu plus précis.

[Alga007]

Oui je veux bien que tu déplaces le poste, j'utilise la suse 10.2 de linux, et la librairie Qt 3.
Pour ce qui est de la compilation, tout compile, mais après je ne dis pas que c'est juste. Le seul problème de compilation que j'ai est ce "erreur de segmentation".
L'article serait déplacé ou? Je ne connais pas encore bien le site, ni le fonctionnement en détails.

[yan]

Oui je veux bien que tu déplaces le poste, j'utilise la suse 10.2 de linux, et la librairie Qt 3.
Pour ce qui est de la compilation, tout compile, mais après je ne dis pas que c'est juste. Le seul problème de compilation que j'ai est ce "erreur de segmentation".
L'article serait déplacé ou? Je ne connais pas encore bien le site, ni le fonctionnement en détails.

Ben déja dit quel lib tu utilise. Fd appartiens à quel lib?
C'est un problème en C ou en C++?

[IrmatDen]

Ben déja dit quel lib tu utilise. Fd appartiens à quel lib?
C'est un problème en C ou en C++?

open/close et les file descriptor c'est du C système (Edit: appliqué à linux dans le cas présent bien sûr)

[Alga007]

fd est un int que j'ai déclaré dans le .h :

Code :

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

1
2
3
4
class dialMeteo
{
 public: int fd;
}

Je l'initialise à 0 dans le constructeur de la classe:

Code :

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

1
2
3
4
dialMeteo::dialMeteo(...........)
{
 fd = 0;
}

Et je l'utilise comme suit:

Code :

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

fd = open(.........);

Je ne vois pas comment détaillé plus, et concernant a librairie il n'appartient à aucune, je l'ai juste déclaré.

[Fred83]

C'est sûrement pas la cause du plantage mais la valeur 0 est valide pour un descripteur de fichier retourné par la fonction open.
regarde le man de open().
Initialise fd à -1 et modifie les tests en conséquence.

Après le close(), il faudrait remettre fd=-1 pour éviter de refermer ultérieurement un descripteur devenu invalide.

Voila, sans plus de code difficile de t'aider plus.

Bon courage.

[Alga007]

Ben en fait j'étais avant sur le forum C++, et on m'a déplacé ici car le problème ne semble pas venir du C++, mais du code en C, de l'utilisation du close. Je vais tenter de l'initialiser à -1, mais à mon avis ça ne reglera pas le pb, car ce n'est pas le test qui n'est pas bon, mais vraiment l'instruction.

[Luc Hermitte]

Cela sent une mauvaise gestion des cycles de vie de fd.
Débugge ou rajoute une trace écran à chaque fois que tu manipules cette donnée -- en précisant bien valeur de fd, et focntion où tu es.

[yan]

Pas du C. Je déplace en C++ ?

Il avait poste dans le forum Qt.

Comme tu le sens mais comme le problème viens de open. Pour moi, c'est plustôt un problème C.

[cfdev]

salut,

mais le pb ne vient surement pas sur le morceau de code posté...aprés le close(fd), il faut vérifier que le handle n'est plus utilisé pour faire un read par exemple

++

[Emmanuel Deloget]

salut,

mais le pb ne vient surement pas sur le morceau de code posté...aprés le close(fd), il faut vérifier que le handle n'est plus utilisé pour faire un read par exemple

++

C'est une autre possibilité. Plus trivial, il semble que l'instance de dialMeteo n'existe pas (soit elle a été détruite avant, soit elle est nulle au moment de l'appel).

Généralement, un seg-fault, ça se debug. On vérifie la call-stack, on reproduit le problème, on regarde qui appelle quoi et dans quel ordre, on met des points d'arrêt, etc.