Previously, the build machine would need to provide dbusxx-xml2cpp
itself, while the library would be used from the contribs (always
forced, as dbusxx-xml2cpp is unmaintained and slightly broken on newer
GCCs).  This change removes the requirement to have the dbus-c++
package installed on the build machine.

Change-Id: I25ad2ee3e80afc33b320de9256b724a3c254f4fb

Maxim Cournoyer authored 4 years ago and Adrien Béraud committed 4 years ago

Otherwise the '-lm' linker flag would not be used and compilation the
opus configure time check would fail like (excerpt from config.log):

    pkg-config --exists --print-errors opus
    check_func_headers opus_multistream.h opus_multistream_decoder_create -I/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/include/opus -L/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/lib -lopus
    test_ld cc -I/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/include/opus -L/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/lib -lopus
    test_cc -I/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/include/opus -L/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/lib
    BEGIN /tmp/ffconf.Ad6jtRUj/test.c
        1	#include <opus_multistream.h>
        2	#include <stdint.h>
        3	long check_opus_multistream_decoder_create(void) { return (long) opus_multistream_decoder_create; }
        4	int main(void) { int ret = 0;
        5	 ret |= ((intptr_t)check_opus_multistream_decoder_create) & 0xFFFF;
        6	return ret; }
    END /tmp/ffconf.Ad6jtRUj/test.c
    gcc -I/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/include -fPIC -D_ISOC99_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_POSIX_C_SOURCE=200112 -D_XOPEN_SOURCE=600 -DPIC -I/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/include -DNDEBUG=1 -O3 -fPIC -I/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/include -DNDEBUG=1 -O3 -std=c11 -fomit-frame-pointer -fPIC -I/include -pthread -I/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/include -I/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/include/opus -L/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/lib -c -o /tmp/ffconf.Ad6jtRUj/test.o /tmp/ffconf.Ad6jtRUj/test.c
    gcc -L/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/lib -L/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/lib -Wl,--as-needed -Wl,-z,noexecstack -I/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/include/opus -L/home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/lib -o /tmp/ffconf.Ad6jtRUj/test /tmp/ffconf.Ad6jtRUj/test.o -lopus
    ld: /home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/lib/libopus.a(celt_decoder.o): in function `celt_decode_lost':
    celt_decoder.c:(.text+0x20d0): undefined reference to `sqrtf'
    ld: celt_decoder.c:(.text+0x20f5): undefined reference to `sqrtf'
    ld: /home/mcournoyer/src/ring-project/daemon/contrib/x86_64-unknown-linux-gnu/lib/libopus.a(bands.o): in function `compute_theta':
    bands.c:(.text+0xe87): undefined reference to `sqrtf'
    ld: bands.c:(.text+0x1178): undefined reference to `sqrtf'

The problem likely was discovered because using a subset of the
required dependencies that didn't pull the math library.

Change-Id: I79bc4f79eaa2288bf20528cd770f1ddafd56dec3

Adrien Béraud authored 4 years ago and Sébastien Blin committed 4 years ago

Change-Id: I2ea856c183df317eab45a5d6d5533ef1b3b6a0df

Change-Id: I2daba297b06c7886d29ba613a6341de33ad7fa86

Change-Id: Iabc63814f515d53f1de8fc539ba49e154843a1cd

IceTransport desctruction shouldn't block on the main thread.
This problem is noticable when calling refuseCall from the UI
thread and results in a blockage lasting MAX_DESTRUCTION_TIMEOUT.
Even if MAX_DESTRUCTION_TIMEOUT is not reached, a polling interval
of 500ms requires this destruction be done async.

Change-Id: Ib6f9c0cff155755de30966875ab76e7288d39405

Vladimir Stoiakin authored 4 years ago and Sébastien Blin committed 4 years ago

Change-Id: I69cc9da36dfbb1258f8f84e9351a6c1c29c2a722

Change-Id: I81b8d9988aaa79d6e67142417e9f004689d69ab9

Kateryna Kostiuk authored 4 years ago and Kateryna Kostiuk committed 4 years ago

When we stop playback stream we should check current audio category.
If it is AVAudioSessionCategoryPlayAndRecord it is means we also have
capture stream running and we should not destroy it.

Change-Id: I4cc0f2ddcd8b26840f4b2968a890c4803c187448

Also adds portaudio api specific device preferences in the config
using strings instead of indices. An empty string will cause the
default communication device to selected.

Change-Id: I64597c2d706acdd6e1422558d7609fc14a7ad818
Gitlab: #463

Ming Rui Zhang authored 4 years ago and Adrien Béraud committed 4 years ago

GitLab: #478

Change-Id: I4aed6f7b171a2194fd0070c5510ca1660adb6204

This reverts commit 6cfe9432.

Reason for revert: causes reference to temporary

Change-Id: I2ff3e94c490ee1f17d3e96486d4a1aa5732d684f

Adrien Béraud authored 4 years ago and Sébastien Blin committed 4 years ago

Change-Id: If1b647a76303362ae0240a0e11505502260b5c8d

Sébastien Blin authored 4 years ago and Adrien Béraud committed 4 years ago

Change-Id: Iedcfe91af40893004c8f640c8a718966db8d7468

Pierre Lespagnol authored 4 years ago and Adrien Béraud committed 4 years ago

Gitlab: #465
Change-Id: Ib86573e5c0e186253c51624b38b9951d77f1a1fa

Change-Id: Ica016084fb36c3f4110fe4fe3281e43d29c65275

Change-Id: I155f5182fbb370116e020151dc3c154cee66c814
GitLab: #407

like SIPCall::onMediaUpdate, we should not restart medias if
the call is ended or a crash will typically happen with multi-device
where first device answer and hangup during the negotiation with
the other.

Change-Id: Ib6bd2a316c8f03ace25926fd20338d72613d0c54

Adrien Béraud authored 4 years ago and Sébastien Blin committed 4 years ago

Change-Id: Id2d03ca3d645d7178019bf05c1951c6a78019095

Adrien Béraud authored 4 years ago and Sébastien Blin committed 4 years ago

Change-Id: I9273985b6c6156e6e5a39388cdf32ed48fccd58d

Mohamed Chibani authored 4 years ago and Sébastien Blin committed 4 years ago

In AudioRtpSession, when a new AudioInput instance is created, its mute state
is always set to default (un-muted) even if the AudioRtpSession is in mute
state.
Fixed by applying the mute state when a new AudioInput instance is created.

Gitlab: #482

Change-Id: I3f7e92cfcfd63bc5c6318f317e7fa96fe5f17a7c

Mohamed Chibani authored 4 years ago and Sébastien Blin committed 4 years ago

SipVoipLink may never be intialized in Manager instance (Manager::init())
leading to a crash when the Manager calls finish(). Added a check
to prevent the crash.

Gitlab: #481

Change-Id: I817a7795ce4341e6214a44e46037b0183fde53c5

Pierre Lespagnol authored 4 years ago and Sébastien Blin committed 4 years ago

Gitlab: #479
Change-Id: I19c6286311936d61c8a850f99b4ed3008b675b48

Mohamed Chibani authored 4 years ago and Sébastien Blin committed 4 years ago

This patch adds many improvements and bug fixes:
1- For PUPNP lib, use the synchronous methods for add and remove
requests. The async versions have many issues mainly at shutdown.
If the callbacks do not arrive on time, the memory allocated by
the lib when performing the request will leak.
2- Rework the init and IGD search of PUPNP.
3- Move the termination step (for libupnp in particular) from
the destructor to a newly added method (termiinate()). This will
prevent a race at shutdown between libupnp threads (callbacks) and
the daemon shutdown process.
4- Do not try to initialize libupnp if the host address is invalid.
This will prevent putting the lib in an unstable state.

Gitlab: #472

Change-Id: I14e5ddcd2b735da374a1f09bcef40f6a86a38133

GitLab: #404
Change-Id: I42792e27fb2bdf21870477ccb821f672033fc25d

Instead, use Bash from PATH.  This approach works universally, even in
environments not strictly adhering to the File Hierarchy
Standard (FHS).

Change-Id: I852c5f4bba3a1bee9b895dcb5da24e5738c4186a

The DirectSound host api implementation in PortAudio has multiple
issues:
- requires utf8 support patch
- can't enumerate the default communication devices
- reports incorrect default sample-rates for devices

The WASAPI host api implementation needs to be patched to access
the default comm devices, but functions better out-of-the-box.

Change-Id: Ie27ee3e1418dd6d734ad6ad7685f2dc44db86007
Gitlab: #463

Andreas Traczyk authored 4 years ago and Adrien Béraud committed 4 years ago

- Enables high-pass filter, AEC, AGC, NS.
- Disables system AEC for Windows

Gitlab: #464
Change-Id: Ife6261a815395263abeca482bb78ad2c90133db3

Allows the start/stop of individual streams as well as both input
and output simultaneously. When both streams are to be started,
full duplex mode is attempted first.

Resets device indices to -1 in the case that the host api has
changed, or if the dring audio config is scrapped somehow.

Change-Id: I1655a34d2111222b6add19f1b36b53bc4a2838ec

Andreas Traczyk authored 4 years ago and Adrien Béraud committed 4 years ago

Hide speexdsp's echo cancellation implementation details behind an
EchoCanceller derived class. An AudioLayer may now instantiate
implementations of the EchoCanceller. This may be platform specific
compile-time or swapped out at runtime.

Introduces a null echo canceller for testing. As the speexdsp aec
is not currently functional, the null echo canceller is replaced
as the default implementation.

GitLab: #454
Change-Id: I169f1e9758afbed884fc42d9d78a69ce28d12fe2

Andreas Traczyk authored 4 years ago and Adrien Béraud committed 4 years ago

Change-Id: Ib7396b12f98947b25bc96605381cbc29594aa1b7

Change-Id: I38ffcd79e3392b1eb8a763914fa182e1f17a2685

Regression was caused by 37125acf where the mute button didn't
work anymore.

Change-Id: I7e3d0953e65848818c9dc0e348759dce63274337

Apply patches for CVE-2021-21375 and CVE-2020-15260 affecting
prproject. After a quick look I don't think those issues apply to
us, as GnuTLS manages the TLS part and we do resolution outside
pjsip (the daemon passes ips), but at least our pjproject will
not be affected.

Note, these patches will be removed when we bump pjproject to 2.11.

Change-Id: Ie9029f5501096c7148416cbb803a40f1c053a3d0

Change-Id: Ia9cf5638678c0e92127ec9bf3631235a2827d49d

Pierre Lespagnol authored 4 years ago and Adrien Béraud committed 4 years ago

During the detach, the audio input need to be unbinded from
subcalls composing the conference, not from all calls.
For the removeConference, the leaving call is unbinded in
removeAudio and the remaining call is not unbinded (no need
to unbind all and re-bind remaining call)

Gitlab: #414

Change-Id: I900acac1293f0f8c6e13a74309167d7356015089

Aline Gondim Santos authored 4 years ago and Sébastien Blin committed 4 years ago

GitLab: #422
Change-Id: Id94eb245d9675d305a0d05da76e62bcd245525ec

- adds a missing space in the configure command line
- removes some python3 warnings

Change-Id: I79458a408a6f3198cb329072abf530ed8f23986a

Change-Id: Iff69c5888acba0a9a165bce4f4106a217dc78134

This allows the rendez-vous to be considered as a
conference and not as participant for the other
conference.

Change-Id: I9053533a605f1928da93ed3d3e9235db8b764591