Handle gcc's overzealous unused-result warning

Modern static analyzers warn when a variable is set but not used.  GCC
warns when the result of a function marked as
`__attribute__((warn_unused_result))` is not set to some variable,
even when the function is cast to `(void)`, which is the recognized
way of indicating the intent to ignore the result.  We treat all
warnings as errors.

The combination of behaviors here makes it difficult if we really
don't care about the result of the function; the obvious workarounds
are precluded.

GCC's maintainers don't consider this to be a bug:

  Why do you think this is a bug?  warn_unused_result is for cases where
  "not checking the result is either a security problem or always a
  bug".

  https://gcc.gnu.org/bugzilla/show_bug.cgi?id=25509#c1

GLIBC however marks functions like `read` and `write` with this
attribute.  While it certainly in most cases is correct to act on
their return value, in some cases we really just don't care.

So when we see that we're building with GCC, and that we're building
with all warnings enabled, we'll just pass -Wno-unused-result to
disable the warning.

ref: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=25509
ref: https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html

Adding the ability to add a radius param when a channel variable is not set in mod_xml_radius

Help the static analyzer in `handle_ice`

Clang's static analyzer thinks we could be using `hosts` here when it
is NULL.  We probably weren't, but it's easy to see how it could think
so.  We were checking whether `from_addr` matched `ice->addr` three
times, and between the second on third time we might have modified the
`ice->addr`; however we only get there if it matched the second time,
so we could only make it not match at that point and avoid the third
branch.  We can't make it match where it did not before.

We'll simplify the logic a bit here so static analyzers (and humans)
can hopefully see this more readily.

Return NULL from `sub_alloc` for zero size

When zero was passed for the size to `sub_alloc`, we were passing this
size on to `malloc` or `calloc`, which is unusual enough that static
analyzers warn about this (POSIX says that either NULL or a pointer
will be returned).

We'll instead just return NULL right away.

Fix spelling "Couldnt" -> "Couldn't"

Handle zero channels when allocating for resampler

When the number of channels was zero, we were calling malloc with a
size of zero.  While defined, this is unusual.  And since we're
initializing the speex resampler as though there were one channel in
this case, we should probably just allocate the space for it.

Refactor to avoid warning about realloc usage

Clang's static analyzer noticed the result of realloc was being
assigned to a pointer of a different type than was used to calculate
the new size.  We can make things simpler and more idiomatic here by
using the correct pointer type and letting C's pointer arithmetic
automatically handle some multiplication.

We also use the distributive property here to simplify the calculation
for memset.

Remove dead assignments in `switch.c`

Handle null argv in `switch.c`

Remove dead assignment

Avoid using undefined memory in `switch_fulldate_cmp`

The `switch_split_date` and `switch_split_time` functions only set as
many variables as they believe exist values in the input string.
Since we didn't have defaults assigned we would read undefined stack
memory if the input string didn't contain e.g. an hour.

With this commit, we use 1970 if no year is present, January if no
month is present, the first day of the month if none is given, and
zero for each of a missing hour, minute, or second.

Refactor `switch_fulldate_cmp`

We're moving the variable declarations down to the smallest possible
scope (a good idea in general) so we can address the use of undefined
memory by the function in a later commit.

Add warning when using HTTPS with mod_curl

mod_curl currently does not verify the authenticity of the peer's
certificate, and does not verify whether the common name on the
certificate matches the server.  This makes mod_curl initiated TLS
connections completely insecure.  We should fix this, but until we do,
we'll warn people that it's not doing what they may think it is.

ref: http://curl.haxx.se/libcurl/c/CURLOPT_SSL_VERIFYPEER.html
ref: http://curl.haxx.se/libcurl/c/CURLOPT_SSL_VERIFYHOST.html

Describe how to hard-wrap text in Emacs

Merge pull request #28 in FS/freeswitch from ~STANGOR/freeswitch:master to master

* commit 'c736cf2f22e4f0a5d490db995ebaed69b9963daa':
  Add sofia gateway parameter "destination-prefix"
  FS-5497 add sofia gateway parameter destination-prefix in case you need to send Invites with prefix only to this gateway

skypopen: new skype client in install/install.pl

mod_graylog2: channel variables can now be automatically added as gelf additional fields in logs w/ session UUID

mod_skinny: roll back, these are not expected to be null terminated, they are fixed length

mod_skinny: fix potential buffer size issue CID: 1214145 1214146

Decode params to `curl_sendfile`

`curl_sendfile` generates a multipart message with Content-Type:
multipart/form-data with no separate Content-Type headers in the parts
for each non-file argument.  These parts therefore default to
text/plain.  However, prior to this commit, we were putting the URL
encoded POST data into these parts, which is not correct.  We should
be putting raw text into the parts.

With this commit, we urldecode each argument key and value before
composing the multipart message.

See:

  http://www.w3.org/TR/html401/interact/forms.html#h-17.13.4.2

Merge branch 'master' of https://stash.freeswitch.org/scm/~stangor/freeswitch

Add sofia gateway parameter "destination-prefix"

FS-5497 add sofia gateway parameter destination-prefix in case you need to send Invites to your provider with prefix only to this gateway

tweak some cid display issues

testing new hook FS-6677 #Waiting #comment this is a test

testing new hook TKT-6677 #Waiting #comment this is a test

testing new hook FS-6677 #new_state #comment this is a test

FS-6696 VS2010 Express fixes for pa and v8

fix VS2010 build warning

FS-6738 #comment fix mac build introduced by 0ee48c2

nm

fix typo

force sofia rebuild

FS-6738 #comment can you test this patch and see if the tcp keepalive can detect the timeout

copy changes from verto ws.c to sofia

FS-5497 add sofia gateway parameter destination-prefix in case you need to send Invites with prefix only to this gateway

Merge pull request #21 in FS/freeswitch from ~PFOURNIER/freeswitch:refer_event to master

* commit 'd1abf274d53958566e4a23e02b6168061c098827':
  REFER_EVENT is missing from the message names array

Merge pull request #24 in FS/freeswitch from ~STANGOR/freeswitch:master to master

* commit '4d0b875d4e7b2ee1fc8f30591b5c98c61080fae5':
  fix mod_fifo presence https://jira.freeswitch.org/browse/FS-6732

FS-6739 #resolve

Cleanup whitespace

libzrtp: update zrtp_sleep for modern libc

usleep is deprecated and disabled in glibc 2.12 unless requested. Use
nanosleep instead if available.

This fixes the following compiler warning:

    ./src/zrtp_iface_scheduler.c: In function 'zrtp_sleep':
    ./src/zrtp_iface_scheduler.c:96:2: warning: implicit declaration of
    function 'usleep' [-Wimplicit-function-declaration]
      usleep(msec*1000);
      ^

libsrtp: do not include unnecessary sources

These two files should only be built when OpenSSL is not enabled. See
the configure script of the original sources and Makefile from upstream.
(RNG_EXTRA_OBJS)

Fixes a -Wimplicit-function-declaration warning.

Fix copy-paste error in '-certs' error message

Merge pull request #25 in FS/freeswitch from ~STEVEAYRE/freeswitch:master to master

* commit '05ef18948d3870c4afcd82ce32e164e4a152a8e1':
  add uptime api to mod_commands

Merging in latest from upstream (FS/freeswitch:refs/heads/master)

* commit 'b60df39b96b05b6c3cf220d5049ea5fc20dfa6e1':
  mod_http_cache: fix configuration so that carriage returns or other whitespace is OK around S3 keys
  mod_http_cache: fixed S3 URL parser to allow mybucketsubdomain.com.s3.amazonaws.com

mod_http_cache: fix configuration so that carriage returns or other whitespace is OK around S3 keys

mod_http_cache: fixed S3 URL parser to allow mybucketsubdomain.com.s3.amazonaws.com

add uptime api to mod_commands

fix mod_fifo presence
https://jira.freeswitch.org/browse/FS-6732

Merge pull request #23 in FS/freeswitch from ~DDRAGIC/freeswitch:fix_mod_blacklist to master

* commit 'fe2a6e8acae2e8104c9480361b1561ee3f120bf1':
  FS-6620 mod_blacklist fix not deleting entries from list

don't overwrite autodial ext from cookie when supplied on the input hash

add homer capture line to websocket transport

FS-6679 #resolve

Fix line endings per .gitattributes

FS-6692 #resolve

fix early media vs answer race

FS-6727 allow 1.1 and 1.2, They should probably fix that soon

REFER_EVENT is missing from the message names array

REFER_EVENT has been added to switch_core_session_message_types_t
a while ago, but was never added to the message_names[] related array.

FS-6667 #comment block sending audio when dmachine is reading

fix channel count in embedded file formats

mod demo so you can change params when not connected

FS-6713 #comment remove debug

mod_skinny: fix calloc argument ordering CID: 1214236 1214235

mod_skinny: calloc param ordering 1214234

mod_skinny: fix potential NPE CID: 1223252

mod_skinny: pointless check CID 1060963

mod_skinny: fix potential overflow CID: 1060947

mod_skinny: start using switch_copy_string so it uses apy_cpystrn - CID: 1214145 1214146 1214131 1214132 1214133 1214134 1214135
1214147 1214148 1214149 1214150 1214151 1214152 1214153 1214154 1214155 1214138 1214139 1214143 1214144

Forces null termination of the destination string.

mod_skinny: CID:1060950 remove some dead debugging code

FS-6713 #resolve

save remote caller id

FS-6620 mod_blacklist fix not deleting entries from list

Silence warning from clang-3.5 dealing with redefining the variable argv which is first defined as a function parameter

Silence a warning in clang-3.5 dealing with implicit conversion from 64bit to 32bit in a function call to switch_ivr_sleep()

FS-6719 #resolve

add checkbox to demo to use stun or not

always start payloads at 102 to avoid webrtc funk

map emails

FS-6690 #resolve

add switch_separate_file_params function so when using relative paths with bracketed params the full path can be constructed with the params in tact

FS-6718 #message adding the patch

Treat `*-min.js` files as minified for git

Treat minified files as binary for git

This prevents e.g. `git log -p` from showing the content changes in
these files by default, which is what we want because line-oriented
diffs of these files are meaningless.

only set the not_ready flag on one side on video unbridge

FS-6712 #resolve

FS-6711 #resolve

FS-6707 #resolve #comment add the param iceServers to verto constructor.  possible values: true: enables previous behaviour, false: nowthe default, use no iceServers, ARRAY of iceServer objects: this object will be passed into the browser as-is

Add DELETE request support to mod_curl

Fix build on Debian sid/jessie

Debian has packaged libpq version 9.4beta2.  We were parsing the minor
version as "4beta2" and then trying to treat it as a constant integer,
which obviously doesn't work.

There's not really a perfect way to handle versions like this, so
we'll just pretend it's 9.4.0.

fix typo that can lead to seg

fix extraneous < in sip url parser

CID 1229134: fix small memory leak

FS-6701 #resolve

FS-6702 #resolve

Improved HDLC abort handling

FS-6631 #resolve

Fixes for DTC handling in the T.38 gateway

using hosted version to avoid blocks, following cb7a338 7596689

trivial compiler fix

queue more messages

tweaks