You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

David Vossel 4672e2805b Merged revisions 225445 via svnmerge from ... https://origsvn.digium.com/svn/asterisk/trunk ........ r225445 | dvossel | 2009-10-22 14:55:51 -0500 (Thu, 22 Oct 2009) | 50 lines SIP TCP/TLS: move client connection setup/write into tcp helper thread, various related locking/memory fixes. What this patch fixes 1.Moves sip TCP/TLS connection setup into the TCP helper thread: Connection setup takes awhile and before this it was being done while holding the monitor lock. 2.Moves TCP/TLS writing to the TCP helper thread: Through the use of a packet queue and an alert pipe, the TCP helper thread can now be woken up to write data as well as read data. 3.Locking error: sip_xmit returned an XMIT_ERROR without giving up the tcptls_session lock. This lock has been completely removed from sip_xmit and placed in the new sip_tcptls_write() function. 4.Memory leak: When creating a tcptls_client the tls_cfg was alloced but never freed unless the tcptls_session failed to start. Now the session_args for a sip client are an ao2 object which frees the tls_cfg on destruction. 5.Pointer to stack variable: During sip_prepare_socket the creation of a client's ast_tcptls_session_args was done on the stack and stored as a pointer in the newly created tcptls_session. Depending on the events that followed, there was a slight possibility that pointer could have been accessed after the stack returned. Given the new changes, it is always accessed after the stack returns which is why I found it. Notable code changes 1.I broke tcptls.c's ast_tcptls_client_start() function into two functions. One for creating and allocating the new tcptls_session, and a separate one for starting and handling the new connection. This allowed me to create the tcptls_session, launch the helper thread, and then establish the connection within the helper thread. 2.Writes to a tcptls_session are now done within the helper thread. This is done by using an alert pipe to wake up the thread if new data needs to be sent. The thread's sip_threadinfo object contains the alert pipe as well as the packet queue. 3.Since the threadinfo object contains the alert pipe, it must now be accessed outside of the helper thread for every write (queuing of a packet). For easy lookup, I moved the threadinfo objects from a linked list to an ao2_container. (closes issue #13136) Reported by: pabelanger Tested by: dvossel, whys (closes issue #15894) Reported by: dvossel Tested by: dvossel Review: https://reviewboard.asterisk.org/r/380/ ........ git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.1@225490 65c4cc65-6c06-0410-ace0-fbb531ad65f3		17 years ago
..
h323	…
misdn	Merged revisions 221844 via svnmerge from	17 years ago
xpmr	…
Makefile	…
chan_agent.c	…
chan_alsa.c	…
chan_console.c	Merged revisions 222176 via svnmerge from	17 years ago
chan_dahdi.c	fix typo, sorry	17 years ago
chan_gtalk.c	Merged revisions 139281,175058,175089 via svnmerge from	17 years ago
chan_h323.c	Fix interpretation of PRIREDIRECTIONREASON set by chan_sip.	17 years ago
chan_iax2.c	Merged revisions 225307 via svnmerge from	17 years ago
chan_jingle.c	…
chan_local.c	…
chan_mgcp.c	…
chan_misdn.c	Merged revisions 222692 via svnmerge from	17 years ago
chan_nbs.c	…
chan_oss.c	…
chan_phone.c	…
chan_phone.h	…
chan_sip.c	Merged revisions 225445 via svnmerge from	17 years ago
chan_skinny.c	…
chan_unistim.c	…
chan_usbradio.c	…
chan_vpb.cc	…
console_board.c	…
console_gui.c	…
console_video.c	…
console_video.h	…
iax2-parser.c	…
iax2-parser.h	…
iax2-provision.c	…
iax2-provision.h	…
iax2.h	…
misdn_config.c	Merged revisions 222799 via svnmerge from	17 years ago
vcodecs.c	…
vgrabbers.c	…