How did the use after free not blowup in the last 3 years since it was modified?
This commit is contained in:
parent
aa222c7aea
commit
b91d6e430a
|
@ -1637,7 +1637,6 @@ static void *SWITCH_THREAD_FUNC o_thread_run(switch_thread_t *thread, void *obj)
|
||||||
fifo_execute_sql_queued(&sql, SWITCH_TRUE, SWITCH_TRUE);
|
fifo_execute_sql_queued(&sql, SWITCH_TRUE, SWITCH_TRUE);
|
||||||
|
|
||||||
status = switch_ivr_originate(NULL, &session, &cause, originate_string, h->timeout, NULL, NULL, NULL, NULL, ovars, SOF_NONE, NULL);
|
status = switch_ivr_originate(NULL, &session, &cause, originate_string, h->timeout, NULL, NULL, NULL, NULL, ovars, SOF_NONE, NULL);
|
||||||
free(originate_string);
|
|
||||||
|
|
||||||
if (status != SWITCH_STATUS_SUCCESS) {
|
if (status != SWITCH_STATUS_SUCCESS) {
|
||||||
|
|
||||||
|
@ -1694,6 +1693,10 @@ static void *SWITCH_THREAD_FUNC o_thread_run(switch_thread_t *thread, void *obj)
|
||||||
|
|
||||||
end:
|
end:
|
||||||
|
|
||||||
|
if ( originate_string ){
|
||||||
|
switch_safe_free(originate_string);
|
||||||
|
}
|
||||||
|
|
||||||
switch_event_destroy(&ovars);
|
switch_event_destroy(&ovars);
|
||||||
if (node) {
|
if (node) {
|
||||||
switch_mutex_lock(node->update_mutex);
|
switch_mutex_lock(node->update_mutex);
|
||||||
|
|
Loading…
Reference in New Issue