- 19 Aug, 2015 2 commits
-
-
Philip Withnall authored
Summary: Some versions of Windows can return ECONNRESET for UDP recvmsg() calls if they would otherwise block. Hence, handle the two equivalently; this should not affect behaviour on Linux, which apparently does not return ECONNRESET for UDP recvmsg() calls at all. https://phabricator.freedesktop.org/T121 Maniphest Tasks: T121 Reviewers: ocrete Projects: #libnice Reviewed By: ocrete Subscribers: stwiname, felixSchl Differential Revision: https://phabricator.freedesktop.org/D227
-
Philip Withnall authored
Summary: Otherwise the base socket will leak. Spotted by Vadim Genkin. https://phabricator.freedesktop.org/T125 Maniphest Tasks: T125 Reviewers: ocrete Projects: #libnice Reviewed By: ocrete Subscribers: vadimgenkin Differential Revision: https://phabricator.freedesktop.org/D228
-
- 18 Aug, 2015 4 commits
-
-
Philip Withnall authored
Otherwise the base socket will leak. Spotted by Vadim Genkin. https://phabricator.freedesktop.org/T125
-
Philip Withnall authored
Summary: As spotted by Felix <felixschlitter@gmail.com>. This is a static function which is totally unused in this compilation unit and is causing build failures with `-Werror=unused-function`. Maniphest Tasks: T123 Reviewers: felixSchl, ocrete Projects: #libnice Differential Revision: https://phabricator.freedesktop.org/D221
-
Philip Withnall authored
As spotted by Felix <felixschlitter@gmail.com>. This is a static function which is totally unused in this compilation unit and is causing build failures with -Werror=unused-function. http://phabricator.freedesktop.org/T123
-
Philip Withnall authored
Some versions of Windows can return ECONNRESET for UDP recvmsg() calls if they would otherwise block. Hence, handle the two equivalently; this should not affect behaviour on Linux, which apparently does not return ECONNRESET for UDP recvmsg() calls at all. https://phabricator.freedesktop.org/T121
-
- 03 Jul, 2015 1 commit
-
-
Jakub Adam authored
A new socket created in nice_tcp_active_socket_connect() should have its writable callback set, because it's possible for it to become a base socket of a peer reflexive candidate, if some is discovered by connection checks on that TCP active candidate. Previously, when such prflx candidate became selected, without write_cb on the socket the agent was never notified about it becoming writable again after the socket's buffer got filled up. This caused the data flow to hang permanently. Reviewed-by:
Philip Withnall <philip.withnall@collabora.co.uk> Reviewed-by:
Olivier Crête <olivier.crete@collabora.com> http://phabricator.freedesktop.org/T117
-
- 01 Jul, 2015 1 commit
-
-
Philip Withnall authored
Recent versions of MinGW define at least ECONNABORTED and EAFNOSUPPORT, so only define the various socket errnos if they are not defined already. Based on a patch by Alexey Pawlow <alexey.pawlow@gmail.com> and Felix <felixschlitter@gmail.com>. Reviewed-by:
Olivier Crete <olivier.crete@collabora.com> Reviewed-by:
Felix Schlitter <felixschlitter@gmail.com> http://phabricator.freedesktop.org/T122
-
- 30 Jun, 2015 1 commit
-
-
Felix Schlitter authored
The C runtime on windows does not implement a printf that understands %zu and other C99 format specifiers. Use G_GSIZE_FORMAT instead. This is further consistent with the rest of the code base that makes use of G_GSIZE_FORMAT throughout. https://github.com/libnice/libnice/pull/3
-
- 24 Jun, 2015 2 commits
-
-
Olivier Crête authored
This way, the regular log will only contain connection-time information.
-
Olivier Crête authored
-
- 01 Jun, 2015 1 commit
-
-
Timo Gurr authored
Error introduced in 20ea22e0, resulting in a configure/build error when building without gstreamer: configure: error: conditional "HAVE_GST_CHECK" was never defined. Usually this means the macro was only invoked conditionally. https://bugs.freedesktop.org/show_bug.cgi?id=90801
-
- 08 May, 2015 1 commit
-
-
Philip Withnall authored
We’ve neglected to manually update this file once too often — it’s been out of date for important new symbols (for example, nice_agent_get_io_stream()) since at least 0.1.11. Since the format is a simple extension of libnice.sym, we might as well automatically generate it at dist time.
-
- 05 May, 2015 3 commits
-
-
Youness Alaoui authored
-
Youness Alaoui authored
This caused issues with thinking local host candidates were peer-reflexive candidates because the nice_address_equal would fail since the scope would be 6 (or some other value) but locally created NiceAddress from a stun response would have the scope set to 0. We ignore the scope when comparing ipv6 candidates when scope is 0 to avoid these kinds of issues. Thanks to ikonst_ for finding these issues
-
Youness Alaoui authored
When adding a remote candidate, if it's the same ip:port, we should also check its type, otherwise it's a new candidate. We can't allow a candidate type to be updated. This caused issues to ikonst_ on IRC where for some reason a host candidate appeared as both host and prflx and the update caused a remote host candidate to be updated to prflx causing a crash when the sockptr was being accessed.
-
- 03 May, 2015 1 commit
-
-
Philip Withnall authored
With the changes in commit 483bdcf8, @name is now guaranteed to be non-NULL. Spotted by Coverity. CID: #109878
-
- 28 Apr, 2015 3 commits
-
-
Olivier Crête authored
-
Olivier Crête authored
-
Olivier Crête authored
-
- 24 Apr, 2015 3 commits
-
-
Philip Withnall authored
sockaddr_to_string() is not needed on UNIX when compiling without HAVE_GETIFADDRS. This fixes a compiler warning in that case.
-
Philip Withnall authored
-
Philip Withnall authored
This fixes compilation when HAVE_GETIFADDRS is not define. Reported on the mailing list by Curieux Tres <trescurieux@yahoo.fr>.
-
- 23 Apr, 2015 6 commits
-
-
Philip Withnall authored
This fixes some compiler warnings.
-
Olivier Crête authored
-
Olivier Crête authored
-
Olivier Crête authored
-
Olivier Crête authored
-
Olivier Crête authored
-
- 22 Apr, 2015 1 commit
-
-
Olivier Crête authored
Fixes SDP parsing which I broke in commit 5972db04
-
- 20 Apr, 2015 10 commits
-
-
Sebastian Dröge authored
Receiving data before the element is ready to push them does not seem like the best idea. https://bugs.freedesktop.org/show_bug.cgi?id=89673
-
-
Olivier Crête authored
-
Olivier Crête authored
-
Olivier Crête authored
This remove a gcc warning
-
Olivier Crête authored
The only valid way to match streams is to count m= lines, see RFC 3264. https://bugs.freedesktop.org/show_bug.cgi?id=90019
-
Olivier Crête authored
Only 6 strings are valid. https://bugs.freedesktop.org/show_bug.cgi?id=90019
-
Olivier Crête authored
The include file names are very generic, at least make the guards a bit less generic. https://bugs.freedesktop.org/show_bug.cgi?id=90013
-
-