diff options
author | Richard Mudgett <rmudgett@digium.com> | 2018-03-22 13:35:04 -0500 |
---|---|---|
committer | Richard Mudgett <rmudgett@digium.com> | 2018-04-09 17:12:53 -0500 |
commit | 7157dcf83bcaa4dba32c96d949456f049b1abbd6 (patch) | |
tree | 4f2e4098136a1789bbb08424514455c707371e98 /include | |
parent | 2a6072a9c414a8fb7ecb1b6598873e60ef8cadf2 (diff) |
pjsip_scheduler.c: Fix ao2 usage errors.
* Removed several invalid uses of OBJ_NOLOCK. These uses resulted in the
'tasks' container being accessed without a lock in a multi-threaded
environment. A recipe for crashes.
* Removed needlessly obtaining schtd object references. If the caller
providing you a pointer to an object doesn't have a valid reference then
you cannot safely get one from it.
* Getting a ref to 'tasks' when you aren't copying the pointer into
another location is useless. The 'tasks' container pointer is global.
* Removed many unnecessary uses of RAII_VAR.
* Make ast_sip_schedule_task() name parameter const.
ASTERISK_26806
Change-Id: I5c62488e651314e2a1dbc01f5b078a15512d73db
Diffstat (limited to 'include')
-rw-r--r-- | include/asterisk/res_pjsip.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/asterisk/res_pjsip.h b/include/asterisk/res_pjsip.h index 26439986b..d3849ad34 100644 --- a/include/asterisk/res_pjsip.h +++ b/include/asterisk/res_pjsip.h @@ -1673,7 +1673,7 @@ struct ast_sip_sched_task; * */ struct ast_sip_sched_task *ast_sip_schedule_task(struct ast_taskprocessor *serializer, - int interval, ast_sip_task sip_task, char *name, void *task_data, + int interval, ast_sip_task sip_task, const char *name, void *task_data, enum ast_sip_scheduler_task_flags flags); /*! |