| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | stunmessage: Revert another function to use sockaddr | Olivier Crête | 2014-05-02 | 5 | -10/+13 |
| | | | | | The farstream unit tests were using it. | ||||
| * | stunagent: Only declare the long term key valid if the username/realm coudl ↵ | Olivier Crête | 2014-04-24 | 1 | -2/+2 |
| | | | | | be read | ||||
| * | stun: Restore argument type to stun_usage_bind_process(), it's used outside ↵ | Olivier Crête | 2014-04-15 | 2 | -9/+11 |
| | | | | | libnice | ||||
| * | Fix possible segfault in stun_message_validate_buffer_length() | Jakub Adam | 2014-04-04 | 1 | -1/+10 |
| | | |||||
| * | build: Check for [s]size_t before redefining them on MinGW | Philip Withnall | 2014-04-03 | 1 | -1/+4 |
| | | | | | | | MinGW defines size_t and ssize_t for us, so we should not unconditionally redefine them in stun/win32_common.h. Add an AC_CHECK_TYPES configure check to avoid this. | ||||
| * | stun: Use struct sockaddr_storage * for any pointer that we write to | Olivier Crête | 2014-03-31 | 16 | -142/+134 |
| | | | | | | | Some platforms have a larger alignment requirement for struct sockaddr_* than for plain struct sockaddr, in that case, we need to make sure that the incoming pointer has the right alignement before taking it in. | ||||
| * | stun tests: Fix a bunch of issues found by more aggressive GCC warnings | Olivier Crête | 2014-03-31 | 4 | -52/+57 |
| | | |||||
| * | stun: Add a fast version of stun_message_validate_buffer_length() | Philip Withnall | 2014-01-31 | 2 | -13/+107 |
| | | | | | | | | | | | | | | | stun_message_validate_buffer_length() is already fast, but requires the entire message to be in a single monolithic buffer. For introducing vectored I/O, this becomes impossible to guarantee. Rather than rewriting the STUN code to natively support vectors of buffers (which would be a huge undertaking, and would probably slow the code down considerably), implement a fast check of whether a message is likely to be a STUN packet which *does* support vectored I/O. This can then be used to determine whether to compact the vector of buffers to a single monolithic one in order to validate the message more thoroughly. | ||||
| * | stun: Use SHA1 iteration functions directly in HMAC calculation | Philip Withnall | 2014-01-31 | 1 | -53/+26 |
| | | | | | | | | | | Rather than using an intermediate method which takes a vector of buffers, just use the SHA1 iteration functions directly in the HMAC calculations. This eliminates several cases where the vector of buffers was of fixed length, and it was actually taking more code to set up and pass the vector than it would have to call the iteration functions directly. | ||||
| * | stun: Add missing switch cases | Philip Withnall | 2014-01-31 | 2 | -0/+11 |
| | | | | | | | | | This appeases GCC’s -Wswitch-enum warning, and makes it more obvious that those enum cases have been explicitly considered, rather than just forgotten about. This introduces no functional changes. | ||||
| * | stun: Fix potential zero-length memset() call | Philip Withnall | 2014-01-31 | 1 | -3/+5 |
| | | | | | GCC warns about this. Might as well prevent the warning. | ||||
| * | stun: Make the MD5 code strict-aliasing correct | Olivier Crête | 2014-01-31 | 2 | -16/+19 |
| | | |||||
| * | stun: Add printf function attribute | Philip Withnall | 2014-01-02 | 1 | -0/+7 |
| | | | | | | This shuts a compiler warning up and allows for format string checking of debug messages. | ||||
| * | stun: Fix format specifier for a size_t variable | Philip Withnall | 2014-01-02 | 1 | -1/+2 |
| | | | | | | | | | | This fix may not be entirely cross-platform, and I have been unable to test whether it is; making it cross-platform is made more difficult by the fact that the STUN code doesn’t use GLib. The PRIuPTR macro was defined in POSIX:2004, and later in C99. http://pubs.opengroup.org/onlinepubs/009696799/basedefs/inttypes.h.html | ||||
| * | configure: Verify that the compiler understands warning cflags | Olivier Crête | 2013-12-26 | 2 | -2/+2 |
| | | | | | Older compilers don't understand all of the flags | ||||
| * | stun: Indications are never authentication when using long term auth | Olivier Crête | 2013-12-23 | 1 | -1/+2 |
| | | |||||
| * | stun: Fix a use of a function with an aggregate return value | Philip Withnall | 2013-12-18 | 2 | -6/+5 |
| | | | | | | | | | | | div() has an aggregate return, which GCC doesn’t like, although this seems like a pretty pointless warning because div_t is the same size as a pointer on 64-bit platforms (probably) and hardly going to cause performance problems by passing it by value. Anyway, it seems easier to simplify the code by using explicit / and % operators inline, than it does to add pragmas and shut the warning up. | ||||
| * | stun: Explicitly avoid a memcpy() from NULL | Philip Withnall | 2013-12-18 | 1 | -1/+3 |
| | | | | | | | | | | | | | If stun_message_append_bytes() is called through stun_message_append_flag(), data will be NULL and len will be 0. This will result in a memcpy(ptr, NULL, 0) call. This probably won’t do any harm (since any reasonable memcpy() implementation will immediately return if (len == 0)), but the standard allows for memcpy() to explode if (data == NULL), regardless of the value of len. In order to be conformant, and to shut up the scan-build static analysis warning about it, only do the memcpy() if (len > 0). | ||||
| * | stun: Remove unused variables | Philip Withnall | 2013-12-18 | 1 | -2/+0 |
| | | |||||
| * | Fix strict aliasing of sockaddr structures | Philip Withnall | 2013-12-18 | 6 | -37/+66 |
| | | | | | | | | | | | | | Casting from one struct sockaddr type to another breaks C’s strict aliasing rules (variables of different types cannot alias). Fix this cleanly by using unions of struct sockaddrs to convert between the types (i.e. type-punning). I wish sockaddr didn’t have to be this painful. See: http://gcc.gnu.org/onlinedocs/gcc-4.4.1/gcc/Optimize-Options.html#Type_002dpunning | ||||
| * | Add missing ‘default’ cases to switches | Philip Withnall | 2013-12-18 | 6 | -3/+36 |
| | | | | | | | | This shuts GCC’s -Wswitch-default warning, and makes the code flow a little more explicit. This introduces no functional changes. | ||||
| * | Use %lu for long unsigned int when calling printf | Rohan Garg | 2012-05-01 | 1 | -1/+1 |
| | | |||||
| * | Use len instead of buf_len to check whether or not data was received properly | Rohan Garg | 2012-03-23 | 1 | -1/+1 |
| | | |||||
| * | Properly ifdef windows headers in order to get compiles working on linux again | Rohan Garg | 2012-03-12 | 1 | -1/+1 |
| | | |||||
| * | Fixed mingw/msys compilation. Definition for intptr_t was missing | Filippo Della Betta | 2012-03-05 | 1 | -0/+1 |
| | | |||||
| * | Fixed compiling on Visual Studio and removed getsockname before bind. Added ↵ | Filippo Della Betta | 2012-03-05 | 1 | -12/+23 |
| | | | | | WSAStartup/WSACleanup on win32 | ||||
| * | Fixed compile on mingw/msys platform | Filippo Della Betta | 2012-03-05 | 1 | -0/+5 |
| | | |||||
| * | set nonblocking mode for socket on Windows | Livio Madaro | 2012-02-23 | 1 | -0/+6 |
| | | |||||
| * | Fixed warnings on Visual Studio platform | Filippo Della Betta | 2012-02-17 | 1 | -1/+1 |
| | | |||||
| * | Added typedef on size_t and ssize_t that are not defined on Visual Studio ↵ | Filippo Della Betta | 2012-02-15 | 1 | -0/+5 |
| | | | | | platform | ||||
| * | Included the replacement header win32_common.h instead of the missing header ↵ | Filippo Della Betta | 2012-02-15 | 1 | -0/+4 |
| | | | | | stdint.h on WIN32 platform | ||||
| * | Use size of the MD5Context, not size of the pointer | Rohan Garg | 2012-02-14 | 1 | -1/+1 |
| | | |||||
| * | stund: Use recvfrom/sendto instead of recvmsg/sendmsg | Youness Alaoui | 2012-02-09 | 1 | -67/+15 |
| | | |||||
| * | stund: Set message length (fixes infinite loop) when receiving unknown ↵ | John Selbie | 2012-02-09 | 1 | -1/+1 |
| | | | | | attributes | ||||
| * | Changes windows specific headers to lower case | Siraj Razick | 2011-12-19 | 1 | -1/+1 |
| | | | | | This helps us when cross compiling libnice for Windows from Linux | ||||
| * | Add support for windows's compilation through MinGW | Nader Morshed | 2011-11-30 | 9 | -15/+25 |
| | | |||||
| * | Remove warnings on stun/tests | Timothy Redaelli | 2011-10-13 | 1 | -1/+2 |
| | | |||||
| * | Fix support for solaris | Apostolos Syropoulos | 2011-09-07 | 1 | -0/+4 |
| | | |||||
| * | Google puts the STUN cookie, but doesn't follow RFC 5389 | Olivier Crête | 2011-08-29 | 1 | -1/+2 |
| | | | | | | Google puts the stun cookie, but it doesn'T follow RFC 5389 at all, so ignore the STUN cookie in Google mode. | ||||
| * | Avoid a possible coverity false positive | Youness Alaoui | 2011-03-23 | 1 | -0/+2 |
| | | |||||
| * | Change variable buffer_len into attribute_len: variable already exists in ↵ | Youness Alaoui | 2011-03-23 | 1 | -5/+5 |
| | | | | | different scope | ||||
| * | Add some new APIs to sha1 module, unused for now | Youness Alaoui | 2011-03-23 | 2 | -9/+75 |
| | | |||||
| * | Fix coverity issues, remove possibly NULL dereferencing | Youness Alaoui | 2011-03-23 | 1 | -2/+4 |
| | | |||||
| * | Also initialize the number of timer retransmissions | Sjoerd Simons | 2011-02-09 | 1 | -0/+1 |
| | | |||||
| * | define SOL_IPV6 to IPPROTO_IPV6 if not defined | Youness Alaoui | 2011-01-17 | 2 | -0/+11 |
| | | |||||
| * | Refuse to create a permission without a peer instead of not sending the ↵ | Youness Alaoui | 2010-12-17 | 1 | -5/+6 |
| | | | | | XOR_PEER_ADDRESS | ||||
| * | Some minor changes, code cleaning | Youness Alaoui | 2010-12-16 | 2 | -2/+1 |
| | | |||||
| * | Fix lines that are > 80 chars | Youness Alaoui | 2010-12-14 | 1 | -1/+2 |
| | | |||||
| * | Fix compilation errors as a result of the rebase | Youness Alaoui | 2010-12-14 | 1 | -1/+1 |
| | | |||||
| * | fix indentation issues | Youness Alaoui | 2010-12-14 | 1 | -34/+34 |
| | | |||||
 |
index : delta/libnice.git | |
| gitlab.freedesktop.org: libnice/libnice.git |
| summaryrefslogtreecommitdiff |