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

FreeTDM: ftmod_misdn uses clock_gettime() which is in librt, add a check

Fixes -Wl,-z,now / -Wl,-z,defs errors about undefined symbols.

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

FreeTDM: Make libpri pri_version() check -Wunused-but-set-variable safe

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

FreeTDM: Move external ftmod_* library dependencies to _LIBADD

Making the build -Wl,--as-needed safe.
Tested on Centos 5.x + Gentoo.

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

FreeTDM: Fix implicit conversion between enumeration types error

Full clang error message:

  error: implicit conversion from enumeration type 'ftdm_bool_t' to
  different enumeration type 'ftdm_status_t' [-Werror,-Wconversion]

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

FreeTDM: Silence -Wtautological-compare warnings emitted by clang

Fixes "warning: comparison of unsigned expression >= 0 is always true".

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

FreeTDM: Use ftdm_unused_arg() to silence -Wunused-argument warnings

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

Relax the image resolutions supported for FAX

The gettimeofday() function spandsp provided for Windows apparently falls over
after 25 hours. I guess not many people are inserting header lines on FAX pages.
Hopefully this update fixes the problem.

move log message to avoid logging null

FS-5399 --resolve

FS-5325 i'll put the nonblocking in but I still think this is a problem with openssl, a short read is not there every single time you connect, When the lib is in full blocking mode it should be handling the complicated actions internally.  As complicated as socket programming is, async ssl is even worse because it requires you to do your own buffering and attemps to read may fail waiting for you to write or vice-versa which should have been handled for us in the lib.  non-interruptable syscalls are getting pretty outdated and not enabling it is much less complex than making every socket app on earth be non-blocking

FS-5360 --resolve fields are exec-on-max-failures exec-on-max-timeouts pin, needs testing

FS-5390 --resolve

FS-5398 --resolve

update to be recorded phrases

test 2, please ignore

test, please ignore

FS-5395: False alarm on a reported valgrind memory leak. The io_private is alloc'd from a memory pool which valgrind sucks at tracking. No need to free this, because it'll get cleared when the memory pool on the rsession is destroyed.

FS-5395: --resolve Resolve another mod_rtmp crash where oh specific hangup scenarios there would be a free of variable that wasn't ever malloced

FS-5085 I discovered that the bowout is happening from both legs at once, we only need to check this condition on the A side of the loopback bridge

Undo previous change

Merge branch 'master' of fs-git:freeswitch

FS-5327 --resolve

Update to be recorded phrases

Merge branch 'master' of fs-git:freeswitch

FS-5288 --resolve

FS-5371 --resolve typo

FS-5389

FS-5389 --resolve

Revert "fixed stuck mod_rtmp session when the call is killed from fs side, such as using uuid_kill"

This reverts commit d8edc031faa41d4e570aad1e347a0ed6c0d6c1d8.

Conflicts:
src/mod/endpoints/mod_rtmp/mod_rtmp.c

FS-5168 --resolve

FS-4871 --resolve

FS-5325

regression parsing rtp-timeout-sec from the profile

FS-5383 --resolve

fix mod_spandsp on systems with libjbig

FreeTDM: Add ftdm_unused_arg() macro to silence "unused parameter" compiler warnings

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