Drop dead assignments

We goto done at the end of these blocks.

CID: 1210646

Drop dead assignment

Drop dead assignments

unused

Remove dead assignment, but why...

This assignment isn't doing anything, so it safe to remove it, but
there remains the question of whether there could have been some
actually necessary intention behind this line.

Drop dead assignment

Avoid dead assignment of return value of execv

execv(3) only returns on error, and will always return -1 in that
case.

Fix a use after free in a 'theoretical' case in switch_cache_db_persistant_execute_trans_full where the pre_trans_execute statement fails, but SQLSetAutoCommitAttr succeeds

check return value for es_say_general_count

make sure old_pid_buffer is null terminated

fix leaked body in error case

don't leak file handle if fd = 0

don't leak if first realloc fails but second one succeeds

split this into 2 functions so its clear if this function allocates or not

More fixes from Coverity scans

make sure receive buffer is always null terminated

don't leak xml string in dl_logger

add better bounds checking in dtmf_accepting array to avoid buffer overrun in mod_voicemail_ivr

fix read beyond buffer length error

A couple more Coverity issues fixed

Bisect with make -j

Don't unset CCACHE_DIR when bisecting

FS-6497: --resolve fix for loop initial declaration not allowed

Another couple of Coverity related cleanups

remove generated file from tree

More elimination of warnings with clang

Eliminate some clang warnings

mod_http_cache: CID 1025858:  Thread deadlock  (ORDER_REVERSAL)

gracefully handle allocation failure

we always have the session in the dialplan function

we always have the caller profile at this point, if for some reason we don't just instead of crashing

we always have the seesion in the app function

remove useless checks of array == 0

do set compiler flag that compiler doesn't support

Eliminated another Coverity issue

Some more coverity issues cleaned up

Cleared some simple Coverity issues

FS-6476 regression where sock would sometimes drop while reading logical frames

Correction to commit 0ba59aa6c2d252886eb4889c5d37fe114e07b4f4 using the correct variable in tel uri param parsing for mod_sofia

silence incorrect warnings about null pointer derference and remove unused code

add heartbeat_use_scheduler variable to force the heartbeat to use the scheduler rather than the media stream and also make it auto-chage between modes when going in and out of the media path

Add helper for running git bisect

this should never happen, but just in case, this will at least calm coverity

handle null 'status' passed to 'switch_channel_perform_presence' more gracefully

only enable passthru_ptime_mismatch when set to a true value, not when it is set to any value

only enable passthru_ptime_mismatch when set to a true value, not when it is set to any value

remove unused var ocp

if we can't open /dev/null, you have much bigger issues to deal with

improve null checks

remove useless variable

remove useless statement

improve error checking for null path

fix bitwise flag unsetting

remove unused variable

don't set variable we don't use, and fix off by 1 buffer overflow in 1 case

remove unnecessary null check, we would have crashed already if that strdup failed

don't use memory after free when using naptr with mod_enum with multiple routes

block of code in here twice was writing over expanded, causing it to potentially leak

add bounds check to keep rtcp packets with > 5 report blocks from creating a buffer overrun

FS-4779 FS-6467 the behaviour from 4479 is now masked behind the param 3pcc-reinvite-bridged-on-ack FS-6467 please test master

fix deref before null check

remove unused variable

FS-6491 --resolve

This seems to be a paradox when running a perl script from a session then executing perl again on the same session from a different thread.
I fixed it by converting any execution of perl in the execute_on_* family of operators to only run background mode which is to store the command in the session stack to be executed only by the session thread instead of on the spot by the outide thread.  changing the execute_on_answer to perl::/path/to/script.pl would also eliminate the crash in code that has not been updated with this patch.

This is just a limitation of embedded perl we have to live with.

fixing commit a7108411233058511c59227617e446f28384dbd0 in switch_config

Previous commit actually broke the functionality since it was now returning if cfg->path[0] was not null.
Also since cfg->path can never be null, this check can be simplified to only check the first index.

check return value of switch_vasprintf

fix assigned but never used issue and code cleanup for readability

we only call this with values, and it would crash if you passed it null anyways, just assert to make it 100% clear

contact->m_url is an array, don't check array for zero as its useless

Fix buffer overrun that could have been exploited with a crafted skinny packet designed to exploit this mistake in bounds checking.

fix memory leaks, tanks to Sharath from packetizer forum.

FS-3332 --resolve

Expanding on 052638186b2784c4142089008adf4289d40c80bc to support boolean tel url params in mod_sofia

BNPH-6470 follow up to commit 68d863a179c90d2524f8e761dedb3aebd09c3e75 removing the original line that performed the curl request to avoid a double request in mod_xml_curl

FS-6470

FS-6482 --resolve

fix issues with heartbeat in bypass media mode

Fixing memory leak that would occur when reloading mod_xml_curl

revert

FS-6477 --resolve

FS-6476 --resolve

mod_rayo: preserve old caller ID behavior if display name is not given

mod_rayo: relax recent caller ID restrictions a bit

ESL-91: --resolve bigger buffer for args

Remove extraneous abort(3) when sending chat

This appears to have been accidentally added in commit
79ebcb104b92fa37ed291e96ab2d611f297e6078 which sought to provide a
mechanism for disabling Sofia's chat interface.  The abort(3) here
achieved that a bit too well.

add enable-chat sofia profile param which defaults to true when not present, set to false to turn off the chat interface in sofia

Drop pinned memcached file download from debian

Correct display of last write timestamp

On start DTMF packets we were showing the last write timestamp as a
signed value when it's an unsigned value, which could result in it
appearing incongruous with later packets where the value was displayed
correctly.

OPENZAP-222 --resolve

FS-6082 --resolve

Dispatch will call you on register, and call you back if you hangup on next register

fix bug in dispatch

FS-6452 mod_spy doesn't react on events without variable_domain_name

don't send heartbeat until the first timeout has elapsed rather than instantly

fix compiler warning

update

FS-6462 --resolve

I found a problem here but it may not completely match your expectations.
I reviewed the RFC 4028 and checked against the code and I discovered we should not be putting a Min-SE in any response at all besides a 422:

section 5:

   The Min-SE header field MUST NOT be used in responses except for
   those with a 422 response code.  It indicates the minimum value of
   the session interval that the server is willing to accept.

I corrected this problem and implemented the 422 response so if you request a value lower than the minimum specified for the profile.
If the value is equal or higher to the minimum, it will be reflected in the Session-Expires header in the response and no Min-SE will be present.

FS-5997 regression from commit 70accd9f272472ac2081283f1927d901b409acb6 this caused some attended transfers to calls with multiple targets to get the abondoned channels to be stuck on write lock

Get rid of a small memory leak when the configs are reloaded in mod_enum

add generic keepalive system and implement it in sofia to send MESSAGE or INFO packets in-dialog at specified interval.

Adds app: enable_keepalive 0|<seconds>
This app can be run in the dialplan or with execute_on_* type variables for B-legs.

Adds sofia param: keepalive-method  : defaults to MESSAGE can also be "INFO"
This param sets which SIP method to use.