Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | xrdp-sesadmin: fix error when there are no sessions | Ben Cohen | 2017-07-06 | 1 | -4/+11 |
| | | | | | | | | | | | | | | | | | | Test case: On a system running xrdp with no sessions running run: xrdp-sesadmin -u=<user> -p=<password> -c=list Expected result: "No sessions." (ignoring debug output) Observed result: "Error getting session list." In the SCP_SERVER_STATE_MNG_LISTREQ case in scp_v1_mng_process() if there are no sessions it ends the scp session, which causes an error in the client. In commit 0017081d the client was changed to report errors, giving the result above. Fix by calling scp_v1s_mng_list_sessions() from scp_v1_mng_process() even when there are no sessions, and if so sending a packet with a count of zero so that the client gets what it expects. | ||||
* | Eliminate APP_CC and DEFAULT_CC | Pavel Roskin | 2017-03-14 | 6 | -18/+18 |
| | |||||
* | Don't include config_ac.h from any header files | Pavel Roskin | 2017-03-04 | 1 | -4/+0 |
| | |||||
* | Include config_ac.h from all source files | Pavel Roskin | 2017-03-04 | 11 | -0/+44 |
| | |||||
* | Remove redundant function declarations | Pavel Roskin | 2017-01-11 | 2 | -11/+0 |
| | |||||
* | Use "void" for empty argument list in declarations | Pavel Roskin | 2017-01-05 | 4 | -4/+4 |
| | | | | | | In C, an empty argument list in a declaration means that the function can accept any arguments. Use "void" instead, it means "no arguments". C++ treats void and empty list as "no arguments". | ||||
* | fix build with --enable-xrdpdebug=yes | volth | 2017-01-04 | 1 | -2/+0 |
| | |||||
* | fix build with --enable-xrdpdebug=yes | volth | 2017-01-04 | 1 | -0/+6 |
| | |||||
* | vnc: change password file to use guid hash | Jay Sorg | 2016-12-03 | 5 | -3/+29 |
| | |||||
* | Fix memory leak: free session data | Pavel Roskin | 2016-10-21 | 1 | -2/+0 |
| | | | | | | | Call scp_session_destroy() in the functions that call scp_session_create() and nowhere else. As found by Valgrind, the session data is not freed if the session is created successfully. | ||||
* | Log malloc errors with LOG_LEVEL_ERROR | Pavel Roskin | 2016-10-17 | 2 | -2/+2 |
| | |||||
* | Use g_new or g_new0 when C++ compiler would complain about implicit cast | Pavel Roskin | 2016-07-08 | 3 | -6/+6 |
| | |||||
* | Fix warnings detected by -Wwrite-strings | Pavel Roskin | 2016-07-08 | 6 | -26/+28 |
| | |||||
* | Remove unused string support in scp_session_set_addr() | Pavel Roskin | 2016-07-08 | 5 | -44/+6 |
| | | | | | | | | Using the same argument for binary and text data is not a good idea. If string support is ever needed, it should be a separate function. Remove SCP_ADDRESS_TYPE_IPV4_BIN and SCP_ADDRESS_TYPE_IPV6_BIN, use SCP_ADDRESS_TYPE_IPV4 and SCP_ADDRESS_TYPE_IPV6, which simplifies the code. | ||||
* | Downgrade "libscp initialized" to LOG_LEVEL_DEBUG, remove line number | Pavel Roskin | 2016-07-08 | 1 | -1/+1 |
| | | | | It's a bad style to start the log with a cryptic warning. | ||||
* | Typo fixes | Pavel Roskin | 2016-05-04 | 3 | -3/+3 |
| | |||||
* | Add unnamed semaphore support on Mac OS | Pavel Roskin | 2016-04-23 | 1 | -11/+11 |
| | | | | | | | 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. | ||||
* | fix a warning | Jay Sorg | 2016-03-16 | 1 | -1/+1 |
| | |||||
* | Move headers from EXTRA_DIST to sources, sort alphabetically | Pavel Roskin | 2016-02-21 | 1 | -5/+18 |
| | | | | There should be no functional difference. | ||||
* | Fix typos | Pavel Roskin | 2016-02-13 | 6 | -7/+7 |
| | |||||
* | Stop redefining libdir, use moduledir for ${libdir}/xrdp | Pavel Roskin | 2016-02-12 | 1 | -1/+1 |
| | | | | | | | libdir is for shared and static libraries. Some xrdp libraries are loaded dynamically from a non-standard directory, so they are used like modules. Having separate libdir and moduledir would eventually allow to separate dynamically loaded modules from shared libraries. | ||||
* | sesman: add missing break | Jay Sorg | 2016-02-11 | 1 | -1/+1 |
| | |||||
* | remove duplicated call to scp_session_set_bpp | Itamar Reis Peixoto | 2016-02-11 | 1 | -1/+0 |
| | |||||
* | Allow sessions with 32 bpp | Pavel Roskin | 2016-02-10 | 1 | -0/+1 |
| | |||||
* | Check return value of scp_session_set_bpp() | Pavel Roskin | 2016-02-10 | 2 | -1/+16 |
| | |||||
* | Change an unconditional #warning to a comment | Pavel Roskin | 2016-01-31 | 1 | -2/+5 |
| | | | | | Notes about software design should not be shown as warnings to everybody who compiles the code. Warnings should be about unexpected conditions detected at the compile time. | ||||
* | More spelling fixes found by codespell and aspell | Pavel Roskin | 2016-01-30 | 2 | -3/+3 |
| | |||||
* | Merge AM_CFLAGS and INCLUDES info AM_CPPFLAGS | Pavel Roskin | 2016-01-29 | 1 | -4/+2 |
| | | | | | | | | | AM_CPPFLAGS is for flags passed to the preprocessor, such as defines and includes. AM_CFLAGS is for flags affecting the compiler, such as debug and optimization settings. INCLUDES is an obsolete name. Users can pass INCLUDES and break compilation. AM_CPPFLAGS is more explicit that the flags come from Automake and should not be overridden. | ||||
* | coverity: Resource leak (2nd round) | Laxmikant Rashinkar | 2014-07-27 | 2 | -3/+0 |
| | |||||
* | coverity: dead code | Laxmikant Rashinkar | 2014-07-26 | 1 | -6/+0 |
| | |||||
* | coverity: fixed resolution leak issues | Laxmikant Rashinkar | 2014-07-19 | 2 | -0/+3 |
| | |||||
* | sesman: added SessionVariables to sesman.ini and some indent cleanup | Jay Sorg | 2014-05-09 | 2 | -14/+14 |
| | |||||
* | added support for using Xorg driver model | Laxmikant Rashinkar | 2014-03-08 | 3 | -2/+26 |
| | |||||
* | ipv6 fix for sesman | Jay Sorg | 2013-04-03 | 1 | -13/+1 |
| | |||||
* | o moved from GNU General Public License to Apache License, Version 2.0 | Laxmikant Rashinkar | 2012-09-19 | 27 | -2691/+2834 |
| | | | | | o applied new coding standards to all .c files o moved some files around | ||||
* | autotools: change top_srcdir to top_builddir for LIBADD | Jay Sorg | 2012-07-13 | 1 | -2/+2 |
| | |||||
* | sesman: missing else and remove some EOL whitespace | Jay Sorg | 2012-06-23 | 1 | -6/+6 |
| | |||||
* | fix merge conflics | norrarvid | 2012-06-18 | 4 | -2/+77 |
|\ | |||||
| * | Improved logging | norrarvid | 2012-06-07 | 1 | -2/+2 |
| | | |||||
| * | timeout added for the reply to come from sesman | norrarvid | 2012-06-05 | 9 | -123/+125 |
| |\ | |||||
| * | | separate pam authentication for gateways | norrarvid | 2012-05-31 | 4 | -2/+77 |
| | | | |||||
* | | | no logic changes, remove warnings and coding style changes | Jay Sorg | 2012-06-10 | 1 | -1/+1 |
| |/ |/| | |||||
* | | Initial syslog support in XRDP | norrarvid | 2012-05-27 | 9 | -123/+125 |
|/ | |||||
* | autotool fixes | Jay Sorg | 2011-07-05 | 1 | -1/+2 |
| | |||||
* | include missing files in make dist | Itamar Reis Peixoto | 2010-11-20 | 1 | -0/+2 |
| | |||||
* | Implemented client IP logging | Nicola Ruggero | 2010-11-04 | 4 | -0/+38 |
| | |||||
* | Major code cleanup: | Nicola Ruggero | 2010-11-03 | 1 | -2/+2 |
| | | | | | | | - Initialized and zeroed out local variables - Check for some null pointers - Fixed some typos - Other minor changes (beautify, etc.) | ||||
* | use do ... while (0) style for parse.h | jsorg71 | 2010-08-09 | 4 | -5/+5 |
| | |||||
* | update copyright year | jsorg71 | 2010-07-04 | 27 | -27/+27 |
| | |||||
* | remove some dead code | jsorg71 | 2009-12-26 | 1 | -6/+0 |
| |