Commit Graph

4183 Commits

Author SHA1 Message Date
matt335672
6500949a18 Fix message ordering in devredir
The drive redirector is not compliant with the message ordering
in [MS-RDPEFS], causing FreeRDP 2.11.2 ro fail on redirecting drives.
2023-10-26 15:19:09 +01:00
matt335672
b782340456
Merge pull request #2830 from matt335672/v0_9_cppcheck-inc
[V0.9] updated Cppcheck to 2.12.1 / provide more includes to Cppcheck
2023-10-18 10:42:15 +01:00
firewave
2c0749d2cb bumped Cppcheck to 2.12.1
- build Cppcheck with matchcompiler for improved performance
- build Cppcheck with Boost for improved ValueFlow performance
2023-10-18 10:14:41 +01:00
firewave
90c7e72656 provide more includes to Cppcheck
- added (temporary) suppression of Cppcheck `shiftTooManyBits` false positives in `libxrdp/xrdp_mppc_enc.c`
- fix Cppcheck `nullPointerRedundantCheck` in `sesman/chansrv/clipboard.c`
2023-10-18 10:14:06 +01:00
matt335672
66c668c44b
Merge pull request #2826 from metalefty/v0.9-no-c99
Cherry-pick downstream patch from EPEL
2023-10-12 10:19:40 +01:00
Koichiro Iwao
37f206d2c8 Cherry-pick downstream patch from EPEL
Same as #2715

Obtained from: https://src.fedoraproject.org/rpms/xrdp/blob/014fe42/f/xrdp-0.9.22.1-no-c99.patch
2023-10-12 14:27:39 +09:00
matt335672
4e713c884f
Merge pull request #2824 from matt335672/v0_9_remove_ss_clipboard_code
[v0.9] remove ss clipboard code
2023-10-11 11:18:50 +01:00
matt335672
f6fa3fc5ba Remove client-server shortcut paste code
When significant amounts of data is coming from the client in a
fragmented CLIPRDR_DATA_RESPONSE PDU, this code provides a way to
start copying it to a requesting client before it is all read.

The only advantage of this code is to provide a slight speedup
before a paste is visible on the server.

There are significant problems with this code. Notably, it is
very difficult to parse Unicode text coming through this route. Each
UTF-16 character can occupy up to 4 bytes, and a fragmentation
boundary could occur at any point within a UTF-16 character.
2023-10-11 10:41:34 +01:00
matt335672
7603c5791b
Merge pull request #2817 from matt335672/v0_9_groups_patch
[V0.9] groups patch
2023-10-06 11:11:32 +01:00
matt335672
e6bc4db423 access_login_allowed: Remove primary group check
This check is now performed within g_check_user_in_group()
2023-10-06 10:56:14 +01:00
matt335672
8323e80668 Add getgrouplist() support to os_calls
On enterprise systems, using getgrouplist() (if available)
is more efficient than iterating over the members of the group,
and is also more likely to work
2023-10-06 10:52:08 +01:00
matt335672
1658d71ee3 Add getgrouplist() detection functionality
Defines the macro HAVE_GETGROUPLIST if getgrouplist() is
available, and defines the type passed to the GID array as
GETGROUPS_T
2023-10-06 10:51:43 +01:00
metalefty
1b49f82e39
Merge pull request #2808 from metalefty/v0.9-release
Release v0.9.23.1
2023-09-28 02:55:42 +09:00
Koichiro Iwao
82dbfa338a Bump version to v0.9.23.1 2023-09-27 21:12:15 +09:00
Koichiro Iwao
2b9b84fb81 Update NEWS for v0.9.23.1 2023-09-27 21:11:14 +09:00
metalefty
fd25fc546a
Merge pull request from GHSA-2hjx-rm4f-r9hw
[V0.9] CVE-2023-42822
2023-09-27 17:37:24 +09:00
matt335672
58c9c1f06a CVE-2023-42822
- font_items in struct xrdp_font renamed to chars to catch all
  accesses to it. This name is consistent with the type of
  the array elements (struct xrdp_font_char).
- Additional fields added to struct xrdp_font to allow for range
  checking and for a default character to be provided
- Additional checks and logic added to xrdp_font_create()
- New macro XRDP_FONT_GET_CHAR() added to perform checked access
  to chars field in struct xrdp_font
2023-09-25 14:02:30 +01:00
matt335672
4db9323413
Merge pull request #2784 from matt335672/v0_9_z3
Cppcheck 2.8 removed the dependency on z3
2023-09-04 10:33:36 +01:00
firewave
8f311633e9 Cppcheck 2.8 removed the dependency on z3 2023-09-04 10:15:25 +01:00
metalefty
4c3dd94dde
Merge pull request #2779 from metalefty/v0.9-release
Release v0.9.23
2023-08-31 11:26:29 +09:00
Koichiro Iwao
6c068fbb95 Bump version to v0.9.23 2023-08-31 10:45:26 +09:00
Koichiro Iwao
7b08610742 Update NEWS for v0.9.23 2023-08-31 10:45:22 +09:00
metalefty
8c5b7cdff3
Merge pull request from GHSA-f489-557v-47jq
[v0.9] Check auth_start_session() result
2023-08-24 10:14:48 +09:00
matt335672
322d11b431 [v0.9] Check auth_start_session() result 2023-08-19 13:26:44 +01:00
matt335672
469a89cf53
Merge pull request #2767 from matt335672/v0_9_clipboard_updates
V0 9 clipboard updates
2023-08-09 14:39:54 +01:00
matt335672
722fe7d74f clipboard: Fix TODO action in clipboard_common.h
Use the official Windows clipboard format names where appropriate

Replace g_file_format_id with g_file_group_descriptor_format_id
as the latter name is more descriptive of what is described in
[MS-ECLIP]
2023-08-08 09:46:20 +01:00
matt335672
15f730524e clipboard: Only advertise text to X11 clients if it is available 2023-08-08 09:46:13 +01:00
matt335672
680bba8a02 clipboard: Tell the X11 client if a selection is unavailable 2023-08-08 09:46:05 +01:00
matt335672
011009681e
Merge pull request #2738 from matt335672/v0_9_bump_cppcheck_to_2_11
[V0.9] bump cppcheck to 2 11
2023-06-23 17:17:16 +01:00
matt335672
946f86f611 Use all available CPUs to build cppcheck 2023-06-23 17:05:43 +01:00
matt335672
7e514bf339 Bump cppcheck to v2.11
This fixes the following errors:-

sesman/tools/authtest.c:64:14: error: syntax error [syntaxError]
    g_printf("xrdp auth module tester v" PACKAGE_VERSION "\n");
             ^
sesman/tools/sesrun.c:165:14: error: syntax error [syntaxError]
    g_printf("xrdp session starter v" PACKAGE_VERSION "\n");
             ^
vrplayer/decoder.h:35:12: error: There is an unknown macro here somewhere. Configuration is required. If slots is a macro then please configure it. [unknownMacro]
    public slots:
           ^
vrplayer/playaudio.h:45:12: error: There is an unknown macro here somewhere. Configuration is required. If slots is a macro then please configure it. [unknownMacro]
    public slots:
           ^
vrplayer/dlgabout.h:22:13: error: There is an unknown macro here somewhere. Configuration is required. If slots is a macro then please configure it. [unknownMacro]
    private slots:
            ^
vrplayer/playvideo.h:49:12: error: There is an unknown macro here somewhere. Configuration is required. If slots is a macro then please configure it. [unknownMacro]
    public slots:
           ^
Additionally, cppcheck now makes use of all available CPUs
2023-06-23 17:05:21 +01:00
matt335672
27ea401e43
Merge pull request #2712 from matt335672/v0_9_allow_long_strings_in_pam_env
[0.9] Remove size limit for PAM environment variables
2023-06-01 15:34:17 +01:00
matt335672
3d2b3b89f4 Remove size limit for PAM environment variables
The current logic in auth_set_env() for PAM environments only allows
environment variables to be around 256 characters in length.
2023-05-31 16:30:58 +01:00
metalefty
f1fa6bbc19
Merge pull request #2703 from metalefty/v0.9-tarball-script
Add script to make release tarball
2023-05-26 23:50:19 +09:00
Koichiro IWAO
6954ab0972 Fix usage comment 2023-05-26 23:32:02 +09:00
Koichiro IWAO
7efcd9280c Reduce prompt timeout 2023-05-26 23:27:20 +09:00
Koichiro IWAO
b3dba2bad4 Add script to make release tarball
to avoid mistake in release procedure.
2023-05-26 18:39:46 +09:00
metalefty
b26ca93176
Merge pull request #2694 from metalefty/v0.9-release
Update NEWS and bump version to v0.9.22.1
2023-05-22 23:41:56 +09:00
Koichiro IWAO
d2a55d9203
Update NEWS and bump version to v0.9.22.1 2023-05-22 23:11:48 +09:00
metalefty
2f214c8452
Merge pull request #2658 from metalefty/v0.9-release
v0.9 release
2023-05-07 15:14:46 +09:00
Koichiro IWAO
71594b5114
Bump version for v0.9.22 2023-05-07 15:03:13 +09:00
Koichiro IWAO
1b3be3e641 Update NEWS for v0.9.22 2023-05-07 15:03:05 +09:00
matt335672
6c9f7f8434
Merge pull request #2653 from matt335672/v0_9_allow_longer_user_wm_str
[V0.9] Allow longer UserWindowManager strings
2023-05-02 12:10:40 +01:00
matt335672
a58c895d1b Allow longer UserWindowManager strings
The UserWindowManager is limited to 31 characters. There appears
to be no good reason for this.
2023-05-02 12:06:30 +01:00
matt335672
929d072622
Merge pull request #2650 from matt335672/v0_9_unnecessary_check_h
[V0.9] Remove unnecessary check h
2023-04-28 10:41:46 +01:00
Jat
4bc8701631 remove unnecessary include check.h 2023-04-28 10:23:26 +01:00
matt335672
d9580dee01
Merge pull request #2634 from lennartS/v0.9_shell
sesman: Prevent the use of 'alternate shell'
2023-04-19 09:58:15 +01:00
Lennart Sauerbeck
d4ea314ac2 sesman: Prevent the use of 'alternate shell'
By setting the new config value 'AllowAlternateShell' to 'no' it is now
possible to prevent the use of an alternate shell, which can be set by
the connecting user.
The default remains unchanged and any shell is allowed if the config
value is not specified. It can also be set explicitly to 'yes' to achieve
the same outcome.

Fixes: #850
2023-04-17 22:34:19 +02:00
matt335672
d7c3809ce8
Merge pull request #2622 from matt335672/v0_9_cirrus_freebsd_update
[V0.9] Bump FreeBSD image to 12.4
2023-03-31 12:11:27 +01:00
matt335672
a046aeea3f Bump FreeBSD image to 12.4 2023-03-31 12:03:14 +01:00