rebuild

merge ws.c change to sofia

css tweak

tweak on last commit

add clear channel for mono participants while in a conference with 3d callers

fold livetables into verto obj and expose moderator mode

FS-6642 --resolve

FS-6646 --resolve

mod_graylog2: additional input checks on LOG_FIELDS[]

freetdm: ftmod_gsm: Added support for hardware DTMF

FS-6420

mod_graylog2: additional fields can now be specified by adding LOG_FIELDS[name=string_value,@#name=number_value] to beginning of log.  Added _microtimestamp additional field since graylog2 only has millisecond timestamp precision.

log websocket write failures

CID:1224272 Dereference after null check.  member can not actually be NULL in any of the scenarios calling this function

Merge pull request #9 in FS/freeswitch from ~PFOURNIER/freeswitch:feature/fax_enable_tep to master

* commit '0fb179acf2246b896f1a66cfb96da68e9d2675f7':
  Add new option to request TEP to be used for an outbound fax call

Merge pull request #7 in FS/freeswitch from ~PFOURNIER/freeswitch:bugfix/t38_require_timer to master

* commit '21ae587063eb3b55c690f47cf9299a539ec25415':
  Disabling Require timer for T.38 re-Invites cause problems

Merge pull request #8 in FS/freeswitch from ~PFOURNIER/freeswitch:bugfix/fax_disable_ecm to master

* commit '5d31c9b9fe2ae6dead163c60974a41fec4fac3fc':
  It was impossible to disable ECM with T.38 in gateway mode

FS-4861 improve logging when using c++ wrapper

FS-4870

FS-4939

FS-5501 fire event on profile start

FS-5516 allow get params in URL

FS-6373

FS-6626 fix winter DST issue

FS-6563

FS-6565

FS-6566

Added explanatory comment on a recent change.

Merge branch 'master' of ssh://stash.freeswitch.org:7999/fs/freeswitch

sync changes from srtp upstream

Added additional checks that HDLC frames arriving at the T.38 gateway engine
begin with the correct 2 bytes.

Add new option to request TEP to be used for an outbound fax call

It was impossible to disable ECM with T.38 in gateway mode

Disabling Require timer for T.38 re-Invites cause problems

Disabling Require timer for T.38 re-Invites tells the remote side it
doesn't need to refresh the session but FreeSwitch will still terminate
the call if the remote session doesn't refresh.

Lets get loud

Fix BLF confusion when dealing with the same extension in multiple domains

updated directory using asr/tts

Update to use PocketSphinx 0.8

Refactor the curl PUT read callback

Advance the data_stream data pointer

Add PUT support to mod_curl `curl` api and app

With both the `curl` api command and application, you can now ask for
data to be PUT where previously it could only be sent via POST.

Fixed a clang-3.5 missing-prototype warning and added doxygen
documentation for switch_channel_set_presence_data_vals. #doxygen

Fixed dead code.

While reviewing code I noticed some dead code. It was not possible to
default to the channel variable because the parameter could not be
both full and empty.

If the parameter is not a non zero length string then the code looked
like it was intending to default to the channel variable
'presence_data_cols'. If neither of these are the case it is a noop.

By enabling the dead code, you now have access to set the
'presence_data_cols' in the dialplan or scripts like lua.

update image

Merge pull request #2 in FS/freeswitch from ~KATHLEEN.KING/freeswitch-fork:doxygen-switch_apr.h to master

* commit '64fc3f7934888175b80e0cdd3a065d717d0a9014':
  Changed the function parameter name in the function definition to match the updated parameter name in the function declaration. #doxygen
  Changed the variable name for clarity.

Fixed trucation of value warning.

There was a parameter mismatch between abs(), which expects an int,
and atol() which returns a long. Since max_drift is defined as an int,
there is no need to pars q as a long rather than an int.

Changed the function parameter name in the function definition to
match the updated parameter name in the function declaration. #doxygen

Removed a useless called to abs.

Clang 3.5 reported the following error: error: taking the absolute
value of unsigned type 'unsigned int' has no effect
[-Werror,-Wabsolute-value]

Subtracting unsigned variables will never be negative and will either
be the small expected value or will wrap to a very big value. This
code is trying to determine if the difference between these timestamps
is greater than 16000.

The variables last_write_ts and this_ts deal with timestamps. In the
normal case this_ts will be a larger timestamp than
last_write_ts. This change will maintain the intended behavior of
reseting the video if the difference is larger than
16000 and in the abnormal case this value would wrap and still exceed
the 16000.

Removed an autological-pointer-compare from src/switch_utils.c.

Building with Clang 3.5 gave the following warning: error: comparison
of array 'iface_out.sin6_addr.__in6_u.__u6_addr8' equal to a null
pointer is always false [-Werror,-Wtautological-pointer-compare]

This is a problem because as it is written the check will never be
true. A pointer to a structure within a structure will never be null. The
intention was either to null check the pointer or to check if the IP
address itself was not zero.

From context in the code this appeared to be a pointer null check so I
removed it.

preserve timer name for recovery

Fixed trucation of value warning.

There was a parameter mismatch between abs(), which expects an int,
and atol() which returns a long. Since max_drift is defined as an int,
there is no need to pars q as a long rather than an int.

update ico

clean up demo a little

clean up demo a little

Merge pull request #4 in FS/freeswitch from ~KATHLEEN.KING/freeswitch-fork:doxygen-switch_regex.h-2 to master

* commit '35755d37d5dd47c2f8470ca9a294b55f124ee1e7':
  Fixed parameter name of variable in switch_regex_match_partial. #doxygen

test commit hooks, whitespace change only

Merge pull request #1 in FS/freeswitch from ~ARESKI/freeswitch:code-style to master

* commit 'b0e86e65cf16f106a4254e10ab8f8a984b3699db':
  misc of python pep8 fixes

reswig

misc of python pep8 fixes

Changed the variable name for clarity.

Fixed parameter name of variable in
switch_regex_match_partial. #doxygen

increase memory for buffers to hold new decoded stereo codecs

fix jb endless loop of missing packets

Update SubmittingPatches doc for JIRA smart commits

only count unmuted channels in positioning

vs2010 reswig

expand table width a bit

expand positioning api usage

FS-6637 try this instead

FS-6637 nevermind, this is already in the code

FS-6637 try this patch

fs_cli: fix compiler error on CentOS 6 caused by recent short uuid logging change

Add support for 16-byte auth tag for AES GCM mode.

Set the algorithm member on cipher_t when allocating AES crypto instance.  Apply same fix to NULL cipher.

Avoid buffer-overflow on short RTCP/SRTCP packets

In `srtp_unprotect_rtcp()` we are not validating that the packet
length is as long as the minimum required.  This would cause
`enc_octet_len` to underflow, which would cause us to try to decrypt
data past the end of the packet in memory -- a buffer over-read and
buffer overflow.

In `srtp_protect_rtcp()`, we were similarly not validating the packet
length.  Here we were also polluting the address of the SRTCP
encrypted flag and index (the `trailer`), causing us to write one word
to a bogus memory address before getting to the encryption where we
would also overflow.

In this commit we add checks to appropriately validate the RTCP/SRTCP
packet lengths.

`srtp_unprotect_rtcp_aead()` (but not protect) did correctly validate
the packet length; this check would now be redundant as the check in
`srtcp_unprotect_rtcp()` will also run first, so it has been removed.

Avoid buffer over-read on null cipher AEAD

In the defined AEAD modes, SRTP packets must always be encrypted and
authenticated, but SRTCP packets may be only authenticated.  It's
possible, therefore, for us to end up in `srtp_protect_aead()` without
the `sec_serv_conf` bit being set.  We should just ignore this and
encrypt the RTP packet anyway.

What we are doing instead is encrypting the packet anyway, but setting
`enc_start` to NULL first.  This causes `aad_len` to underflow which
will cause us to over-read in `cipher_set_aad()`.

If we could get past that, we would try to read and write memory
starting at 0x0 down in `cipher_encrypt()`.

This commit causes us to not check the `sec_serv_conf` bit and never
set `enc_start` to NULL in `srtp_protect_aead()`.

`srtp_unprotect_aead()` does not contain a similar error.

Prevent buffer overflow from untrusted RTP/SRTP lengths

When computing the start address of the RTP data to encrypt or SRTP
data to decrypt (`enc_start`), we are using `hdr->cc` (the CSRC
count), which is untrusted data from the packet, and the length field
of an RTP header extension, which is also untrusted and unchecked data
from the packet.

This value then pollutes our calculation of how much data we'll be
encrypting or decrypting (`enc_octet_len`), possibly causing us to
underflow.

We'll then call `cipher_encrypt()` or `cipher_decrypt()` with these
two values, causing us to read from and write to arbitrary addresses
in memory.

(In the AEAD functions, we'd also pollute `aad_len`, which would cause
us to read undefined memory in `cipher_set_aad`.)

This commit adds checks to verify that the `enc_start` we calculate is
sane based on the actual packet length.

Fix misspelling in comment

FS-6540 This should proxy all refers now, the param has changed to proxy-refer

fix volume adjustments in stereo conference

some refactoring to make new stereo stuff more stable

tweaks

check for member channel

Check for too many SRTP errors before warning

We're checking whether we've hit the warning threshold before checking
whether we should just end the call.  This causes an off-by-one error
where we take one SRTP error more than intended.

This commit reverses the order of the tests.

add openal code to mod_conference

check for proper openal symbol to enable functionality

add 44.1khz codec defs

adjust ringing var

FAX tweaks

Fixed updating of the modem type after a FAX ECM CTC.

Handle SRTP_READ_ERROR in mod_rayo

Allow more SRTP errors before killing call

In a carrier interop we saw the call get killed for SRTP failures
during a reinvite.  We're wondering if the SRTP errors may have been
transitory and if it may have recovered after a few more packets.

It's debatable whether we should kill calls at all for SRTP auth
failures; semantically the right thing to do when a MAC fails is to
ignore the packet completely.  So raising this limit to 100 packets
shouldn't do any harm.  With this change we still warn at 10 errors
and every 10 errors thereafter.

Relay cause of hangup on SRTP failure

We hangup the channel after receiving 10 SRTP packets in a row with a
bad auth tag or that are replayed.  Prior to this commit we were
indicating a normal clearing.  When doing interop and looking first at
packet traces, this made freeswitch's behavior look surprising.  With
this commit we'll indicate more loudly what's happening.

Fix misspelled function

switch_rtp_set_invalid_handler has been misspelled as
switch_rtp_set_invald_handler going all the way back to the
beginning.  So while it's possible that someone somewhere could be
relying on this misspelling, I think it's more likely that no one has
used it much and that's why it wasn't spotted.  We don't even use it
ourselves anywhere anymore.

Introduced in commit: 828e03715f67f358671c6691c3e9c0f2a66ec6ec

Add editor footer to core files missing it

Allow reincarnation from mod_sofia's shutdown-on-fail

mod_sofia's parameter shutdown-on-fail now accepts the value
"reincarnate-now".  This will cause the switch to exit immediately
with a non-zero exit code so that the supervisor can recover the
switch.  For this to work you have to pass in -reincarnate or
-reincarnate-reexec to freeswitch.

Ensure mod_sofia params can be unset or reset

This is the result of auditing each mod_sofia profile parameter to
ensure that it can be unset or reset after being set.  One use-case
for this being done correctly is so a later parameter in a
configuration file can reliably override an earlier one, which is
useful for setups with layered include files.

Avoid leaking memory on sofia profile reload

We were leaking the string values of apply-candidate-acl parameters.

Terminate format_cdr json files with newline

In UNIX, text files by definition end with a newline.

Allow setting format of log filename in format_cdr

This commit allows you to set a `log-file` string parameter in a
format_cdr profile.  This string is a template that may (and should!)
contain variables.  This template will be expanded and used as the
file name of the CDR to be written.  This parameter should contain
only the template for the file name itself; the path is relative to
the `log-dir`.