Hi,

Recently I found a bug in ts_append() function of the tsilo module. It works for most users until they start using some functions in branch routes of newly created branches.
The root of the problem is that static buffer from pv_get_buffer() can be overwritten by some branch_route actions. In the end in the ts_append() lock_entry_by_ruri locks one mutex and unlock_entry_by_ruri unlocks another one.

You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/1138

Commit Summary

• Fix memory leak in ldap module that happens after calling ldap_result_next()
• Merge branch 'master' of https://github.com/kamailio/kamailio
• tsilo: fix deadlock in ts_append()

File Changes

• M src/modules/tsilo/tsilo.c (18)

Patch Links:

• https://github.com/kamailio/kamailio/pull/1138.patch
• https://github.com/kamailio/kamailio/pull/1138.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.