669 Commits (a8fcbf8c2dc48f0e015093e061c5b7efb63fbdc1)

Author	SHA1	Message	Date
Koichiro IWAO	35bc7c1d65	sesman: unbreak FreeBSD build after #523	8 years ago
metalefty	731d4be670	Merge pull request #521 from proski/xorg-log ... Change default Xorg logfile to .xorgxrdp.%s.log	8 years ago
Pavel Roskin	9cbf3d1bb8	Change default Xorg logfile to .xorgxrdp.%s.log ... The log file is created in the home directory, %s is replaced with the display number (e.g. 10). If Xorg is run setuid root, it can refuse to run if the log path is absolute.	8 years ago
Pavel Roskin	04414e0fd4	Disable privilege elevation by setuid for the X server	8 years ago
Jay Sorg	f14e3dc8ac	sesman: must save guid in session data	8 years ago
Jay Sorg	9a517b34f0	vnc: code cleanup	8 years ago
Jay Sorg	57905d71ad	vnc: change password file to use guid hash	8 years ago
Pavel Roskin	e46d15ca84	Fix C++ errors and warnings in FUSE code	8 years ago
Pavel Roskin	2ab321a7c6	Use fuse_ino_t for FUSE inodes, not tui32 or int	8 years ago
Pavel Roskin	f622a17c75	Remove write-only FileId variable	8 years ago
Pavel Roskin	46f4025372	Cast pointer to uint64_t through tintptr to avoid warnings ... Cast to long would damage a pointer on Win64.	8 years ago
Pavel Roskin	e92c4f3b40	Use constant strings to fix warnings	8 years ago
Pavel Roskin	662172012b	Call devredir_cvt_slash() on the path copy, it changes the argument	8 years ago
Pavel Roskin	a4e3952d65	Disable or remove dead code to prevent it from throwing warnings	8 years ago
Pavel Roskin	a547cc3784	Include needed files in chansrv_fuse.c, remove duplicate declarations	8 years ago
Pavel Roskin	bf39fed6e4	Fix misuse of NULL for arithmetic type	8 years ago
Pavel Roskin	4234ed278e	Use FUSE_CFLAGS and FUSE_LIBS from pkg-config ... Include <fuse_lowlevel.h>, not <fuse/fuse_lowlevel.h>, the include path should be set up for that. Don't define _FILE_OFFSET_BITS in one file, FUSE_CFLAGS would enable it for the whole xrdp-chansrv build if needed.	8 years ago
Pavel Roskin	15c4c47811	Remove "-ac" from X server parameters, it's insecure and unnecessary ... Fedora patches sesman.ini to remove "-ac", and that doesn't cause any ill effects.	8 years ago
Pavel Roskin	7fce469cbc	Fix format warnings if FUSE is enabled	8 years ago
Jay Sorg	25fd585d55	chansrv: some fixes to log file path	8 years ago
Itamar Reis Peixoto	f7b0b0d89c	Merge pull request #488 from metalefty/docs ... Update man pages and config files	8 years ago
metalefty	9deaa078fc	Merge pull request #477 from proski/vnc-strerror ... Improve error messages about VNC password file	8 years ago
metalefty	8516e6f97b	Merge pull request #476 from proski/auth ... Fix compile warnings for non-default authentication methods	8 years ago
metalefty	b89956d06e	Merge pull request #485 from proski/code-standards ... Code standards	8 years ago
metalefty	d335c93a75	Merge pull request #480 from proski/strncat ... Fix potential buffer overflow in strncat() invocation	8 years ago
Pavel Roskin	35b336272b	Fix compilation with C++	8 years ago
Pavel Roskin	16c3bbdb8d	Remove all tabs in sources	8 years ago
Pavel Roskin	e9d064e390	Don't try to create .vnc directory if it exists ... Report system error if it cannot be created.	8 years ago
Pavel Roskin	60029514a6	Report system error if the VNC password file cannot be created	8 years ago
Pavel Roskin	e9a0324cb3	Fix compile warnings for non-default authentication methods ... Include string.h for strcmp(). Don't define _XOPEN_SOURCE, it conflicts with libc headers and hides putpwent(). Make input strings constant. Fix functions to match their declarations.	8 years ago
Pavel Roskin	92423a466e	Fix potential buffer overflow in strncat() invocation ... strncat() will copy at most the specified number of characters and append the null character on top of that. strlen() doesn't count the final null character.	8 years ago
Koichiro IWAO	cb1960e0fc	config: use semicolon to comment out descriptions in config files ... and use number sign to comment out actual configurations. For example: ; if set to true, enables foobar #foobar=true	8 years ago
Koichiro IWAO	4f2d94505a	config: unify all boolean values in config to true/false	8 years ago
Pavel Roskin	0d72cced46	Omit numbers from the "param" keys, they are not needed ... Users assume that they need to renumber the parameters. That makes parameter editing more involved than it needs to be.	8 years ago
metalefty	94901c9989	Merge pull request #454 from metalefty/change-chansrv-log-path ... chansrv: change chansrv log path to ${XDG_DATA_HOME}/xrdp	8 years ago
Dominik George	c90bc13d39	Move -logfile /dev/null option for easier commenting in debug scenarios.	8 years ago
Koichiro IWAO	7d017482f6	chansrv: change chansrv log path to ${XDG_DATA_HOME}/xrdp ... like Xorg's logfile is written to ${XDG_DATA_HOME}/xorg/Xorg.n.log. If XDG_DATA_HOME is not defined, the log path will be ${HOME}/.local/share/xrdp.	8 years ago
Pavel Roskin	b28a986071	Fix memory leak in xrdp-sesman on config reload	8 years ago
Pavel Roskin	dc60a80b86	Memory leak fix: keep X server path in the parameter list ... g_cfg->xorg_params, g_cfg->vnc_params and g_cfg->rdp_params don't have auto_free enabled, so removing an item from one of those lists won't free its contents. It's better not to change those lists, as they represent the actual config file and could be reused. Instead, omit the 0th parameter (the executable path) from copying to xserver_params. Found by Valgrind.	8 years ago
Pavel Roskin	e17a56efb6	Call auth_end() exactly once in scp_v0_process() ... As discovered by Valgrind, it wasn't called at all in case of a successful session creation, which leaked memory.	8 years ago
Pavel Roskin	65ac8e758b	Fix memory leak: free session data ... 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.	8 years ago
Pavel Roskin	a370306f75	Get rid of EXTRA_LIBS, use variables with more specific names	8 years ago
Pavel Roskin	5c69f3cd57	Eliminate EXTRA_FLAGS, it's a poor name, use AM_LDFLAGS	8 years ago
Pavel Roskin	0422734fc7	Merge EXTRA_DEFINES and EXTRA_INCLUDES into AM_CPPFLAGS ... AM_CPPFLAGS is a documented Automake variable for C preprocessor flags that should not be overridden when compiling the package. There is no need to have two additional variables that are ultimately merged into AM_CPPFLAGS. Their names are also confusing. EXTDA_DIST is a documented Automake variable. Everything else that starts with "EXTRA" is not.	8 years ago
Pavel Roskin	6fef1e4eb5	Use const pointers in function arguments when possible	8 years ago
Pavel Roskin	c3b99de739	Fix wrong comments in sesman config printing code	8 years ago
Pavel Roskin	bf08892622	Fix incorrect description of auth_userpass() return value	8 years ago
Pavel Roskin	a618d4f757	Don't use final newline in log calls, it's already appended	8 years ago
Pavel Roskin	424cef464b	Log malloc errors with LOG_LEVEL_ERROR	8 years ago
Pavel Roskin	945833a94f	Don't close socket in scp_process_start(), the caller closes it	8 years ago