Change-Id: I08bb65e78bcc475ea948e2fe7af2a72efab64425

This patch contains following changes:

* getLocalAddress : wasn't return the negotiated IP if available
* getRemoteAddress : do not check for negotiation state
* some debug added

This fixes bugs during call establishement when IPv6 pair is negotiated,
but an IPv4 TURN is given as candidate (and not selected,
but used as default IP).

Change-Id: I89a973c16674b24cce35dc6dd9433554cb3a41bd
Tuleap: #891
(cherry picked from commit b1e576a3)

Philippe Gorley authored 8 years ago and Philippe Gorley committed 8 years ago

This iterates over all DRM devices under
/dev/dri (card* and renderD*) and returns the first
device that works

Change-Id: I4a4aea565122abf81786b0ccc69539631f10b5f8
Tuleap: #1082
(cherry picked from commit d6a90bf9)

Commit ac44cbaa introduces a regression
in the support of vp8 video codec. The change uses v1.6.0 of libvpx
for all platforms except OSX. But this version causes a segfault
into the vp8 codec code.

This patch reverts libvpx in contrib to the previous working version,
for all platform, except for ANDROID as the author of the incrimined
commit has used v1.6.0 as a version buildable on ANDROID with clang.

WARNING: current android client build libvpx but doesn't enable VP8
coded! So we don't have any way to test the library on android!

Change-Id: I764907c28d24dcbe90fb69748bed5cf684d594f3
Reviewed-by: Stepan Salenikovich <stepan.salenikovich@savoirfairelinux.com>
(cherry picked from commit 4eda26c3)

If an incoming call fails due to some error (eg: 503), its ring
tone is never stopped. This stops the ring tone and audio streams
of the call on failure. Otherwise they keep playing and cannot be
stopped by the user without killing the daemon.

Change-Id: I8e3948812fab5d024a9571a99122b452dc4ca092
Tuleap: #1012
(cherry picked from commit 664a6ebf)

Missing ffmpeg/libav build flags in v4l2 Makefile.
This fix the build of this last when deps are installed
on system (i.e. using contrib, where the path doesn't contain
the implementation name).

Change-Id: I966e501ddfdeba95a87ff89ee77faa4938ebe15d
Tuleap: #1058
(cherry picked from commit bd90c365)

Philippe Gorley authored 8 years ago and Guillaume Roguez committed 8 years ago

This patch also implements a way for the Call object to restart
the media receiver

Change-Id: I1090575f14425d383eb5471a388c4c9e8d888b0e
(cherry picked from commit 660bc340)

Now correctly takes into account user setting. Will no longer try
to retrieve data from the GPU buffer if acceleration has failed.
Uses hardware acceleration when flushing the video stream.

Change-Id: Id7787a181b3822e8c7da0e8c2ce2cdfa302a3ddd
(cherry picked from commit c49f1368)

Change-Id: I2f3504946b28721b2e98eb060becc9a9fc49ca0f
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>
(cherry picked from commit 52a53711)

Manager::createSinkClient() returns a nullptr set shared_ptr
when called with an id already associated to a sink client.
This is not the waiting behaviour: returns the existing client.

This patch fixes that and also add documentation to the public
API to explicit this behaviour.

Change-Id: Idc9a0b4f7d7a06e2bdafe965891469903f0d5d99
(cherry picked from commit a06588ed)

Set .NOTPARALLEL target to enforce non-parallelizable
top Makefile rules. This protect agains two parallel builds
of inter-dependant libraries.
But each package continue to use the multi-job option if given.

Reviewed-by: Alexandre Viau <alexandre.viau@savoirfairelinux.com>
Change-Id: I98ae532dcc924ef5cae1ea235a807469ed77ed99

This patch remove the forcing on GNU/Linux platforms
to use the system provided boost.
This causes the boost deps to always be compiled.
This solves many issues with badly installed systems.

Change-Id: Ia6725882b09e9d19837da382c9c3afb88e735096

This patch adds a new virtual API to Account, flush(),
that does nothing by default. Concrete classes may overload it
when it's needed to remove filesystem entries created by them.

Then this patch adds also overloads of this method in SIPAccountBase and
RingAccount to delete all files added by these account classes.

Finaly, the removeAccount() external API calls flush by default, but
internal calls do not, to keep a conservative behaviour
and not destroying account files due to yml parsing errors.

Change-Id: I52c4e225f4d41f0e3d74f63838e56a769011b58e
Tuleap: #988

getentropy function as been added to MacOS 10.12 and iOS 10.
During GnuTLS compilation, the function can be detected in the new
10.12 SDK, but at runtime it fails to find the appropriate method on
the system.

It means that we can produce packages that won't work on some OS
versions. For precaution this commit patches GnuTLS to stop trying to
enable this function on all macOS versions.

It also removes another patch in GnuTLS for OSX that wasn't used anymore

Change-Id: I3af8b111f65e38d2d01da600c899d5717e82d8e9
Tuleap: #911

clock_gettime function as been added to MacOS 10.12 and iOS 10
so launching the Ring App would result in symbol not found on OS
before 10

Change-Id: Ifcd841379483ef9b8d17f54ee5b99f0a2526f94a
Tuleap: #911

Anthony Léonard authored 8 years ago and Guillaume Roguez committed 8 years ago

Change-Id: I10fc2928c110de960ef99e4ed3a33c9880c72b75
Tuleap: #1042

* spinoff asio as an independent contrib module
* build restbd and asio by default with daemon
* requires clang to build on Android (GCC 4.9 not supported).

Tuleap: #1042
Change-Id: I21ab2b052655b051176e7649dc58d0cd8946a92e

AccountMessageStatusChanged would never get emitted when using dbus.

Change-Id: I452bada6863131166545d917f1836894e1d84100
Tuleap: #1071

If BATCH_MODE is set to a non-empty string, downloads and uncompress
tools are silented to reduce the size of the output console log.
This is not enable by default to let manual usage fully verbose.

This is useful in case of automatic builds.

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

remove a nullptr access when a call is cancelled during
the ICE initialization of an outgoing call.

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

* adapt contrib flags for clang
* fix pjsip using CFLAGS to compile c++
* fix minor OpenSL issues

Change-Id: I8c45441052fb5efef44d54a3cb6f75407e543b7b
Tuleap: #575

Change-Id: Ibc9ffdbc0c3c7ec7648627338495ac1181a6ab8b
Reviewed-by: Nicolas Jager <nicolas.jager@savoirfairelinux.com>

Change-Id: I93db8c765b72ad42b7b89256558d19d7ca31d2e7
Reviewed-by: Nicolas Jager <nicolas.jager@savoirfairelinux.com>

This patch fixes PJSIP crash (assertion) due to too long
hostname given to resolve method.
We have a check for that but as PJSIP may prefix our given
name by such string as "_sip._udp.", the check is not enough.
This patch adds a security marge of 12 bytes to handle that.

Change-Id: Icb47e7d105ef0c8a9a10b1984f5e95bede623f8c
Tuleap: #969

This patch fixes regression introduced in commit 6f57f002.
clang++ was used in place of g++ on Android platform.

Change-Id: Idcb7baafa842eb1f68e9a9ed02e5d7949e1588d5
Tuleap: #682

Adrien Béraud authored 8 years ago and Guillaume Roguez committed 8 years ago

Change-Id: Ia972ca8b3ae99f382ca7d7433071e4c3dd07a6ee
Tuleap: #682

Contains code common to all hardware accelerations to be implemented.

Enables the VAAPI acceleration for intel linux systems, with support
for H.264, H.263 and MPEG4.

To use VAAPI, you need libva, libva-x11 and libva-drm.

Hardware acceleration is enabled by default, and can be disabled with
"./configure --disable-accel".

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

Guillaume Roguez authored 8 years ago and Adrien Béraud committed 8 years ago

This patch refactors some audio classes and its implementation
in respect of daemon coding rules and modernize its
implementation by using modern C++ facilities.

Change-Id: Ia45e33bfe43b2a60997ece7c2810054405210e26

A simple thread pool.
It will start a new thread if all current threads
are busy, up to the number of cores on the host.
Keeps pending tasks on a queue.

Adapts few uses in the daemon.

Change-Id: Ia94a64bcdd4d8d8b16cf84e5a735c5246b08c13d

Anthony Léonard authored 8 years ago and Guillaume Roguez committed 8 years ago

A new instance of TelephoneTone was created every time the
sample rate was changed. Consequently, the selected tone
was lost each time it happened and the ALSA backend was
trying to read an empty buffer in an infinite loop when the
contact wasn't answering the call.
The precise changes are :
 * A setSampleRate was added as a method in tonelist[.cpp|.h]
 * A buildTones method was also created to prevent code
   redundancy.
 * setSampleRate is used instead of recreating the object in
   tonecontrol.cpp

Change-Id: I44a86345953068848d0304516d502de5c37bb113
Tuleap: #168
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

Loïc Siret authored 8 years ago and Guillaume Roguez committed 8 years ago

This patch handles a nullptr returned by Manager::getCallFromCallID.

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

Downloading archive is less network bw consuming and almost faster
than cloning the full requested branch.

Reviewed-by: Alexandre Viau <alexandre.viau@savoirfairelinux.net>
Change-Id: I6f93179a18c13d411a83896afa3e8cc134bbd33f

Downloading archive is less network bw consuming and almost faster
than cloning the full requested branch.

Reviewed-by: Alexandre Viau <alexandre.viau@savoirfairelinux.net>
Change-Id: I7581a546088dd173fbb2f2c1e50aeae64eb274cb

We're facing a PJSIP issue when forcing a TURN server in case
of unavailability of this last.
So we prefer to revert this patch until we found a solution
for solving that.

This reverts commit bd2d9d27.

Reviewed-by: Alexandre Viau <alexandre.viau@savoirfairelinux.com>
Change-Id: I8ee88ae237a1ce56b74371cb513c7f98af1a5399

Change-Id: I6befc194929f0b4b601371d7ef4a0880108a2ce4
Tuleap: #1000

Give a default TURN server at Ring account creation.
This server is a proposed by Savoir-faire Linux.

Change-Id: I95b4c8d3e9a7efe0fcd109a5ba7246927723dced

This modifies the restbed contrib rules to use a specific version of
asio in order to make the build reproductible and more stable.

Change-Id: Ieb620da14894b8574fe57d775d13c446c792a27b
Tuleap: #1017

TlsSession state holder 'state_' is loaded twice
when we check for its modification in the TlsSession
state machine code.
std::atomic_compare_exchange_strong returns false when
'state_' is not equals to 'old_state' and in this case
'old_state' is changed for the current value of 'state_'.
So reload it right after the returns is not efficient
and may also have an unexpected corner case.

This patch re-uses the value of 'old_state' for next
processing.

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

- add "alone" files in update-copyright tool
- make a pass of this last tool to fix dates

Change-Id: I1f276a132c0d5468b81400bbb50889e80823f45c
Signed-off-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>
Reviewed-by: Stepan Salenikovich <stepan.salenikovich@savoirfairelinux.com>

When SIPS transport receives a BYE sip method,
the TLS connection is also closed at mostly the same moment.
But the handling of the BYE causes PJSIP trying to also send
an acknowledgement to this BYE. As the send cannot succeed
as the TLS session is closed and as the send return an error
in such case, PSIP ends the current INVITE by an error
and not by a success as it should do to process the BYE.
This patch fixes this problem by queueing PJSIP msg when
the TLS session is disconnected.

Change-Id: I416c1e98a8934bd794a654358c50f58141c76bac
Tuleap: #998