diff options
author | Miklos Szeredi <mszeredi@suse.cz> | 2013-03-19 17:16:14 +0100 |
---|---|---|
committer | Miklos Szeredi <mszeredi@suse.cz> | 2013-03-19 17:16:14 +0100 |
commit | 44088bc7fbe7c9234c090756dbf10742b1a281b1 (patch) | |
tree | acaeb43a0eff6a826efd3ef5bcdf2b0b6881f3fa | |
parent | 22d0d4275a693968eafed42a1656695fcddcea01 (diff) | |
download | android_external_fuse-44088bc7fbe7c9234c090756dbf10742b1a281b1.tar.gz android_external_fuse-44088bc7fbe7c9234c090756dbf10742b1a281b1.tar.bz2 android_external_fuse-44088bc7fbe7c9234c090756dbf10742b1a281b1.zip |
libfuse: fix thread cancel race
Exiting a worker my race with cancelling that same worker. This caused a
segmenation fault.
Reported and tested by Anatol Pomozov
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | lib/fuse_loop_mt.c | 2 |
2 files changed, 8 insertions, 0 deletions
@@ -1,3 +1,9 @@ +2013-03-19 Miklos Szeredi <miklos@szeredi.hu> + + * libfuse: fix thread cancel race. Exiting a worker my race with + cancelling that same worker. This caused a segmenation + fault. Reported and tested by Anatol Pomozov + 2013-02-04 Miklos Szeredi <miklos@szeredi.hu> * libfuse: fix crash in unlock_path(). Patch by Ratna Manoj diff --git a/lib/fuse_loop_mt.c b/lib/fuse_loop_mt.c index 7ae5890..82e3001 100644 --- a/lib/fuse_loop_mt.c +++ b/lib/fuse_loop_mt.c @@ -241,9 +241,11 @@ int fuse_session_loop_mt(struct fuse_session *se) while (!fuse_session_exited(se)) sem_wait(&mt.finish); + pthread_mutex_lock(&mt.lock); for (w = mt.main.next; w != &mt.main; w = w->next) pthread_cancel(w->thread_id); mt.exit = 1; + pthread_mutex_unlock(&mt.lock); while (mt.main.next != &mt.main) fuse_join_worker(&mt, mt.main.next); |