summaryrefslogtreecommitdiffstats
path: root/sesman
Commit message (Collapse)AuthorAgeFilesLines
* Fix incorrect use of "it's" and "its", adjust wordingPavel Roskin2016-06-211-1/+1
|
* Remove unused variable in devredir_cvt_from_unicode_len()Pavel Roskin2016-05-061-2/+1
|
* Remove unused variables from dev_redir_proc_query_dir_response()Pavel Roskin2016-05-061-15/+5
|
* Check XGetWindowProperty() return code in clipboard_event_property_notify()Pavel Roskin2016-05-061-0/+10
|
* Use better symbols to protect include filesPavel Roskin2016-05-052-4/+4
|
* Typo fixesPavel Roskin2016-05-045-5/+5
|
* Merge pull request #360 from proski/format-warningsjsorg712016-05-0311-55/+59
|\ | | | | Warning fixes
| * Fix warnings for unused variables read from byte streamsPavel Roskin2016-04-233-8/+6
| |
| * Fix recently introduced warnings in session.cPavel Roskin2016-04-211-10/+10
| | | | | | Use of tabs is fixed as well.
| * Fix warnings about unused variables and functionsPavel Roskin2016-04-214-2/+8
| |
| * Fix more format warningsPavel Roskin2016-04-214-35/+35
| |
* | Add unnamed semaphore support on Mac OSPavel Roskin2016-04-231-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.
* Merge pull request #359 from metalefty/sesman/configurable-xserver-pathjsorg712016-04-183-11/+25
|\ | | | | sesman: do not hardcode Xorg/X11rdp/Xvnc executables
| * sesman: do not hardcode Xorg/X11rdp/Xvnc executablesKoichiro IWAO2016-04-072-7/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In RHEL and its clones, /usr/bin/Xorg is set suid-root. To execute Xorg with user privileges, /etc/pam.d/xserver needs be edited [1], or suid bit of Xorg binary needs to be dropped. In order to keep Xorg and /etc/pam.d/xserver untouched, preparing non-suid version of Xorg as /usr/bin/Xorg.non-suid for example is the simplest solution. However, Xorg.non-suid cannot be executed since it is hardcoded to execute Xorg in sesman. This change makes more flexible to execute Xorg with non-standard name or not in PATH environment variable. [1] https://www.centos.org/forums/viewtopic.php?t=21185
| * sesman: brief xorg_params, no logic changeKoichiro IWAO2016-04-071-4/+2
| |
* | Do not attempt to trap SIGKILL part 2Koichiro IWAO2016-04-111-6/+0
|/ | | | it doesn't work on any OS.
* Fix compile error "format not a string literal and no format arguments ↵NerdKnight2016-03-281-1/+1
| | | | [-Werror=format-security]"
* fix a warningJay Sorg2016-03-161-1/+1
|
* fix warnings, move some calls to os_callsJay Sorg2016-03-162-10/+12
|
* Fix all format warnings and some unused variable warningsJay Sorg2016-03-143-4/+3
|
* Don't attempt to intercept SIGKILL, it doesn't work on any OSJay Sorg2016-03-143-3/+0
|
* Merge pull request #333 from proski/key-installItamar Reis Peixoto2016-02-241-9/+4
|\ | | | | Key install (generate rsakeys.ini during make install)
| * Use SCRIPTS for executable scripts, don't use "chmod 755"Pavel Roskin2016-02-221-5/+3
| |
| * Use "dist_" prefix to minimize use of EXTRA_DISTPavel Roskin2016-02-221-4/+1
| | | | | | "dist" is ignored in presense of "noinst", so keep noinst_man_MANS.
* | Merge pull request #326 from metalefty/pulseaudio-6.0jsorg712016-02-232-1/+10
|\ \ | |/ |/| pulse: fix build pulseaudio 6.0 or higher
| * pulse: fix build pulseaudio 6.0 or higherKoichiro IWAO2016-02-182-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | Discovered in #321. The number of argument for pa_rtpoll_run have been changed since 6.0. >=6.0 : int pa_rtpoll_run(pa_rtpoll *f); <6.0 : int pa_rtpoll_run(pa_rtpoll *f, bool wait); Check pulseaudio version by PA_CHECK_VERSION macro introduced since pulseaudio 0.9.16. In case PA_CHECK_VERSION is not defined, pa_rtpoll_run takes 2 arguments.
* | Move headers from EXTRA_DIST to sources, sort alphabeticallyPavel Roskin2016-02-214-38/+58
|/ | | | There should be no functional difference.
* xrdp-sesadmin: show help and exit if mandatory arguments are not givenPavel Roskin2016-02-141-2/+9
| | | | Don't mark password as mandatory, it's asked if not provided on the command line.
* Fix typosPavel Roskin2016-02-1323-43/+43
|
* Fix connection month display in xrdp-sesadminPavel Roskin2016-02-131-1/+1
| | | | | | | tm_mon in `struct tm` uses 0 for January. `struct session_data` and `struct SCP_DISCONNECTED_SESSION` don't specify how dates are stored. But considering the code in sestest and sesadmin, all date components are stored as displayed.
* Use lowercase program names for syslogPavel Roskin2016-02-133-3/+3
| | | Other programs typically use the same case as their execulables.
* Stop redefining libdir, use moduledir for ${libdir}/xrdpPavel Roskin2016-02-121-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.
* Don't ignore files known to gitPavel Roskin2016-02-122-0/+2
| | | | | | | | | | | | | That can cause unexpected behavior, especially with third party tools. The ignored files will be lost if the source tree is re-imported to another git repository, unless special care is taken. Whitelist all non-generated makefiles. To whitelist instfiles/pam.d/xrdp-sesman, add path to all generated executables listed in the top-level .gitignore, sort .gitignore alphabetically. Add mkinstalldir, it's used by Automake on some systems.
* Add missing spaces in the strings that are split for line wrappingPavel Roskin2016-02-123-3/+3
|
* xcon: include unistd.h, it's needed for select() on MacOSPavel Roskin2016-02-121-0/+1
|
* startwm.sh: make sure sourced scripts are readablePavel Roskin2016-02-121-29/+26
| | | | | | | | | Strictly speaking, both existence and readability should be checked. However, chances of ~/.profile being a directory or a FIFO are very small compared to its chances of being non-readable due to some misconfiguration. Put "if" and "then" on the same line for consistency with other checks. Improve comment and pseudocode formatting.
* sesman: add missing breakJay Sorg2016-02-111-1/+1
|
* remove duplicated call to scp_session_set_bppItamar Reis Peixoto2016-02-111-1/+0
|
* Allow sessions with 32 bppPavel Roskin2016-02-101-0/+1
|
* Check return value of scp_session_set_bpp()Pavel Roskin2016-02-102-1/+16
|
* Merge pull request #309 from proski/develItamar Reis Peixoto2016-02-0813-30/+34
|\ | | | | Fixes for autotools, compile warnings and MacOS compatibility
| * Change an unconditional #warning to a commentPavel Roskin2016-01-311-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.
| * xcon: include sys/types.h for select() and related macrosPavel Roskin2016-01-311-0/+1
| |
| * rail: fix declaration of rail_desktop_resize()Pavel Roskin2016-01-311-1/+1
| | | | | | | | Arguments without a type default to int. The caller passes a pointer to XEvent, so use that type.
| * clipboard: undefine previously defined log levelPavel Roskin2016-01-301-0/+2
| |
| * More spelling fixes found by codespell and aspellPavel Roskin2016-01-302-3/+3
| |
| * Use standard autotools means to detect X11Pavel Roskin2016-01-302-4/+10
| | | | | | | | | | | | | | | | Use AC_PATH_XTRA to search for X11 in configure.ac. In Makefiles, add X_CFLAGS to AM_CFLAGS for the source compilation. Add X_LIBS to LDFLAGS. Add X_PRE_LIBS and X_EXTRA_LIBS to LDADD. With this patch, X Windows system is correctly detected on Mac OS X.
| * clipboard_file: include sys/time.h for struct timeval, needed on Mac OS XPavel Roskin2016-01-301-0/+1
| |
| * Merge AM_CFLAGS and INCLUDES info AM_CPPFLAGSPavel Roskin2016-01-295-20/+10
| | | | | | | | | | | | | | | | | | 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.
| * Don't use a symlink where #include would sufficePavel Roskin2016-01-261-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | Absolute symlinks break when the source is moved to another location. In the best case, the symlink will be replaced with a copy, but that creates code duplication. A user who got the packaged source may decide to change one config.c without realizing that another config.c will get the same changes. Don't ignore config.c, it's a regular source file now, albeit very short. Testing done: compiled inside and outside build directory.