Discussion :

[gorgonite]

voici ma classe

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
20
21
22
23
24
25
26
27
28
29
30
31
32
class Semaphore {
	public:
		static const int WAIT_FOREVER = -1;
		static int num;
 
	protected:
		sem_t id;
                char name[20];
 
	public:
		Semaphore() { 
			sprintf(name,"%d",num);
			num++;
			if (num >= SEM_VALUE_MAX) {
				printf("attention nombre maximal de semaphores atteint\n");
			}
		}
		Semaphore(sem_t id) : id(id) { strcpy(name,"\0"); }
 
	public:
		virtual bool take(int to) { return (sem_wait(&id) == 0); }
		void take() { take(WAIT_FOREVER); }
		virtual void give() { sem_post(&id); }
		virtual void flush() { 
			int tmp;
			do {
				sem_getvalue(&id,&tmp);
			} while (tmp > 0);
		}
		virtual ~Semaphore() { sem_destroy(&id); free(name); }
};
int Semaphore::num = 1;

et mon main

Code :

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

1
2
3
4
5
6
int main(int argc, char* argv[]) {
	sem_t sem;
	sem_init(&sem, 1, 0);
	Semaphore cSem;
	return 0;
}

tout passe jusqu'à la fin de sem_init

mais avec cSem je me prends un segmentation fault... et je ne comprends pas d'où il vient

ps: les timeout ne sont pas encore implémentés... je sais

[Laurent Gomila]

Code :

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

virtual ~Semaphore() { sem_destroy(&id); free(name); }

Tu détruis la sémaphore "id", mais dans le cas de cSem "id" n'a pas été initialisé.
De plus tu libères "name" avec free, alors qu'il n'a pas été alloué avec malloc. De toute façon en C++ on n'utilise ni malloc / free, ni un tableau de char pour les chaînes. Un petit tour par la FAQ pourra t'aider pour ça.

[gorgonite]

Tu détruis la sémaphore "id", mais dans le cas de cSem "id" n'a pas été initialisé.

pas fait gaffe effectivement...

en revanche, lorsque je fais

Code :

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

id = sem_open(name,O_CREAT);

dans Semaphore() avec id de type sem_t* il me fait quand même cette erreur... tu saurais pourquoi ?

De plus tu libères "name" avec free, alors qu'il n'a pas été alloué avec malloc. De toute façon en C++ on n'utilise ni malloc / free, ni un tableau de char pour les chaînes.

je sais... mais j'ai fait un copier/coller un peu rapide d'un programme C que j'avais fait et qui marchait, et en plus, j'ai changé le char* en char[]

[Laurent Gomila]

Est-ce que tu as une bonne doc ou un tutoriel sous la main ? D'après la page de man de sem_open, si tu spécifies O_CREAT il faut utiliser la version qui prend deux paramètres supplémentaires pour initialiser la sémaphore.

Code :

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

sem_t * sem_open(const char *name, int flags, mode_t mode, unsigned int value);

Pense également à vérifier les éventuelles erreurs à chaque appel d'une fonction de type sem_xxx.

[gorgonite]

Est-ce que tu as une bonne doc ou un tutoriel sous la main ? D'après la page de man de sem_open, si tu spécifies O_CREAT il faut utiliser la version qui prend deux paramètres supplémentaires pour initialiser la sémaphore.

Code :

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

sem_t * sem_open(const char *name, int flags, mode_t mode, unsigned int value);

ben en fait, je l'ai fait à partir d'un tuto sur internet... et en plus ça compile sans ces deux paramètres (qui au passage m'intéresse beaucoup, car je ne peux pas faire de sem_init après...)

si tu as un lien, je suis preneur

Pense également à vérifier les éventuelles erreurs à chaque appel d'une fonction de type sem_xxx.

ça je sais... mais vu que j'avance au pas à pas avec gdb, je n'en ai pas encore besoin (dès que ça marchera, je le ferai )

[Laurent Gomila]

si tu as un lien, je suis preneur

Je n'y connais rien, donc à part les pages de man ("man sem_xxx" sous Google si tu ne les as pas chez toi), pas de lien intéressant non.