Custom Query (2195 matches)

#2204 added the capability to add remote certificate on OpenSSL. However the change introduce new bug which allow memory being allocated continuously without being released when TLS transport is used.

Thanks to George Joseph for the report.

Original report

Multiple requests should reuse the same connection if one already exists to the remote server. When a transport error occurs, the next request should establish a new connection and any following requests should use that same one. With r6002 patch, when a transport error occurs, every new request creates a new connection so you can wind up with thousands of open TCP sockets, possibly exhausting file handles, and increasing memory usage. Reverting commit r6002 (and r6021) restores the expected behavior.

After investigation, the pjsip_tpmgr_acquire_transport2() actually tries to lookup any matching transport to be reused, but unfortunately the search does not check if the matching transport is in shutdown state. And later, when it realizes that the transport is in shutdown state, it simply drops the transport and create a new one.

Thanks to George Joseph for the report.

Currently the message buffer is simply set to the packet buffer:

dst->msg_info.msg_buf = dst->pkt_info.packet;

while for connection oriented transports, the SIP message boundary is not always in the beginning of the packet buffer.

Thanks to Andrew A. Chernyh for the patch.