diff options
author | dvossel <dvossel@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-10-12 15:58:56 +0000 |
---|---|---|
committer | dvossel <dvossel@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-10-12 15:58:56 +0000 |
commit | 490eb51c618ecf6780ffcf725b78e34bdefc9544 (patch) | |
tree | 505eeef8a85d37106348b1c42ca8bd5e2ebd3339 | |
parent | ae258132dd58c4a62c165397c6226b21ed0c66a2 (diff) |
Fixes manager.c crash.
This issue was caused by improper use of the mansession lock and
manession_session lock. These two structures are confusing to begin
with so I'm not surprised this occurred. I fixed this by consistently
making sure we use each of these locks only to protect the data
in the corresponding structure. We had mismatched usage of these
locks which resulted in no mutual exclusivity occurring at all.
(closes issue #17994)
Reported by: vrban
Patches:
mansession_locking_fix.diff uploaded by dvossel (license 671)
Tested by: vrban
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.8@291227 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r-- | main/manager.c | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/main/manager.c b/main/manager.c index 231c42353..c5e313e9e 100644 --- a/main/manager.c +++ b/main/manager.c @@ -1978,11 +1978,11 @@ static int set_eventmask(struct mansession *s, const char *eventmask) { int maskint = strings_to_mask(eventmask); - mansession_lock(s); + ao2_lock(s->session); if (maskint >= 0) { s->session->send_events = maskint; } - mansession_unlock(s); + ao2_unlock(s->session); return maskint; } @@ -2771,7 +2771,7 @@ static int action_waitevent(struct mansession *s, const struct message *m) /* XXX maybe put an upper bound, or prevent the use of 0 ? */ } - mansession_lock(s); + ao2_lock(s->session); if (s->session->waiting_thread != AST_PTHREADT_NULL) { pthread_kill(s->session->waiting_thread, SIGURG); } @@ -2795,14 +2795,14 @@ static int action_waitevent(struct mansession *s, const struct message *m) s->session->send_events = -1; } } - mansession_unlock(s); + ao2_unlock(s->session); /* XXX should this go inside the lock ? */ s->session->waiting_thread = pthread_self(); /* let new events wake up this thread */ ast_debug(1, "Starting waiting for an event!\n"); for (x = 0; x < timeout || timeout < 0; x++) { - mansession_lock(s); + ao2_lock(s->session); if (AST_RWLIST_NEXT(s->session->last_ev, eq_next)) { needexit = 1; } @@ -2816,7 +2816,7 @@ static int action_waitevent(struct mansession *s, const struct message *m) if (s->session->needdestroy) { needexit = 1; } - mansession_unlock(s); + ao2_unlock(s->session); if (needexit) { break; } @@ -2830,7 +2830,7 @@ static int action_waitevent(struct mansession *s, const struct message *m) } ast_debug(1, "Finished waiting for an event!\n"); - mansession_lock(s); + ao2_lock(s->session); if (s->session->waiting_thread == pthread_self()) { struct eventqent *eqe = s->session->last_ev; astman_send_response(s, m, "Success", "Waiting for Event completed."); @@ -2851,7 +2851,8 @@ static int action_waitevent(struct mansession *s, const struct message *m) } else { ast_debug(1, "Abandoning event request!\n"); } - mansession_unlock(s); + ao2_unlock(s->session); + return 0; } @@ -4544,21 +4545,20 @@ static int get_input(struct mansession *s, char *output) } res = 0; while (res == 0) { - /* XXX do we really need this locking ? */ - mansession_lock(s); + ao2_lock(s->session); if (s->session->pending_event) { s->session->pending_event = 0; - mansession_unlock(s); + ao2_unlock(s->session); return 0; } s->session->waiting_thread = pthread_self(); - mansession_unlock(s); + ao2_unlock(s->session); res = ast_wait_for_input(s->session->fd, -1); /* return 0 on timeout ? */ - mansession_lock(s); + ao2_lock(s->session); s->session->waiting_thread = AST_PTHREADT_NULL; - mansession_unlock(s); + ao2_unlock(s->session); } if (res < 0) { /* If we get a signal from some other thread (typically because @@ -4571,7 +4571,7 @@ static int get_input(struct mansession *s, char *output) return -1; } - mansession_lock(s); + ao2_lock(s->session); res = fread(src + s->session->inlen, 1, maxlen - s->session->inlen, s->session->f); if (res < 1) { res = -1; /* error return */ @@ -4580,7 +4580,7 @@ static int get_input(struct mansession *s, char *output) src[s->session->inlen] = '\0'; res = 0; } - mansession_unlock(s); + ao2_unlock(s->session); return res; } |