Merge branch 'upstream' into tizen

It is updated to 0.1.21.

Change-Id: If2d9e7f4f256bcc1b2a522402cc1540dcb1bd89e
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>

Merge tag '0.1.21' into upstream

Libnice release 0.1.21

Change-Id: I9709b8167a04c0a8eedb3f33fe47a21cc6f7f0c8

version 0.1.21

interfaces: Only use `ifr_ifindex` if OS supports it

This fixes compilation on e.g. macOS and FreeBSD, where it is missing.

version 0.1.20.1

version 0.1.20

version: When set nano 0 to be a string

Setting it as a int makes meson abort.

gst: Use gst-full compatible registration functions

gst: Remove GStreamer 0.10 support

The meson build already required 1.x so this has been dead code for a while now.

test-consent: Don't try to set the consent prop after creation

It was already set at creation time, so all we got was a warning.

address: Add introspection-friendly to_string_dup

nice_address_to_string expects an already-allocated string, which is not
great for introspection. Adds nice_address_to_string_dup, which does the
same thing, but returns a newly allocated string.

test-turn: Make it work against coturn

agent: Allow getting UDP candidates from a TURN server over TCP

Even if the TCP candidates are disabled.

agent: Resolve TURN server IP addresses

This should enable having servers that do both IPv4 and IPv6. And
ideally should make life easier for app developers.

agent: Resolve STUN server IP address

This should enable having servers that do both IPv4 and IPv6. And
ideally should make life easier for app developers.

candidate: fix assertions caused by API guards in new address API

The checks were reversed, requiring for instance a non-relay candidate
for calling nice_candidate_relay_address()

Redirect stun_debug to dlog

 - TIZEN avoid stun_debug_disable() called by environment variables

Change-Id: I606e1718c7422f1f47e756e34cbacf2bef74388e

gitlab-ci: Fix typo

ci: Expose full code coverage report to GitLab

ci: Add code coverage to MRs

Add configuration file for lcov

This is useful for accurately testing code coverage. You can use this with:

  meson configure -Db_coverage=true
  meson test
  ninja coverage

ci: Export JUnit report

ci: Use fdo templates

conncheck: Limit ichecks based on property

Limit the number of stored incoming checks based on the property
limiting the number of total connectivity checks instead of using
the fixed limit on the number of remote candidates.

Conncheck: always updating remote_consent.last_received for selected pair on keepalive

Change-Id: Iab47642897e57154fb422881470c125bf637343a

Conncheck: recheck consent timeout overflow fix

Change-Id: I1dba3b74119f7a3dab016c060696d379ab84d26e

udp-bsd: Explicitly bind to the interface when creating a socket

Disable stun/tools/*

'stund' and 'stunbdc' installed to /usr/bin/ are excluded to compiling
which are not necessary.

Not applying ASLR issue since updated to 1.18 also can be resolved with it.

[Version] 0.1.19-1
[Issue Type] Binary

Change-Id: Ic9f25590b9eb76a913a6ef4b5aa316dfa679a269
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>

interfaces: Add API to find the interface index from a local address

This will find some interface that has the address and will not work
correctly if multiple interfaces have the same address. But in any
case, nothing in libnice expects that and we would need to break the
API to make that work.

Merge branch 'upstream' into tizen

It is updated to 0.1.19.

Change-Id: I119abe2c8b875469483bae1fd67f6d0a166b778d

Merge tag '0.1.19' into upstream

Libnice release 0.1.19

Change-Id: I959060bdca7d21eec19e99d3ce5ff9ed3412a8ca

candidate: Copy stun_server with candidate

Bug reported and fix suggested by Masato Takahashi

Fixes #164

version: Add NICE_CHECK_VERSION to the documentation

address: Add Boxed Type

This should make the library more easily usable for bindings

agent: Support adding optional headers to HTTP proxy client

One use case for this is adding an ALPN header which is a MUST
requirement when a HTTP proxy is used in WebRTC (see RFC8835, section
3.4).

Conncheck: renomination: set use_candidate for pending check on early inbound STUN with NOMINATION attr

Conncheck: always updating remote_consent.last_received for selected pair on keepalive

NICE_CHECK_VERSION: fix unexpected "git = next version" assumption

0.1.18.1 should not satisfy a check for 0.1.19, especially not
for libraries that release once a year at best.

If someone needs more control they should do a feature check for
the symbol in the headers or lib.

Fix NICE_CHECK_VERSION in public install

Install a nice-version.h so that the version defines
are available.

https://gitlab.freedesktop.org/libnice/libnice/-/merge_requests/232

Conncheck: recheck consent timeout overflow fix

meson_option: Enable dlog

[Version] 0.1.18-2
[Issue Type] Log

Change-Id: I666e75c78ef1309b523ec2bb79781b2313f1bfad
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>

candidate: Add utility function to get STUN server address

Can be useful to know which STUN server was used to discover the given candidate.

docs: Add index for symbols added in 0.1.20

Add NICE_CHECK_VERSION macro

agent: Initialize variable

There is a codepath where it can be used without being filled.
Found by coverity.

version 0.1.19.1

version 0.1.19

candidate: Add utility function to get TURN relay address

Can be useful to know which TURN server is being used as relay server for a given candidate.

agent: Hold ref while doing async close

It's possible that the callback will be called synchronously during the
close function, in this case, we need to be ready to the last reference
to the agent to be dropped in this callback.

This fix was suggested by Fabrice Bellet

Fixes #153

Don't try to use link-local addresses outside local network

If we have gathered a host IP-address that is link-local we should
never try to use it for anything else than trying to match it with
another link-local address. Some routers seem to have problems with
traffic from link-local addresses destined at external IP-addresses.
By definition link-local addresses should stay local so there's no
reason to access STUN/TURN from it or try to form a candidate with
another address that is not link-local.

agent: Fix warning about required glib version

../agent/discovery.c: In function ‘on_refresh_remove_timeout’:
../agent/discovery.c:203:13: warning: Not available before 2.60
  203 |       G_GNUC_FALLTHROUGH;
      |             ^~~~~~~~~~~~~

../agent/conncheck.c: In function ‘priv_turn_allocate_refresh_retransmissions_tick_agent_locked’:
../agent/conncheck.c:1656:13: warning: Not available before 2.60
 1656 |       G_GNUC_FALLTHROUGH;
      |             ^~~~~~~~~~~~~

The comment /* fall through */ is enough to cause GCC and Clang to not
emit the implicit-fallthrough warning.

agent: Convert g_print() to nice_debug()

This was probably added by mistake.

Update spec file and meson files

Change-Id: I155c032e191ebbc0bcfaf62e6904c1ad28f5dfee
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>

meson: Fix openssl setting

This should fix builds on platforms openssl

Change-Id: If13f747fe358f1e7e0043356043a5085781d09c6

meson: Simplify finding OpenSSL

Remove duplicated calls to dependency() so we can have fallback in all
cases.

Change-Id: Ic278734a9824e1646aec50a1d807b0451f8bf834

meson: Fix name of variable

This is a fix over the recent fix in commit 574f44b4

Change-Id: I922f0895ae13d5802aea4775860f642d18c8d86c

Fix error when crypto/ssl libs are found without pkg-config

In that case crypto_dep is a list and we can't call .found() method.

Change-Id: Ie63d78530d9a935312263bf9843f76667b17c4f6

Merge branch 'upstream' into tizen

It is updated to 0.1.18.

Change-Id: Ife96288be82519f7cec61c48d5f6f53edd98de79

Merge tag '0.1.18' into upstream

Libnice release 0.1.18

Change-Id: I3af1a67390b647bd36d23002f396bc2406659c7b

Initialize git

Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>

Exclude gst plugins from build

Gstreamer plugins move on to the gstreamer mono git since 1.20.0.
 : gstreamer/subprojects/gst-plugins-bad/ext/nice

Change-Id: Ib006250b5261ded170f6ba17725d79a24ed000b2
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
(cherry picked from commit 4e38bd1424281f45de01779ec1b1f2b301107f5a)

meson: Fix openssl setting

This should fix builds on platforms openssl

gitlab-ci: Allow manual runs

tcp-active: Fix use-after-free in error case

Fixes issue found by Coverity

agent: Clear the original message pointer instead of a copy

In one case, the message pointer gets replaced by the rfc4571_message
one, so it wasn't getting cleared as expected.

gstnicesink: disable drop out of segment on nicesink

It's very tricky and unintuitive to have webrtcbin not sending data
because of the segment.

Fix https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/907

test-io-stream-common: Avoid CPU starvation by yielding

Busy-looping is not a good idea, especially not when run under Valgrind,
where such a thread may result in well-behaved threads running thousands
of times slower. While passing --fair-sched=yes to Valgrind avoids the
issue, it's still better to make our busy-looping less aggressive.

agent: Consume data before removing disconnected socket

This was caught by tests randomly failing (timing out).

iostream: Implement close() for ICE-TCP also

component: Clear buffer pointers to avoid potential double free

In case close() is called more than once.

test-io-stream-thread: Handle partial I/O

test-io-stream-closing-write: Handle partial I/O

test-io-stream-closing-read: Handle partial I/O

test-io-stream-common: Remove incorrect assertion

We cannot assume that the message will be completely filled even in
reliable mode.

test-io-stream-common: Fix GSource callback signature

Cannot pass g_main_loop_quit as the GSourceFunc as its return type is
void.

test-io-stream-common: Remove dead code

Due to off-by-one errors. Since the code added above now handles what
this code tried to handle, we can go ahead and remove it.

agent: Add support for bytestream TCP

conncheck: don't ignore local socket errors

With this patch we ensure that local socket errors during connection
establishment are properly transmitted to the connection check layer, so
the related pair can be put in state failed when needed. The local
socket errors we are interested in, are those occuring when the local
source address cannot be bound anymore, because the underlying interface
vanished for example.

In particular, we don't ignore errors coming from g_socket_bind() with
tcp active sockets, that perform a bind to *whatever* local address is
available, in case it cannot bind to the specified address.

This behaviour was demonstrated with the test-tcp example, that tried
to bind to the IPv4 loopback address on a socket initially created for
the IPv6 loopback.

meson: Add agent include dir to uninstalled pkgconfig file

Meson picks up the includedirs from the library target that it is
building the pkgconfig file for. Since libnice's headers are split
into the nice/ and agent/ subdirs, we need to add agent/ to
include_directories: so that it's automatically added to the
uninstalled pkgconfig file by Meson.

agent: Handle failure to create TCP TURN socket

E.g. if the server address is IPv6 but the local system cannot reach it,
so connect() fails right away.

conncheck: schedule the connection check in most relevant places

The patch invokes the conncheck timer creation where it makes most
sense: when a new pair is inserted in the conncheck list, and when a
pair is added to the triggered check list (because the conncheck
scheduler works on these two lists).

stun: Use a specific variable to enable Win32 crypto

This should make it possible to use OpenSSL or GnuTLS also on Windows if
required.

gitlab-ci: Try to disable CI on non-MR/non-master branches

gitlab-ci: Add OpenSSL specific build

meson: Simplify finding OpenSSL

Remove duplicated calls to dependency() so we can have fallback in all
cases.

meson: Fix name of variable

This is a fix over the recent fix in commit 574f44b4

tests: Use g_assert_true() instead of g_assert() everywhere

Remove g_assert_cmp* macros outside of the unit tests

agent: ensure suceeded and discovered pairs fail the same time

This patch ensures that related succeeded-discovered pairs change to
state failed simultaneously, to avoid leaving dangling pointers if one
is freeed while the other is still in the conncheck list. Such
transition is very rare, and only occurs in regular nomination mode,
when the network conditions change between the time the pair is
initially discovered and the time it is rechecked with the use-candidate
flag.

Fix error when crypto/ssl libs are found without pkg-config

In that case crypto_dep is a list and we can't call .found() method.

agent: Unify constructors into nice_agent_new_full

Instead of having 3 independent implementations, make the simpler
versions delegate into the full one.

The simplest constructor needs to pass "no option", for which no
constant exists in the NiceAgentOption enum. Thus the new
NICE_AGENT_OPTION_NONE constant is added to reflect this possibility.

Use native crypto support on Windows by default

As we only need a few primitives there isn't much OS-specific code
needed, and for applications that don't already depend on OpenSSL or
GnuTLS it simplifies the build process quite a lot.

Note that we use the vintage WinCrypt APIs as the Cryptography Next
Generation APIs are only available on Vista and newer.

component: reset the stun agent on ice restart

The stun agent may contain references to the password previously stored
in some remote candidates, freeed by nice_component_restart(), that were
used by keep-alive stun requests. These stun replies may arrive later
after ice has been restarted. Since the remote candidates are freeed
when ice is restarted, the stun agent must be reset to get rid of these
related references.

gitlab-ci: Update Windows image

gitlab-ci: Update ci-fairy to upstream recommended script

Change nice_debug_verbose as dlog message

Change-Id: I6d759bf95019e6fcd02bede420bd88d6f980f18e

nice: Bring new symbols to 0.1.19

It's not released yet and we don't do even/odd in this project.

nice: Export nice_candidate_transport_to_string

Also update the Since to 0.1.19, because it's not really usable now

nice: Export nice_candidate_type_to_string

Also update the Since to 0.1.19, because it's not really usable now

Use dlog to debug agent

 - Log tag is LIBNICE_AGENT

Change-Id: I19252689850078b8452e4cd7ae51dc04f275bf4f