Discussion :

[Neuromancien2]

Bonjour,

Je débute en C. Je m'inspire du code proposé ici pour créer des classes me permettant de gérer facilement les sémaphores et segments de mémoire partagée.

Fichier ipcobj.h

Code :

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

1
2
3
4
5
6
7
8
9
struct sIPCObject{
	int iSemNbr;
	int iSemId;
};
 
struct sIPCObject *sIPCObject_new(int, int);
void sIPCObject_Delete(struct sIPCObject **this);
int sIPCObject_getSemNbr(struct sIPCObject *this);
void sIPCObject_CreateSem(struct sIPCObject *this);

Fichier ipcobj.c :

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
#include <sys/sem.h>
#include <sys/shm.h>
#include <ipcobj.h>
 
struct sIPCObject *sIPCObject_new(int iSemNbr, int iSemId) {
	struct sIPCObject *p = (struct sIPCObject *)malloc (sizeof (struct sIPCObject));
	if (!p)
		return NULL;
	p->iSemNbr = iSemNbr;
 
	return p;
}
 
void sIPCObject_Delete(struct sIPCObject **this) {
	if(*this)
		free(*this);
 
	*this = NULL;
}

Ensuite, je crée un objet sIPCObject avec :

Code :

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

1
2
3
4
struct sIPCObject *myIPCObjet;
int main(int argc, char *argv[]) {	
myIPCObjet = sIPCObject_new(1, 1);
sIPCObject_Delete(&myIPCObjet);

L'affectation de la variable iSemNbr me provoque un Memory fault(coredump).

[Franck.H]

Tu peux simplifier ton malloc ainsi (le cast n'est pas utile ici) :

Code :

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

struct sIPCObject *p = malloc (sizeof (*p));

Par ailleurs, je trouve tous ces struct bien lourd, tu peux éviter cela en faisant un typedef sur ta structure soit:

Code :

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

1
2
3
4
5
typedef struct {
	int iSemNbr;
	int iSemId;
}
sIPCObject;

Maintenant tu n'a plus besoin d'utiliser le mot clé struct comme par exemple, ton malloc deviendrais:

Code :

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

sIPCObject *p = malloc (sizeof (*p));

Ton main n'est par ailleurs pas complet et ton code est vraiment illisible:

Code :

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

1
2
3
4
5
6
7
int main(int argc, char *argv[])
{
   struct sIPCObject *myIPCObjet = sIPCObject_new (1, 1);
   sIPCObject_Delete (&myIPCObjet);
 
   return 0;
}

Evite les globales comme tu viens de le faire !


Une meilleure construction de ton constructeur serait:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
struct sIPCObject *sIPCObject_new (int iSemNbr, int iSemId)
{
   struct sIPCObject *p = malloc (sizeof (*p));
 
   if (p != NULL)
   {
      p->iSemNbr = iSemNbr;
   }
 
   return p;
}

Plusieurs return dans une même fonction n'est pas très joli surtout si ca peut être évité facilement


A part cela, je ne vois pas trop le problème, il ne devrait pas en avoir

[Neuromancien2]

Merci. J'ai toujours un core dump avec la fonction suivante :

Code :

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

1
2
3
4
5
6
7
int sIPCObject_getSemNbr(sIPCObject *this) {
	int iSemNbr = 0;
	if(this)
		iSemNbr = this->iSemNbr;
 
	return iSemNbr;
}

Appelée dans le main() avec :

Code :

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

int n = sIPCObject_getSemNbr(myIPCObjet);

[Franck.H]

Il faudrait un code complet reduit au minimum et compilable pour que je puisse voir car là je ne vois pas du tout de quoi ca viens, tout me semble correct sauf si je suis passé à côté de quelque chose

[Emmanuel Delahaye]

Merci. J'ai toujours un core dump avec la fonction suivante :

Code :

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

1
2
3
4
5
6
7
int sIPCObject_getSemNbr(sIPCObject *this) {
	int iSemNbr = 0;
	if(this)
		iSemNbr = this->iSemNbr;
 
	return iSemNbr;
}

Appelée dans le main() avec :

Code :

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

int n = sIPCObject_getSemNbr(myIPCObjet);

Ca ne devrait pas si la valeur de myIPCObjet est valide. Comment a-t-elle été initialisée ?

Poste le code complet et compilable réduit au minimum qui produit ce problème. Il y a autre chose.