More explicit and defensive code so if the init fails we know we handling this pointer safely, because  rc_config_init will free the pointer if there was a problem, but it will not set the pointer to NULL.

Another location where mod_rtmp would leak memory.

Fix improper handling of a double linked list that could have caused a memory leak.

Appears to have been a copy paste error because this doesn't do what the api claims, and it also leaks the reference to the new node.

Fixing a memory leak. name needs to be freed since it's being strdup'd lower.

bump

bump

FS-5470 --resolve

FS-5466 --resolve

FS-5453 --resolve

FS-5042 changed event body creation

FS-4871: reduce logging a bit

FS-5453 --resolve this is a different issue, we just have to hangup the loopback channel so the loopback and the sip don't both execute the extension

swig

FS-5042 --resolve

FS-5453 --resolve

FS-5431: --resolve fix socket creation loop for mod_skinny to avoid segfault

More phrases

esl: Don't abort the esl_listen() loop(s) if accept() returns with errno == EINTR

ivrd doesn't use sigaction + SA_RESTART for SIGCHILD. An exiting
child process will interrupt accept() in the parent to handle
the signal, which makes accept() return -1 (errno = EINTR) after
the sighandler function returns.

Signed-off-by: Stefan Knoblich <stkn@openisdn.net>

ivrd: fflush() stdout before entering esl_listen()

Fixes repeatedly printing the 'Starting forking listener' message,
if stdout has been redirected into a logfile (for example).

Signed-off-by: Stefan Knoblich <stkn@openisdn.net>

Add new to-be-recorded phrases highlighting weekly conf call (to add to new demo IVR)

FS-5440: don't assert on bad input rtmp data

FS-5402 --resolve the problem is ent originate has many channels and many causes to parse so we need to check it against all the causes

FS-5360 change defaults

Merge branch 'master' of ssh://git.freeswitch.org:222/freeswitch

mod_http_cache: fix example config

FS-5455 try this

windows fix trivial compiler warning

FS-5429: --resolve add minimal idivert support for skinny

FS-5379: --resolve fix skinny phone codec agreement issues due to incorrect hardwired ptime, update field names for future coding to be more descriptive

FS-5225: --resolve add ability to dial by pressing line button

FS-5267: --resolve avoid null deref if sessions missing

mod_http_cache: a couple more unit test cases

mod_http_cache: remove unnecessary memset() now that switch_b64_encode() is fixed

mod_http_cache: fix unit test now that switch_b64_encode is fixed

FS-5436 --resolve

FS-5402 revert wait till tomorrow

FS-5402

FS-5453 --resolve

FS-5457

FS-5376 --resolve

FS-5242

FS-5211 --resolve

ftmod_libpri: Fix possible NULL-ptr dereference crash in on_restart(_ack)()

Catch single channel RESTART/RESTART ACK events for non-B-channels,
that would lead to a NULL pointer dereference, because those do not have
per-channel private data (chan->call_data = chan_priv == NULL).

Signed-off-by: Stefan Knoblich <stkn@openisdn.net>

mod_http_cache: forgot to add write to http_cache:// format

Add mod_say_pl to debian build

Spelling fix

FS-5360

getting read_impl too soon here

FS-5444 --resolve

mod_http_cache: added write file format

mod_http_cache: added native Amazon S3 support

Fixed a weakness in error UDPTL lost packet recovery. Hopefully this addresses
FS5202.

Add mod_xml_radius to the modules list, though disabled by default.

Theoretical memory leak where if attr_init failed, then check would have still been malloc'd but not used.

Minor code cleanup. calling va_start without calling va_end doesn't leak does it? Either way, it's fixed now.

full_contact must be freed by sofia. There isn't a chance of this having blown anything up, but Coverity is right that this should be initialized with NULL to be safe against future refactors.

Was causing a very small leak on naptr lookups. Upon review just happened to be a typo missing a bang.

My bad. some how a make didn't catch this issue after a cleanup refactor. In this instance buf is being treated as an 8 byte number, not a pointer.

Coverity reported an Uninitialized pointer read. Upon further digging it appears that there is a code path where incoming packets can come in out of order, so this section of code tries to see if it can find the missed packets. As per FS-5202 there is a case where under heavy load the packet exists, and has most of the packet parsed, but still has a NULL pointer for the packet buffer. These two lines would at least help detect the edge case.

Edge case for when you are subscribing to presence on a valet parking lot. In which case there would be a small leak.

Possible leak if someone were to try and add extra headers to a channel that had either an inbound info on that session or the session were to be deflected. I don't see a way that these extra headers could be added outside of a C module subscribing to the session states, but just to make sure I'm adding these so that just like other extra header locations it's cleaned up.

Alice meet rabit hole. There is a leak here on every invite that comes back from a status 180 when the other side is not FS, but you have presence enabled. Thanks to Coverity scan for location this condition. This 'might' not have been leaking if sofia was automatically freeing this when the sofia object was destroyed, but all other locations where sip_header_as_string where called they were all followed by a su_free, so I assume not.

If the mod_sofia global presense hash is null, then make sure to free the matches variable before going to done.

FS-5443: --resolve leaked because the JS api's copy and return new malloc'd pointers.

When message events were added, the debug field name was overlooked. This would have only been hit when debug level was above 'DEBUG1' and at that point it would have incorrectly chosen debug display names.

Minor typo. User is being passed into this fuction as a long pointer, not a long.

read() does not NULL terminate statbuf, but strstr expects statbuf to be null terminated.

This was writing a null one space to the right, outside of the malloc'd buffer. On further review this uses apr_vsnprintf which always returns null terminated.

FS-5434 --resolve

remove flag

fix silence_stream size bug

media/srtp refactoring pass

FS-5441: --resolve

how...?

Since there isn't neutral ground, on the rtmp state callback we actually have to force the write lock of the session, but we only need a try lock on removing the session from the rsession->session_hash. This removes a deadlock caused when we had to enforce with locks that the rsession couldn't be destroyed if the FS session was in the middle of a hangup.

Seems the session wasn't coming to the hangup in a locked state.

Seems having a read lock still prevents a write lock to be taken, even if the thread that is trying to get the write lock currently has the read lock.

Include the write lock since we're changing the tech_pvt

Some tweaks to the image translate code

FS-5430 --resolve

trivial fix compiler warning

FS-5418 --resolve

FS-5208 --resolve

fix some minor regression in media code

FS-5011 try to fill gaps

Fixed a silly in the FAX image resolution settings

FS-5325 force rebuild

FS-5416 --resolve just missing the part that registers it

fix stun lookup returning 1 for port

FS-5407 --resolve

FS-5410 --resolve

FS-5406 --resolve

FS-5325 windows compile

Require OpenSSL

add context

FS-5325

FS-5085 the last change helped weed out the real bug which is this commit

spandsp windows fix syntax for gettimeofday

FS-5403 --resolve