summaryrefslogtreecommitdiffstats
path: root/common
Commit message (Collapse)AuthorAgeFilesLines
* obey coding standard, no logic changeKoichiro IWAO2017-01-121-2/+2
|
* TLS: log TLS version and cipherKoichiro IWAO2016-11-224-0/+22
|
* Merge pull request #460 from metalefty/ipv6/logsItamar Reis Peixoto2016-11-201-29/+104
|\ | | | | common: Fix client IP address logging when IPv6 is enabled
| * common: Address family it not always AF_INET6Koichiro IWAO2016-11-041-36/+84
| | | | | | | | even if XRDP_ENABLE_IPV6 defined.
| * common: Fix client IP address logging when IPv6 is enabledKoichiro IWAO2016-11-041-25/+52
| | | | | | | | | | | | | | | | g_tcp_accept() and g_sck_accept() should use sockadd_in6 when IPv6 is enabled. The former code logs client IP address always "0.0.0.0" in such case. Fixes #412.
* | Use g_get_strerror() instead of strerror(errno) for portabilityPavel Roskin2016-11-081-2/+2
| |
* | Install headers used by X11rdp and xorgxrdpPavel Roskin2016-11-061-3/+5
|/ | | | | Installing the headers makes it possible to compile xorgxrdp as a separate package, without xrdp sources.
* Use static inline functions for OpenSSL 1.0 backportPavel Roskin2016-11-011-15/+26
| | | | | | | | Conditional preprocessor directives spread throughout the code set a bad example. The new backport code is located in one place. The compiler checks argument types. The backport code has no access to the caller variables. The main code has all advantages of the new, more compact API.
* Add backwards compatibility to OpenSSL < 1.1.0.Dominik George2016-10-271-0/+15
|
* Fix ssl_calls for OpenSSL 1.1.0, closes #458.Dominik George2016-10-271-31/+30
|
* Use const pointers in function arguments when possiblePavel Roskin2016-10-176-8/+8
|
* Remove text2bool() from log.h, there is no such functionPavel Roskin2016-10-171-10/+0
|
* Don't use final newline in log calls, it's already appendedPavel Roskin2016-10-171-4/+4
|
* Fix outgoing connections on Mac OSXPavel Roskin2016-09-081-1/+17
| | | | connect() on an already established connection returns error with errno set to EISCONN. Treat it as success.
* Add socklen_t check with substitution, use socklen_t as neededPavel Roskin2016-09-081-63/+12
|
* Improve debug information when closing a socketPavel Roskin2016-09-081-10/+76
| | | | | | | Don't assume AF_INET family. Don't assume the socket is connected. Report local address and port. Don't try to close non-sockets and invalid file descriptors. Report errors getting socket information and closing the socket. Use more appropriate log levels.
* always set SSL_OP_NO_SSLv2 in TLS optionsJay Sorg2016-08-251-4/+1
|
* new options for xrdp.ini disableSSlv3=yes and tls_ciphers=HIGH and code to ↵Alex Illsley2016-08-255-9/+31
| | | | implement
* Fix clipboard when text/filename contains non-ASCII charactersKoichiro IWAO2016-08-181-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | broken by #314. This is compatible with the fix introduced in #314. To use non-ASCII text/filename in clipboard, chansrv needs to be run with LC_CTYPE=*.UTF-8 because the behaviour of mbstowcs(3) function called in chansrv depends on LC_CTYPE[1]. However #314 made LC_CTYPE=C in chansrv context. Even if LANG and LC_* are set in .bashrc, /etc/profile, /etc/locale.conf or something like that, it doesn't affect in chansrv context because chansrv doesn't source any of them unlike sesman. So do not set LC_CTYPE to blank or "C" in g_init() in order to get g_mbstowcs and g_wcstombs to work properly with non-ASCII UTF-8 characters in any context. Setting LC_CTYPE to *.UTF-8 doesn't obstruct applying system language in RHEL [2]. [1] Linux man page says: The behavior of mbstowcs() depends on the LC_CTYPE category of the current locale. [2] https://bugzilla.redhat.com/show_bug.cgi?id=1290820
* Include stdint.h, don't redefine MAX/MIN constants if already definedJay Sorg2016-08-122-5/+17
|
* Pixman fixesJay Sorg2016-08-112-4/+2
|
* add region intersectJay Sorg2016-08-081-0/+3
|
* add more advances region support using pixmanJay Sorg2016-08-084-1/+2699
|
* Merge pull request #390 from proski/june21jsorg712016-08-059-53/+62
|\ | | | | Cleanups and C++ compatibility
| * Declare unified module interface and use it in modulesPavel Roskin2016-07-081-1/+10
| | | | | | | | | | This fixes loading modules compiled with a C++ compiler. Remote thandle type, it's unused. Use tintptr for module data. Don't cast pointers to long, they won't fit on Win64.
| * Use enum logLevels consistently for log levelsPavel Roskin2016-07-081-2/+2
| |
| * Use char* for TLS send and receivePavel Roskin2016-07-082-6/+6
| | | | | | | | This is consistent with ssl_tls_read() and ssl_tls_write(). C++ warnings are fixed without adding any casts.
| * Use correct types for ssl and ctx fields in struct ssl_tlsPavel Roskin2016-07-081-2/+2
| |
| * Fix signed to unsigned comparisons reported by g++ 6.1.0Pavel Roskin2016-07-081-1/+1
| |
| * Use g_new or g_new0 when C++ compiler would complain about implicit castPavel Roskin2016-07-082-2/+2
| |
| * Introduce glib style macros for allocating memory with typePavel Roskin2016-07-081-0/+7
| |
| * Fix warnings detected by -Wwrite-stringsPavel Roskin2016-07-084-11/+11
| |
| * Make program_name constant, don't duplicate or free itPavel Roskin2016-07-082-9/+3
| |
| * Fix format warnings in log_message() callsPavel Roskin2016-06-211-6/+4
| |
| * Check string format in log_messagePavel Roskin2016-06-213-12/+13
| | | | | | | | | | Move "printflike" definition to arch.h, it's used both by log.h and os_calls.h.
| * Fix incorrect use of "it's" and "its", adjust wordingPavel Roskin2016-06-211-1/+1
| |
* | sesman: env_set_user, fix potential bof issuesspeidy2016-07-222-10/+14
|/
* xrdp_wm: fix for multimon, draw login window and log window on primaryspeidy2016-05-171-0/+1
| | | monitor
* Merge pull request #370 from proski/typosItamar Reis Peixoto2016-05-062-6/+6
|\ | | | | Typo fixes and more
| * Use uniform comments for GUIDsPavel Roskin2016-05-051-5/+5
| |
| * Typo fixesPavel Roskin2016-05-041-1/+1
| |
* | Merge pull request #332 from metalefty/freebsd/ipv6jsorg712016-05-051-40/+47
|\ \ | |/ |/| common: add log for g_tcp_connect
| * common: no logic change, indent 2 -> 4Koichiro IWAO2016-02-231-39/+39
| |
| * common: no logic change, fix alignment, whitespacesKoichiro IWAO2016-02-231-2/+2
| |
| * common: add log for g_tcp_connectKoichiro IWAO2016-02-231-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | in case getaddrinfo(3) might fail. In FreeBSD, AI_V4MAPPED support for getaddrinfo(3) was very recently implemented[1]. Most of FreeBSD systems in the world do not have this implementation yet. This will be a problem when AI_V4MAPPED isn't supported and xrdp is built with IPv6 option. In such a case, g_tcp_connect always fails. Of course getaddrinfo(3) might fail in other cases. The log helps us to know what's happening. [1] https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=198092
* | Merge pull request #360 from proski/format-warningsjsorg712016-05-032-1/+5
|\ \ | | | | | | Warning fixes
| * | Fix more format warningsPavel Roskin2016-04-211-1/+1
| | |
| * | Include config_ac.h from common/os_calls.h to enable all format warningsPavel Roskin2016-04-211-0/+4
| | |
* | | trans: set non blocking socket after acceptJay Sorg2016-04-271-1/+1
| | |
* | | Add unnamed semaphore support on Mac OSPavel Roskin2016-04-231-0/+15
|/ / | | | | | | | | | | | | sem_init() is not functional on Mac OS. Use the Grand Central Dispatch implementation. Make libscp_lock.c use semaphores through the thread_calls wrapper.