freeswitch-stable: cleanup hotplug script

- clean up variable names
- replace pgrep with pidof
- remove LOG_WARN and use LOG_ERR instead

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

freeswitch-stable: don't force disable on install

Don't force freeswitch to off during postinstall. It's not common
practice (anywhere) and it may be confusing.

Add a link to the Wiki, too, and bump the revision.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

freeswitch-stable: move hotplug into main pkg

There is no point in keeping the hotplug script in an extra package.
Move it to the main package.

The priority of the script is changed from 99 to 90 as the freeswitch
init script uses the same.

The postinstall script will check if 99-freeswitch exists. If it does a
warning is displayed with the suggestion to uninstall the hotplug
package.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

freeswitch-stable: procd compatibility

procd runs as pid 1. When freeswitch realizes that its parent has pid 1,
it thinks that it has been orphaned and terminates the console thread.
That's why procd isn't able to get freeswitch's console output.

This commit mutes the check in src/switch_console.c. Now console output
is properly logged. The workaround parameters "-nc -nf" are replaced by
a proper "-c" ("console").

Additionally this commit hands control over killing freeswitch to procd.
It's no longer done by the script. This simplifies things quite a bit.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #377 from dhewg/pull/asterisk

asterisk related updates

Merge pull request #383 from micmac1/kam-awk-and-mips-tweak-master

kamailio-5.x: fix awk issue and some tweaks

kamailio-5.x: enable FAST_LOCK for MIPS

This tweaks MIPS builds to support a faster locking mechanism.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

kamailio-5.x: enable parallel build

Enable parallel build for Kamailio package.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

kamailio-5.x: make jsonrpc filter portable

With OpenWrt default awk - so awk from busybox - the filter is broken,
causing jsonrpc calls to fail, i.e. when running "kamctl ps". Below
patch makes the filter portable. Patch was already accepted upstream.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #381 from micmac1/ast-opus

asterisk-opus: new package

asterisk-opus: new package

Add package for Opus codec support plugin. Variants for both asterisk13
and asterisk15 included.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #379 from robho/yate_internal_regexps

yate: Revert to internal regex implementation

asterisk-13.x: update to 13.23.1

Update to the current 13.x release, see [0].
Drop merged patches and refresh the rest.

Get rid of the unknown --without-pwlib configure switch.
Drop the direct ncurses dependency, it's unused in case of an external
libedit [1].

[0] http://downloads.asterisk.org/pub/telephony/asterisk/ChangeLog-13-current
[1] https://issues.asterisk.org/jira/browse/ASTERISK-27761

Signed-off-by: Andre Heider <a.heider@gmail.com>

asterisk-15.x: update to 15.6.1

Update to the current 15.x release, see [0].
Drop merged patches and refresh the rest.

Get rid of the unknown --without-pwlib configure switch.
Drop the direct ncurses dependency, it's unused in case of an external
libedit [1].

[0] http://downloads.asterisk.org/pub/telephony/asterisk/ChangeLog-15-current
[1] https://issues.asterisk.org/jira/browse/ASTERISK-27761

Signed-off-by: Andre Heider <a.heider@gmail.com>

pjproject: update config_site.h

Sync asterisk settings, see [0].

[0] https://github.com/asterisk/asterisk/blob/master/third-party/pjproject/patches/config_site.h

Signed-off-by: Andre Heider <a.heider@gmail.com>

yate: Revert to internal regex implementation

.. because the musl implementation doesn't seem to be fully compatible
with yate. We switched to the musl implementation in f6ad95d.

Yate has this regexp:
^([[:alpha:]][[:alnum:]]+:)?/?/?([^[:space:][:cntrl:]@]+@)?([[:alnum:]._+-]+|[[][[:xdigit:].:]+[]])(:[0-9]+)?

Given a string like

  sip:012345678@11.111.11.111:5060;user=phone

musl's regexec() returns these matches:

index start end
0     -1    0
1     0     32   sip:012345678@11.111.11.111:5060
2     -1    -1
3     0     14   sip:012345678@
4     14    27   11.111.11.111
5     27    32   :5060

.. but this is what yate expects:

index start end
0     -1    0
1     0     32   sip:012345678@11.111.11.111:5060
2     0     4    sip:
3     4     14   012345678@
4     14    27   11.111.11.111
5     27    32   :5060

Fixes #378

Signed-off-by: Robert Högberg <robert.hogberg@gmail.com>

Merge pull request #376 from micmac1/fs-1.8.2-master

freeswitch-stable: bump to 1.8.2

freeswitch-stable: bump to 1.8.2

- bump version
- go back to using release tarballs (unavailable when 1.8.1 was
  initially released)
- add OpenWrt mirror

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #370 from micmac1/kama-mas-cve

(master) kamailio-5.x: add fix for CVE-2018-16657

Merge pull request #373 from micmac1/master-AST-2018-009

(Master) Fix for AST-2018-009

asterisk-15.x: fix for AST-2018-009

Add upstream fix for AST-2018-009: Remote crash vulnerability in HTTP
websocket upgrade

The vulnerability affects the res_http_websocket.so module.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-13.x: fix for AST-2018-009

Add upstream fix for AST-2018-009: Remote crash vulnerability in HTTP
websocket upgrade

The vulnerability affects the res_http_websocket.so module.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

kamailio-5.x: add fix for CVE-2018-16657

In Kamailio before 5.0.7 and 5.1.x before 5.1.4, a crafted SIP message with
an invalid Via header causes a segmentation fault and crashes Kamailio. The
reason is missing input validation in the crcitt_string_array core function
for calculating a CRC hash for To tags. (An additional error is present in
the check_via_address core function: this function also misses input
validation.) This could result in denial of service and potentially the
execution of arbitrary code.

Patch from upstream.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #368 from micmac1/fs-cpe

freeswitch-stable: add PKG_CPE_ID

freeswitch-stable: add PKG_CPE_ID

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #367 from micmac1/fs-uci

freeswitch-stable: replace init config defaults with UCI

freeswitch-stable: replace defaults with uci

This commit replaces /etc/default/freeswitch with /etc/config/freeswitch.
This way the init and hotplug configuration can be done with uci instead of
having to edit a file.

This also does away with the busybox ntpd warning. ntpd uses the same
configuration in system and it looks like busybox's ntpd is not used when
ntpd is installed.

Lastly some log strings are amended to start with a lowercase letter.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

freeswitch-stable: disable parallel build

There is a discussion ongoing if parallel builds should be enabled by
default. For freeswitch parallel builds are known to fail when certain
modules are enabled. This commit preemptively disables parallel builds
explicitly.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #365 from micmac1/fs-perl-master

freeswitch-stable: fix perl setup

freeswitch-stable: fix perl setup

A recent commit in packages/lang/perl is causing a problem with the
perlmod.mk include. Work around this by including perlver.mk instead and
setting up PERL_CMD and PERL_SITELIB manually.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #360 from micmac1/kam-cve-master

kamailio-5.x: add fix for CVE-2018-14767

Merge pull request #346 from micmac1/asterisk-uclibc-iconv

asterisk: fix iconv related build issue

Merge pull request #351 from micmac1/sccp-nls

asterisk-chan-sccp: remove iconv hack

Merge pull request #352 from micmac1/dongle-nls

asterisk-chan-dongle: remove iconv hack + update snapshot

kamailio-5.x: add fix for CVE-2018-14767

CVE-2018-14767: "In Kamailio before 5.0.7 and 5.1.x before 5.1.4, a
crafted SIP message with a double "To" header and an empty "To" tag
causes a segmentation fault and crash. The reason is missing input
validation in the "build_res_buf_from_sip_req" core function. This could
result in denial of service and potentially the execution of arbitrary
code."

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #358 from micmac1/fs-1.8-sigkill

freeswitch-stable: add SIGKILL to init script

freeswitch-stable: add SIGKILL to init script

Sometimes freeswitch doesn't exit after receiving the SIGTERM signal.
This can be reproduced by sending SIGTERM to a freeswitch instance which
is initializing (which can take quite some time).

Instead of just giving up and exiting - leaving a hung freeswitch
process on the system - this commit adds some lines to the init script
that send SIGKILL to freeswitch in case the attempt to terminate it with
SIGTERM fails.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #347 from micmac1/yate-uclibc-iconv

yate: upgrade to 6.1.0 and fix uclibc build issue

Merge pull request #357 from micmac1/fs-1.8-fs_cli

freeswitch-stable: fix garbled output in fs_cli

freeswitch-stable: fix garbled output in fs_cli

Fix a typo. Sent to upstream via Jira FS-11309.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #356 from cotequeiroz/freeswitch-stable_memory-leak

freeswitch-stable: fix memory leak in mod_event_multicast

freeswitch-stable: fix memory leak in mod_event_multicast

Applied a patch, submitted upstream at
https://freeswitch.org/jira/browse/FS-11193
that fixes two memory leaks in mod_event_multicast.c

Signed-off-by: Eneas U de Queiroz <cote2004-github@yahoo.com>

Merge pull request #355 from micmac1/fs-1.8-mods

freeswitch-stable: add new modules

freeswitch-stable: add new modules

1. fail2ban
2. raven
3. video_filter

All three are new in FreeSWITCH 1.8.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #354 from micmac1/fs-1.8

freeswitch-stable: bump to 1.8.1

freeswitch-stable: upgrade to 1.8.1

- Bump version from 1.6.20 to 1.8.1 (includes OpenSSL 1.1.0 support).
- Upstream didn't provide a source tarball yet, switch to git to grab
  the release.
- freeradius-client got updated to 1.1.7 by upstream, update version and
  checksum accordingly.
- tiff, libvpx and pcre patches are no longer needed, drop them.
- Refresh the remaining patches. Only minor changes were needed.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

freeswitch-stable: remove mod_v8

mod_v8 is too difficult to maintain and tricky to cross-compile. This
commit removes it.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-chan-dongle: update snapshot

Includes USSD encoding fix for E3531.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-chan-dongle: remove iconv hack

Remove the hack for iconv on uclibc. Instead include nls.mk which will
sort out the iconv depends.

This changes the iconv patch. AC_SEARCH_LIB doesn't work properly for
openwrt/lede. Instead use AC_CHECK_LIB.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-chan-sccp: remove iconv hack

Instead just include nls.mk.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #350 from micmac1/fs-nls3

freeswitch-stable: fix gsmopen inbuf handling

freeswitch-stable: fix gsmopen inbuf handling

inbuf is only const in libiconv-full when compiled against uclibc.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #349 from micmac1/fs-nls2

freeswitch-stable: fix sed script

freeswitch-stable: fix sed script

The sed script used on libs/unimrcp is used to set LDFLAGS using
APR_SETVAR. Since nls.mk is included there are LDFLAGS with commas. But
the macro APR_SETVAR uses commas to separate arguments, so now the
LDFLAGS are only getting partially copied.

Solve this by surrounding LDFLAGS with brackets ([...]).

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #348 from micmac1/fs-nls

freeswitch-stable: fix iconv related issues

freeswitch-stable: include nls.mk, remove hack

Removes the uClibc iconv hack. Instead nls.mk is included to sort out
iconv depends.

mod_gsmopen isn't able to figure out if iconv's inbuf is const. In
OpenWrt this is the case with libiconv-full, not with libiconv-stub. To
aid mod_gsmopen update 270-fix-uclibc-iconv-in-gsmopen.patch so that we
can pass on if inbuf is const via CFLAGS.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

freeswitch-stable: fix APR_TRY_COMPILE_NO_WARNING

The macro included in apr/apr-util is incorrect. It results in extra
brackets being put into conftest.c, for example:

  #include "confdefs.h"

[
  #include <stddef.h>
  #include <iconv.h>
  ]
int main(int argc, const char *const *argv) {
[
  iconv(0,(char **)0,(size_t *)0,(char **)0,(size_t *)0);
  ]
  return 0; }

The test always fails with:

error: expected identifier or '(' before '[' token

The following checks are affected:

- variable length arrays
- type of inbuf parameter to iconv

Fix from upstream (Apache).

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-15.x: fix iconv related build issue

libmariadb 10.2 needs to be linked in together with iconv. Include nls.mk
to help the build system find the iconv library.

Also update func_iconv's depend. We want to use the variable
$(ICONV_DEPENDS) provided by nls.mk. It resolves to
+BUILD_NLS:libiconv-full. This cannot be handled properly right now, as the
module depends filter cannot handle +<SYMBOL>:<package> depends. So add
another field to the Makefile's module function for complex depends, which
are passed on unfiltered.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-13.x: fix iconv related build issue

libmariadb 10.2 needs to be linked in together with iconv. Include nls.mk
to help the build system find the iconv library.

Also update func_iconv's depend. We want to use the variable
$(ICONV_DEPENDS) provided by nls.mk. It resolves to
+BUILD_NLS:libiconv-full. This cannot be handled properly right now, as the
module depends filter cannot handle +<SYMBOL>:<package> depends. So add
another field to the Makefile's module function for complex depends, which
are passed on unfiltered.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

yate: update to 6.1.0

Upgrade yate. The video codec mapping patch needed some small changes.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

yate: fix mod-mysqldb compile

Since libmariadb 10.2.x requires to be linked in together with iconv,
include nls.mk to help the build system find it.

Also fix a type in PKG_CONFIG_DEPENDS.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #343 from micmac1/AST-master

(for master) asterik: fix AST-2018-007 and AST-2018-008

asterisk-15.x: patch AST-2018-007 and 008

Patches from upstream for:

AST-2018-007
AST-2018-008

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-13.x: patch AST-2018-008

Patch from upstream for AST-2018-008.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #339 from micmac1/fs-pcre

freeswitch-stable: fix incompatible pointer type

freeswitch-stable: fix incompatible pointer type

Currently compiles fail with:

  CC       libfreeswitch_la-switch_regex.lo
src/switch_regex.c: In function 'switch_regex_compile':
src/switch_regex.c:40:9: error: return from incompatible pointer type [-Werror=incompatible-pointer-types]
  return pcre_compile(pattern, options, errorptr, erroroffset, tables);
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
Makefile:2100: recipe for target 'libfreeswitch_la-switch_regex.lo' failed
make[5]: *** [libfreeswitch_la-switch_regex.lo] Error 1

Fix cherry-picked from upstream.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #332 from guidosarducci/siproxd-procd-cleanup

siproxd: switch to procd, restructure/cleanup code, bugfix

siproxd: restructure code and use procd for init

Switch to use procd for init, with support for jails. Clean up code and
restructure callback processing to be more robust and understandable. This
also fixes a bug processing multiple siproxd config sections, as reported
by Michael Kuron.

Signed-off-by: Tony Ambardar <itugrok@yahoo.com>

siproxd: don't apply default values already built-in

Signed-off-by: Tony Ambardar <itugrok@yahoo.com>

Merge pull request #330 from micmac1/master

asterisk: remove sounds cache

asterisk-15.x: remove sounds cache

When there is no sounds cache declared the build system will not attempt to
download any sound packs or their SHA1 checksums.

This is to be preferred because:

  a) the build may occur offline, causing it to fail
  b) plain http is used by the build system for downloading

There is no drawback here because the standard sound packs are included in
the Asterisk source tarball already.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-13.x: remove sounds cache

When there is no sounds cache declared the build system will not attempt to
download any sound packs or their SHA1 checksums.

This is to be preferred because:

  a) the build may occur offline, causing it to fail
  b) plain http is used by the build system for downloading

There is no drawback here because the standard sound packs are included in
the Asterisk source tarball already.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #326 from micmac1/yate-fix-mysql-detection

yate: fix mysql detection

yate: fix mysql detection

When configure is called with '--with-mysql="$(STAGING_DIR)/usr"' it tries
to detect mysql manually instead of using mysql_config. This doesn't always
work:

<snip>
checking for MySQL in /var/lib/buildbot/slaves/slave-lede-builds4/i386_pentium4/build/sdk/staging_dir/target-i386_pentium4_musl/usr... no
<snip>

This commit changes the configure argument to '--with-mysql=yes/no'. When
'yes' is selected configure uses mysql_config and then the detection works
reliably.

<snip>
checking for MySQL using mysql_config... yes 10.1.32
checking whether MYSQL_OPT_RECONNECT is declared... yes
checking whether MYSQL_OPT_READ_TIMEOUT is declared... yes
checking whether MYSQL_OPT_WRITE_TIMEOUT is declared... yes
checking whether mysql_set_character_set is declared... yes
<snip>

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #321 from micmac1/chan-lantiq-fix-target-build

asterisk-chan-lantiq: fix dep for nonshared builds

Merge pull request #322 from micmac1/baresip-up

Baresip suite: version bumps

Merge pull request #323 from micmac1/ast-up

Asterisk: version bumps

Merge pull request #324 from micmac1/ast-modules-up

asterisk-channel modules: bump sccp and dongle

Merge pull request #325 from micmac1/kamailio-up

kamailio-5.x: bump to 5.1.3

kamailio-5.x: bump to 5.1.3

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-chan-sccp: bump snapshot

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-chan-dongle: bump snapshot

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-15.x: bump to 15.3.0

- patches refreshed
- menuselect is bootstrapped from main bootstrap script now, extra call
  removed
- drop a module that was removed upstream for being incomplete

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-13.x: bump to 13.20.0

- patches refreshed
- menuselect is bootstrapped from main bootstrap script now, extra call
  removed

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

baresip: bump to 0.5.9

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

rem: bump to 0.5.3

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

re: bump to 0.5.8

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

asterisk-chan-lantiq: fix dep for nonshared builds

Despite all previous efforts chan-lantiq is not built by the build bots.
Checking the logs of a target build reveals that the chan-lantiq builds are
not even attempted.

During the target builds CONFIG_ALL is not set, meaning no regular package
gets selected. CONFIG_ALL_NONSHARED on the other hand is set. But
chan-lantiq still will not be selected, because it depends on "asterisk",
meaning asterisk needs to be selected first for chan-lantiq to become
available.

This commit changes the asterisk depend to "+asterisk", meaning asterisk
will be selected if chan-lantiq is selected.

This was tested with the SDK. A seed config file was added with the
following:

CONFIG_ALL_NONSHARED=y

Running menuselect showed that now chan-lantiq is selected
automatically. This should fix the problem.

The problem with "+asterisk" is that this breaks the asterisk menus for all
asterisk plugins/utilities that follow chan-lantiq alphabetically.  To
address this put the chan-lantiq modules into their own sub menu "Telephony
Lantiq". This is not nice, but the alternative would be to rename the
packages to something like asterisk13-z-chan-lantiq. This would make it
harder to find the packages. Maybe somebody has a better idea in the
future.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #320 from micmac1/chan-lantiq-no-variants

asterisk-chan-lantiq: avoid VARIANTs

asterisk-chan-lantiq: avoid VARIANTs

chan-lantiq currently is still not built by the build bots. Split the
package in two, one for Asterisk 13, the other for 15, to avoid a VARIANT
build. Maybe the build bots do not support nonshared VARIANT builds.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

Merge pull request #315 from micmac1/siproxd-fix-package-names

siproxd: remove underscores from package names

Merge pull request #312 from micmac1/baresip2-fix-package-names

baresip: remove underscores from package names

Merge pull request #313 from micmac1/rtpproxy-fix-package-names

rtpproxy: remove underscores from package names

Merge pull request #316 from micmac1/kamailio-fix-package-names

kamailio-5.x: remove underscores from package names

Merge pull request #317 from micmac1/yate-fix-package-names

yate: remove underscores from package names

Merge pull request #311 from micmac1/asterisk-fix-package-names

Asterisk: fix package names

Merge pull request #318 from micmac1/fs-pp-tone-down-flags

freeswitch-stable: tone down perl/python flags

freeswitch-stable: tone down perl/python flags

Tidy up the flags passed on to FS build system for Perl and Python
modules. Some flags seem a bit too random to include - this commit
removes them.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

yate: remove underscores from package names

Underscores should not be used in package base names as they're used as
semantic separators by opkg.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>

kamailio-5.x: remove underscores from package names

Underscores should not be used in package base names as they're used as
semantic separators by opkg.

Signed-off-by: Sebastian Kemper <sebastian_ml@gmx.net>