Memory fault avec une structure

Version imprimable

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:

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:

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:

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

04/01/2008, 11h30
Franck.H
Tu peux simplifier ton malloc ainsi (le cast n'est pas utile ici) :

Code:

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:

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:

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

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

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:

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 :aie:
04/01/2008, 14h19
Neuromancien2
Merci. J'ai toujours un core dump avec la fonction suivante :
Code:

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:

int n = sIPCObject_getSemNbr(myIPCObjet);
04/01/2008, 14h23
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 :aie:
04/01/2008, 14h25
Emmanuel Delahaye
Citation:
Envoyé par Neuromancien2

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

Code:

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:

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.