aboutsummaryrefslogtreecommitdiff
path: root/philo_three/src
diff options
context:
space:
mode:
authorCharles Cabergs <me@cacharle.xyz>2021-01-14 10:57:02 +0100
committerCharles Cabergs <me@cacharle.xyz>2021-01-14 10:57:02 +0100
commit0ed344821bc3c872100a18289d0384d51af7e970 (patch)
tree8b246d0494eff661bc044aa9b8cc38bf37084efc /philo_three/src
parent051e5fa4382a629b76f32e46d3e766ed2eb306fc (diff)
downloadphilosophers-0ed344821bc3c872100a18289d0384d51af7e970.tar.gz
philosophers-0ed344821bc3c872100a18289d0384d51af7e970.tar.bz2
philosophers-0ed344821bc3c872100a18289d0384d51af7e970.zip
Fixing infinite wait on guacamole when calling sem_closeHEADmaster
Diffstat (limited to 'philo_three/src')
-rw-r--r--philo_three/src/helper.c9
-rw-r--r--philo_three/src/main.c22
2 files changed, 12 insertions, 19 deletions
diff --git a/philo_three/src/helper.c b/philo_three/src/helper.c
index 3b718bc..a6e4c6e 100644
--- a/philo_three/src/helper.c
+++ b/philo_three/src/helper.c
@@ -6,19 +6,12 @@
/* By: cacharle <me@cacharle.xyz> +#+ +:+ +#+ */
/* +#+#+#+#+#+ +#+ */
/* Created: 2021/01/10 13:58:26 by cacharle #+# #+# */
-/* Updated: 2021/01/10 14:24:26 by cacharle ### ########.fr */
+/* Updated: 2021/01/14 10:54:26 by cacharle ### ########.fr */
/* */
/* ************************************************************************** */
#include "philo_three.h"
-void h_destroy_sem(const char *name, sem_t *sem)
-{
- if (sem != SEM_FAILED)
- sem_close(sem);
- sem_unlink(name);
-}
-
bool h_sem_create(const char *name, unsigned int value, sem_t **sem)
{
if (value == 0)
diff --git a/philo_three/src/main.c b/philo_three/src/main.c
index a3128a0..158ca25 100644
--- a/philo_three/src/main.c
+++ b/philo_three/src/main.c
@@ -6,13 +6,13 @@
/* By: cacharle <marvin@42.fr> +#+ +:+ +#+ */
/* +#+#+#+#+#+ +#+ */
/* Created: 2020/02/15 00:45:24 by cacharle #+# #+# */
-/* Updated: 2021/01/10 14:25:49 by cacharle ### ########.fr */
+/* Updated: 2021/01/14 10:54:56 by cacharle ### ########.fr */
/* */
/* ************************************************************************** */
#include "philo_three.h"
-static int st_destroy(t_philo_conf *conf, pid_t *pids, long int philo_num)
+static int st_destroy(pid_t *pids, long int philo_num)
{
long int i;
@@ -23,12 +23,12 @@ static int st_destroy(t_philo_conf *conf, pid_t *pids, long int philo_num)
kill(pids[i], SIGKILL);
free(pids);
}
- h_destroy_sem(PHILO_SEM_NAME, conf->forks);
- h_destroy_sem(PHILO_SEM_STDOUT_NAME, conf->sem_stdout);
- h_destroy_sem(PHILO_SEM_FINISH_NAME, conf->sem_finish);
- h_destroy_sem(PHILO_SEM_MEAL_NUM_NAME, conf->sem_meal_num);
- h_destroy_sem(PHILO_SEM_START_NAME, conf->sem_start);
- h_destroy_sem(PHILO_SEM_GRAB_NAME, conf->sem_grab);
+ sem_unlink(PHILO_SEM_NAME);
+ sem_unlink(PHILO_SEM_STDOUT_NAME);
+ sem_unlink(PHILO_SEM_FINISH_NAME);
+ sem_unlink(PHILO_SEM_MEAL_NUM_NAME);
+ sem_unlink(PHILO_SEM_START_NAME);
+ sem_unlink(PHILO_SEM_GRAB_NAME);
return (1);
}
@@ -103,16 +103,16 @@ int main(int argc, char **argv)
if (conf.philo_num == 0 || conf.meal_num == 0)
return (0);
if (st_setup(&conf, &pids) != 0 || st_start(&conf, pids, h_time_now()) != 0)
- return (st_destroy(&conf, pids, conf.philo_num));
+ return (st_destroy(pids, conf.philo_num));
if (conf.meal_num != -1)
{
if (pthread_create(&thread_meal_num,
NULL, (t_routine)st_routine_meal_num, &conf) != 0)
- return (st_destroy(&conf, pids, conf.philo_num));
+ return (st_destroy(pids, conf.philo_num));
pthread_detach(thread_meal_num);
}
sem_wait(conf.sem_finish);
sem_wait(conf.sem_finish);
- st_destroy(&conf, pids, conf.philo_num);
+ st_destroy(pids, conf.philo_num);
return (0);
}