Module: sip-router Branch: pd/pua_fix DELETED Commit: b93149c756d3e983c70608938f1142ed43ee1834 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b93149c7...

Author: pd peter.dunkley@crocodile-rcs.com Committer: pd peter.dunkley@crocodile-rcs.com Date: Thu Aug 11 17:28:19 2011 +0100

modules/tm, modules_k/pua: Fix for concurrency issue in PUA module

- It is quite possible for the mandatory NOTIFY request sent by a presence server on establishment of a SUBSCRIBE dialog to over-take (either on the wire or within Kamailio) the 2xx response to the SUBSCRIBE. When this happens Kamailio outputs an error message and does not update the rls_presentity table.

- The change to the tm module is to make t_request_outside take the same arguments (and exhibit mostly the same behaviour) as t_request. This is safe to do as a search of the code-base has shown that t_request_outside was not actually used anywhere. The difference between t_request and t_request_outside is that t_request frees the dialog structure it creates, whereas t_request_outside leaves the dialog structure so that the caller can use it to find things like the Call-ID and local tag generated for the request.

The hash table implementation in pua has been modified to enable temporary dialogs to be found (new function get_temporary_dialog()). A temporary dialog contains the minimal information that was available when the SUBSCRIBE request was sent. Temporary dialogs are replaced with proper ones when a 2xx response is received. The delete_htable() function has been updated so that it can delete both full and temporary dialogs.

pua.c has been modified to fix a bug in db_update() - n_query_cols was being incorrectly decremented in a certain case within a double loop. db_update() has also been changed to cope with needing to insert temporary (and therefore not fully filled in) dialogs into the database.

send_subscribe.c has been modified to create temporary dialogs whenever an initial SUBSCRIBE is sent. The SUBSCRIBE callback function searches for and removes any temporary dialogs relating to the transaction before it returns. In normal (non error handling behaviour) temporary dialogs are not removed until after a full dialog has been created and added to the hash table.

---

modules/tm/uac.c | 11 +++++- modules/tm/uac.h | 4 +- modules_k/pua/hash.c | 75 +++++++++++++++++++++++++++++++--------- modules_k/pua/hash.h | 1 + modules_k/pua/pua.c | 62 +++++++++++++++++++++++++-------- modules_k/pua/send_subscribe.c | 75 ++++++++++++++++++++++++++++++++++++++- 6 files changed, 191 insertions(+), 37 deletions(-)

Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=b931...