Avoid dereferencing potential null value

Remove assignment without effect

Avoid dereferencing potential null value

Handle potential malloc failure better

Fix unlikely memory leak in ESL

Remove assignment without effect in fs_cli

Avoid dangling reference at end of fs_cli

Not that it really matters since we're returning to the OS momentarily
at that point anyway.

Add common include paths for jni.h and jni_md.h

FS-5479 --resolve

fix unused parameter warning on windows

Fix variable type in mod_nibblebill

Work around error in pocketsphinx-0.7

  fsg_search.c:263:2: error: non-void function 'fsg_search_reinit' should return a value [-Wreturn-type]

Remove self-assignment without any effect

clang:

  error: explicitly assigning a variable of type 'uint16_t' (aka 'unsigned short') to itself [-Werror,-Wself-assign]

Remove extraneous parentheses in mod_say_pl

Add missing newline at end of mod_say_fa.c

Remove extraneous parentheses

This resolves a clang warning:

  error: equality comparison with extraneous parentheses [-Werror,-Wparentheses-equality]

Add commit note about shell escaping

Disregard the supplemental commit text on
commit 04c3b913aa077f34dbeec710310da9cbeb8bcbd3.

switch_util_quote_shell_arg() is fine.

Fix typo

Remove unused variable in mod_rtmp

Unused variables break the build with recent versions of gcc and
clang.

Quote arguments to shell when sending email

Note that the function switch_util_quote_shell_arg is implemented
incorrectly for unix shells, so this is still not safe until that
function is fixed.

Reported-by: Nathan Neulinger <nneul@mst.edu>
FS-5450 --resolve

Fix memory leaks in mod_tts_commandline

We were using both switch_util_quote_shell_arg and
switch_string_replace without freeing the results even though both
functions malloc.

Leave room for switch_escape_string to add a terminating null

Thanks-to: Nathan Neulinger <nneul@mst.edu>
FS-5448 --resolve

Fix comment spelling

Thanks-to: Mariusz Czułada <manieq.net@gmail.com>

Fix memory corruption in mod_erlang_event

Thanks-to: Alexandre Snarskii <snar@snar.spb.ru>
FS-5465 --resolve

Don't setup ZRTP without a session and channel

FS-5472 --resolve

debian: wheezy is the new stable

And jessie is the new testing.

The king is dead, long live the king!

Handle a failure of strerror_r

This fixes the build on gcc-4.7.3.  The build was breaking with:

  error: ignoring return value of 'strerror_r', declared with attribute warn_unused_result [-Werror=unused-result]

Add missing text editor local variable blocks

Fix a possible deadlock scenario in the impossible scenario from the last patch.

Redundant code. The if/else block makes more sense than the equivalent in two separate if's.

Better to not pass tmp_socket to setsockopt if there was an error here creating the socket.

It's been missing a break there for ~4 years and instead using the 8k max_ms values(the ptime_div value is the same).

Cleaning up logically dead code.

This condition should NEVER be hit, but in case the started thread is unable to locate itself in the EVENT_DISOATCH_QUEUE_THREADS array, it's best to return null here and kill the unexpected event thread rather than segfault FS by writing outside of the bounds of the EVENT_DISPATCH_QUEUE_RUNNING array.

Small off by one error. If the array is of size 144, then an index of 144 inclusive is an error.

Let's bit a bit safer when dereferencing pointers.

Removing logically dead code

If there is a problem reading the dictionary file, don't leak memory...

Removing unused code

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.