FS-6713 #resolve

FS-6440 regression from this patch on single CPU or VM machines causes startup race

fix jb endless loop of missing packets

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

FS-6621

remove useless variable

git push
Merge branch 'v1.2.stable' of ssh://git.freeswitch.org/freeswitch into v1.2.stable

FS-6612 backport to 1.2 for endusers still on 1.2 that require this

sync up jb changes

Use portable version of strerror_r()

FS-6596 --resolve

FS-6440 --resolve

FS-6533: --resolve fix gcc 4.9 warning due to useless right-hand operand of comma expression

FS-6538: silence gcc 4.9 throwing incorrect warning

Override config on cowbuilder update

This allows new settings, e.g. new mirrors, to propagate into the
chroot images.

Retry for longer on failed cowbuilder operations

There's really nothing to do but retry.

Retry the cowbuilder create operation on failure

Sometimes while the debian repositories are updating there are
sporadic signature failures.  It's annoying to have these break the
build, and the only thing to do is to retry, so we'll retry here
automatically.  We were already retrying on the update operation that
can fail in a similar manner.

Check for execv(3) errors when reincarnating

When -reincarnate-reexec is given we run execv to restart FS.  If
argv[0] isn't a full pathname then execv is going to fail.  While not
common for a FS system started by init, this is a common occurrence
when FS is started from the shell.

Now if execv fails, we'll try execvp.  If that fails too then we'll
fall back on the normal reincarnation behavior.

Previously what would happen in that case is god would descend from
the heavens and become mortal.  Leaving heaven absent, all hope for
reincarnation was lost.

(That is, we'd simply return from reincarnate_protect and the
supervisor process would become the new instance of FS, so the trick
would only work once.)

Add fsctl command to summon reincarnation

If you start freeswitch with -reincarnate or -reincarnate-reexec, FS
will restart automatically in the event of an unexpected exit.

Currently, you can cause FS to immediately call exit(0) with `fsctl
shutdown now`, or you can have it call abort() with `fsctl crash`.
Which are both nice, but if you have reincarnation engaged, you really
might want FS to call exit([non-zero]) so the great supervisor
immediately breathes life back into your system.

This is now available via `fsctl shutdown reincarnate now`.

Improve a parameter name for fs_cli

What we momentarily called log-uuid-chars is now better called
log-uuid-length.  Setting log-uuid-length will specify a truncation
length for UUIDs displayed by setting log-uuid.

Add log-uuid-short option to fs_cli

If log-uuid-short is set, or -S is passed to fs_cli, we only display
the first 8 hex digits of the UUID.  The log-uuid-chars option may
instead be set to specify some other truncation length for the UUID.

Convert esl_true and esl_false to functions

Prior to this commit, an expression such as:

  esl_true("true") ? 42 : 0

...would return 1 rather than 42.

freeswitch-config-rayo.spec remove some dependencies to high def audio and specific fs version

fix regression from regression

FS-6446 regression fixed

make device state code more accurate

Conflicts:
src/switch_core_state_machine.c

FS-6446 --resolve

Conflicts:
src/mod/endpoints/mod_sofia/sofia.c

rayo config: increase mod_shout outscale to mpg123 default of 32768

mod_rayo: remove code that is no longer needed

mod_rayo: http_put of received fax was broken

Avoid useless NAT log message

If FS is not behind NAT, then every call generates at least three
INFO-level log messages:

  [INFO] switch_nat.c:589 NAT port mapping disabled

This is useless noise.  The message is only interesting if you do have
NAT enabled but mapping disabled, which might indicate a configuration
issue.

With this change, we just skip the entire nat_add_mapping function if
the NAT system isn't initialized or we're not behind NAT.

Add support for number-alias in directory app

FS-5913 --resolve

FS-6418 FS-6427 add profile param NDLB-proxy-never-patch-reinvites to solve both issues

FS-6530 Chinese say now say "one" before "ten" for numbers >99

CID 1214210: mod_ssml - resource leak when parsing SSML

CID 1214263: mod_ssml - remove unused pointer value

Avoid polluting CFLAGS for mod_snmp

Putting `net-snmp-config --cflags` into CFLAGS causes major pollution;
it overrides optimization and debugging levels, warnings, and more.
While normally we do want to automatically locate library headers,
there has to be a better way to do this.  libsnmp is normally in the
usual place and doesn't need special handling.  Perhaps people with
libsnmp in a weird place should just need to add the -I flag to their
CFLAGS before build.

Cleanup makefile library dependencies of mod_snmp

Calling out to net-snmp-config --agent-libs causes transitive
dependencies to get pulled in, but we don't need those -- a sensible
dynamic linker pulls those in automatically.  Trying to track the
transitive dependencies manually would be a losing battle.

People were recently hitting this on Debian sid/jessie, where libpci
is in the transitive dependency list but isn't otherwise one of our
build dependencies.

version bump

FS-6515 --resolve

CID: 1211940 mod_rayo - unlikely dereference of NULL pointer

CID: 1211941 mod_rayo - unlikely dereference of NULL pointer

CID: 1211943 mod_rayo - removed dead code when changing output speed

CID: 1211944 mod_rayo - remove set of unused pointer

CID: 1211946 mod_rayo - dead code after switch statement

CID: 1211947 mod_rayo - allocated array with unsigned char * element size instead of unsigned char

CID: 1211951 mod_rayo - fix unlikely dereference of null pointer

mod_rayo: fix compiler error to nlsml.c from attempted coverity fix

CID: 1211948 fix memory leak of decoded iks message in function parse_plain_auth_message in mod_rayo

CID: 1211950 fix memory leak of iksparser in function nlsml_parse in mod_rayo

FS-6506 --resolve Technically when you are not using groups you do not add a <users> tag inside <domain> you just put several <user> tags in <domain> This patch will consume an extraneous <users> tag and move on making it work either way.

add quickie targets for fr sounds

sofia is treating all notify sending as if they were natted rather than when nat is detected.  This causes issues on some polycoms that use strange ports in their contact for getting notifies that do not match the port the calls come from

FS-6503 --resolve remove unused code

FS-6500: try to do the bounds checking properly using element of the enum

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

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

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.

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

FS-6485: --resolve fix -lz include order issue breaking ubuntu 14.04 build

mod_soundtouch: commit 34d5b6e8657e28298470c09a614fc586b8bac761 broke CentOS build

FS-6477 --resolve

Conflicts:
src/mod/applications/mod_conference/mod_conference.c

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

mod_rayo: relax recent caller ID restrictions a bit

Refactor out extraneous call to gettime

We were calling switch_epoch_time_now() twice for no good reason.

Edit out gcc-ism from soundtouch lib

This allows building mod_soundtouch in clang as clang doesn't support
gcc's -fcheck-new.

Remove ancient FS-2746 patch from tree

Remove completely obsolete patches from tree

These haven't been touched in years; they can't possibly still be
applicable.

Remove stray executable bits

Add uuid_ring_ready API command

We already had uuid_answer and uuid_pre_answer, so we might as well
add uuid_ring_ready.

Fix typo in comment

Avoid leaking memory on mod_cdr_sqlite unload

Avoid leaking memory on mod_voicemail unload

Avoid leaking memory on mod_hash unload

Avoid leaking memory on unload of mod_dptools

Automated leak detectors find leaked memory on shutdown.  Cleaning up
after ourselves on shutdown eliminates noise from these reports.

Hide warning about sometimes initialized variable

The rc variable here was always initialized but the compiler couldn't
see that because of the lack of an unconditional else clause.

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.

Move loop exit conditionals

The net effect here is the code looks more "regular" and reads more
linearly.

Avoid considering more phrase actions after break

Previously we would continue considering phrase actions even after
receiving a break action; we would only break on the next input
clause.  It appears the intent here was to break before the next
action.

Improve comment

Consolidate two conditionals

If we get SWITCH_STATUS_BREAK then we didn't get
SWITCH_STATUS_SUCCESS.

Remove variable assignment without effect

We're breaking out of the loop here anyway, so setting done to true is
useless.

Fix memory leak in phrase macro playback

We were leaking memory when break_on_match was set or when we received
back SWITCH_STATUS_BREAK from a callee as we were failing to free
field_expanded_alloc.

Remove tautological conditional

If pattern is null we're setting it to a non-null value, so this
branch will always be taken.

Use `git diff -w` or `git log -p -w` to see what's going on in this
commit.

Fix memory leak on memory error

In the event of a memory error, we were trying to free a null pointer
while leaking the allocation for field_expanded_alloc.

Move variables down into loop where they're used

These variables aren't used outside of this for loop, so they should
be declared within it.

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

FS-6407 --resolve mod_rayo: support SIP URI caller ID w/ display name

add parsing of user params on invite

stable branch version of spandsp changes in 4178688b4a7c77cc5ca296fd6bc0b91fea0d0f2a

FS-6460 --resolve fix leaked RTP ports that cause mod_skinny to fail after N calls

FS-6447 --resolve return subscriber-absent platform-code 20 if <dial> is attempted to user that is not registered

remove SSML from mod_rss

mod_rayo: fixed input crash introduced in 2014-04-07 commit

Suppress spurious warning in phrase macro playback

Prior to this commit, if anything at all went wrong in
switch_ivr_phrase_macro_event() we would generate a warning like this:

  [WARNING] switch_ivr_play_say.c:348 Macro [macro_name]: 'pattern_name' did not match any patterns

This is clearly misleading.  The natural thing to do on seeing that
message is to verify that the language files are there, and that the
pattern really does exist in that macro.  But none of that was usually
the problem.  The message would be generated if the language wasn't
found, or if the channel had gone away, for example.

With this commit, we verify that we actually tried looking for the
pattern before displaying the warning about the pattern not matching.

Avoid playback on dead channels in voicemail

For years we've been generating spurious messages like:

  [WARNING] switch_ivr_play_say.c:348 Macro [voicemail_ack]: 'saved' did not match any patterns

This would happen when the caller hangs up during the playback of
certain prompts in the voicemail system where we weren't checking the
return value of vm_macro_get().  Looking closely at the log, it's
clear we were calling down into switch_ivr_phrase_macro() long after
the channel was gone.

The message above is also misleading -- switch_ivr_phrase_macro()
would have been able to find that pattern just fine, but it never
actually looked because the channel was gone.  We'll clean up that
message in a follow on commit.

Avoid crash on event without content-type

If we received an event without a content-type header we were
dereferencing a null pointer leading to a seg fault.

Reported-by: Ico <ico@voip-io.org>
ESL-90 --resolve

ESL-89: fix install path and single_command require

Conflicts:
libs/esl/ruby/single_command.rb