GitLab

The files in this directory are outdated (for sflphone) and no longer
used.

Change-Id: Ic8aa820c842b31414656c7dae0be433932941bca
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

This commit offers an alternative to the dbus client to communicate with
a headless daemon. It uses restbed, a cpp framework for RESTful services.

This commits wraps up:
- The source of the restbed implementation (in bin/restcpp)
- The integration of the target to the configure.ac
- documentation
- Add external restbed library to contrib:
  - rules.mak file, to download the version 4.0, and compile it
  - Patch for ASIO, a dependency of restbed
  - Patch for the CMakeLists.txt file, to correct the installation path

[guillaume.roguez@savoirfairelinux.com: merged with Simon Zeni patch to support restbed into contrib]
[guillaume.roguez@savoirfairelinux.com: add tuleap ticket number]
[guillaume.roguez@savoirfairelinux.com: fixed configure.ac --without-restbed behavior]

Change-Id: Id49a1a04b05aac1e803981833abe5564785fb801
Tuleap: #498
Reviewed-by: Alexandre Lision <alexandre.lision@savoirfairelinux.com>
Signed-off-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

A regression introduced by commit 88405c00 causes
framerate emulation not using the correct timing.
milliseconds was wrongly used in place of microseconds.
This patch changes this fact.

Change-Id: I29453582ce4f0a8f34838caea4a2ee9be5592dbe
Tuleap: #914

ZRTP is not longer supported since long now.
Many code and API are deprecated and public API is stubbed.
This patch removes all this dead code.

Change-Id: Id86315ecf461deee7c81b303e19a5d4d9b5c3cc1
Tuleap: #889

This feature provides relevant advanced information during a call.
A user starts it with a refresh time and listens on a callback signal
to get the updated information. This information is presented in a form
of a map containing Call ID, codecs, framerate and resolution.

The public API has:
- startSmartInfo(<timeMS>)
- stopSmartInfo()
- CallSignal::SmartInfo(map) -- callback signal

Change-Id: Iaafc12b5b53aee8ad008b93536f72a4152c4ee02
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Change-Id: I843431c6062a66d01661c22cf87c5d18ab0bf810
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Change-Id: I976dd85f6f497dfaa2441cfc45f521f7999262e6
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Into the darwin OS detection the AC_EGREP_CPP macro is used.
This causes an bugged configure file issued and this one
prevents Linux platform to generate shared-library.

Following this link:
https://www.gnu.org/software/autoconf/manual/autoconf-2.69/html_node/AC_005fACT_005fIFELSE-vs-AC_005fTRY_005fACT.html

Using AC_EGREP_CPP is a bad way to test a C preprocessor.
This patch changes this call by AC_COMPILE_IFELSE + AC_LANG_PROGRAM calls:
- this is a better way to process
- this fix the linux shared build

Change-Id: I150666f2bc45a48d021110025f2d486a9750d333
Tuleap: #699

possible data race fixed in dtor of ShmHolder.

Coverity CID # 1296815

Change-Id: I85c8158ec8fd8ada8feb28720350f2237a1097f3
Tuleap: #909

Change-Id: I04da2e05e6bb8d769acfdbda809517ca489b37eb
Reviewed-by: Stepan Salenikovich <stepan.salenikovich@savoirfairelinux.com>

Fix definition of RingAccount::createIceTransport
to pass by reference and not by value as ICE option
structure is not small (more than 200 bytes).

coverity CID #1369566

Change-Id: I41f17a266384e28e1cab9a17e17c8436f07a22d4
Tuleap: #909

This patch removes a non-valid character inside HAVE_COREAUDIO.
This char is difficult to see if you editor is not able
to show non-valid char.

But carefully checking configure log show this text without the patch:

./configure: line 6831: $'\302\240test': command not found

You can also see it using Emacs that detects such bad char.

Change-Id: I1e1e62fb6e03c42ac9d9758bc44db33fad088482
Tuleap: #909

A break missing into a switch in MediaDecoder::correctPixFmt
causing invalid handling of AV_PIX_FMT_YUVJ440P input value.

Change-Id: I216797df58938212e83195dfc9dbfe4d0aba5a8f
Tuleap: #909

Add a return to an error handling of unwaited rx IM msg.

Coverity CI # 1296815

Change-Id: Ib4503f639e80f7a24e039043f70f8597b6f141d4
Tuleap: #909

This patch fix two calls to strdup() without any free().

Change-Id: I8619794e71d4165c154ba6ddd89bf6237e444a67
Tuleap: #906

Set true as default value to members:
allowPeersFromHistory_
allowPeersFromContact_
allowPeersFromTrusted_

Coverity CID # 1369576

Change-Id: I38499608ee35692e96c511f08d261eea8ad9b72b
Tuleap: #909

Set default value of mutex_nesting_level_ to 0.

Coverity CID # 1291689

Change-Id: Ib794fe5a90679669c837e2ad8064998caae01f42
Tuleap: #909

AC_CHECK_LIB was not correctly checking for the presence of zlib.
We use PKG_CHECK_MODULES instead to add -lz to libring_la_LDFLAGS.

Change-Id: Icfffb898030f886aae8c2c82dc5327f923f0f78e
Tuleap: #746
[stepan.salenikovich@savoirfairelinux.com: modified to use PKG_CHECK_MODULES and ZLIB_LIBS variable instead of -lz]
Signed-off: Stepan Salenikovich <stepan.salenikovich@savoirfairelinux.com>

Init default value of caValidationOutput_ member of TlsValidator class.
This prevents warns from static coverage tests.

(issue detected by coverity.com analysis)

Change-Id: Ib9a930bb233208cd99b7003d1a72b6d8003e3e9d
Tuleap: #909

The file size was wrongly checked in loadFile,
giving a always true comparaison.

(issue detected by coverity.com analysis)

Change-Id: I2aa4f5c737df3a3b815caebc97bdc3820df8b534
Tuleap: #909

remove unreachable code

Change-Id: I6c9fbccc0693d9edfc3367477db60fc04a4ddba0
Reviewed-by: Stepan Salenikovich <stepan.salenikovich@savoirfairelinux.com>

When receiving an incoming SIPS or Ring call, enforces the
secure state of the media as it should be per the SIP
standard.

Also prevents to accidentally configure an insecure
transport on a secure call.

Change-Id: I28836c76f79218a026b07d7c182adbaaf50aedbb
Tuleap: #901

This patch prevents SocketPair::getRtcpInfo() to lock a mutex
during a consequent memory copy and reduce copy cpu load by using
move iterators.

Change-Id: Ie729cb0677c45639f5a92ae842e93032c26f1a02

PA default device is selected by passing nullptr to
pa_stream_connect_playback and pa_stream_connect_record,
which is the way described in the official documentation:
https://freedesktop.org/software/pulseaudio/doxygen/stream_8h.html#ab9544f6677af133fbe81bf8a21eb489c

We used to attempt to guess the default device by incorrectly using
information from the PA server (pa_context_get_server_info).

Change-Id: Ifa8109b0ba86b223a2eca3ff136822b338fbf730
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Cleanup and factorise incoming call matching API.

Change-Id: I4b6ceb24372a189c3bf3b5bbbaf97c59912140ec
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

This reverts commit 1e249e4f.

The idea is "good" in theory, but the implementation of
setSecure() causes an exception error.
We need to find a better solution.

Change-Id: I4062388782c07ba3d5a9ab8eb9df655be676d9e4

Change-Id: Ia59165ea7d200c7c7cde666edb3918517119f199
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

This caused an "infinite loop" for audio media resulting
in CPU being overcharged for nothing

This fix the problem on MacOS and on GNU/Linux when using PulseAudio
There is still a similar problem with Alsa.

Change-Id: I70e111c59125143d60b657898b04a0b65a638bd6
Tuleap: #576

Change-Id: I802006aa7a139413da6e857e2030f817d06e9941
Reviewed-by: Andreas Traczyk <andreas.traczyk@savoirfairelinux.com>
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

SIP mandates a secure media transport when using
a secure SIP transport. To respect this, Ring
disables media when a secure call has no
secure media configured (sipcall.cpp:834).
Instead of failing, a better behavior would be to
force enabling media encryption when TLS is active at
the account level.

Change-Id: I25b77248be5d50bf016f959f0a06ec4850fa642e
Reviewed-by: Andreas Traczyk <andreas.traczyk@savoirfairelinux.com>

Add replace dirname with _splitpath to facilitate native Windows
building.

Change-Id: I57712f9f9642c221555b522c312484fed0012f03
Tuleap: #797

The Microsoft compiler will include the common header <intrin.h>
regardless of the use of quotes. The file "intrin.h" should be renamed,
and all references to it changed, as well as defining the UNUSED macro
to nothing on the Windows platform.

Change-Id: Id04d36e70d149746be8d78eed09aa963514b2810
Tuleap: #801

Change-Id: I439bec2be0978e991511e42af464da765939d593
Tuleap: #805

Remove a return preventing DH params to be generated.
This was introduced by commit 74151247

Tuleap: #792
Change-Id: I131166f7fef655d93425d10da59af04d6104e7f6

SdesNegotiator::negotiate() does an invalid check on given offered crypto-suite.
The current setup is made in a way that this function always returns
the remote crypto suite, supported or not! Dangerous.

This patch fixes that by correctly comparing crypto-suite names.

Change-Id: I0ef022486e00b5fef91d2552b83d57463282a683
Tuleap: #747

archiver::compress used to rely on compressBound, but the actual
compressed size might be smaller, leading to larger output (with
trailing zeroes).

Change-Id: Ic33614f2d6bc648e59a76dec7275e71466ecaff1
Tuleap: #888

This patch reduces the maximal payload of all UDP stream to 1232 bytes.
This value is computed from the minimal MTU requested by rfc 2460
to support IPv6.
This value ensures that common networks (going to IPv6) support
a large enough packet size to be efficient, but a minimal size to
not be fragmented. This last case has a great incidence on packet drop
and reordering and validated by heuristic.

Change-Id: Icee79bb25b2fdcc70fef3efb4527209f8309be61
Tuleap: #887

Add a Microsoft port of dirent.h to support directory parsing
functionality. It is named windirent.h to avoid cases where the
compiler searches for headers in local directories first regardless
of chevron wrapping.

File source: https://github.com/tronkko/dirent
commit bf060f1569b156a6df15ea01ea0b4b5465243f06

Change-Id: I45bcf59883949a390b5471b62013a6d95d36903e
Tuleap: #799

* Add Microsoft compiler support;
* Work around the issue compiling on Apple's LLVM 7.3.0;
* Add Network id feature;
* License OpenDHT compatible with GPL>=3;
* Adding cryptographic API;
* Consistent library naming between diffrent build tools.

Change-Id: I4e7a53619d89f7b2034326ea8d5f75848335c533
Tuleap: #556

The new --no-checksums parameter allows for skipping checksums
verifications. This can be used to replace tarballs and still use the
contrib system.

This is useful for atleast the two following cases:

 - Distro developper needs contrib to patch the source, but has the
   original source in the distribution.

 - Distro developper repacks the source, breaking the checksum.

Change-Id: I015ce3d0f6f4fb2f5fd5ec2f95528bfb4b045298
Tuleap: #879