Commit Graph

23068 Commits

Author SHA1 Message Date
efea17a5fe [client,SDL] Handle window sizing on scaled displays 2026-03-04 17:48:09 +09:00
akallabeth
0a7fd3811c Merge pull request #12415 from akallabeth/compile-issues
Address some GCC compile issues
2026-03-03 17:49:58 +01:00
Armin Novak
4b4368fc19 [compiler,gcc] fix function pointer casts
use dedicated macro WINPR_FUNC_PTR_CAST to cast function pointers
without warnings.
2026-03-03 17:24:10 +01:00
Armin Novak
782c3d3069 [client,common] check _fseeki64 return
propagate error conditions back the call chain
2026-03-03 17:15:12 +01:00
Armin Novak
0d99896094 [core,return] improve return value checks
propagate error conditions back the call chain
2026-03-03 17:15:12 +01:00
Armin Novak
1a8e72074e [winpr,return] improve return value checks
propagate error conditions up the call chain
2026-03-03 17:15:11 +01:00
Armin Novak
45494b02a3 [core,server] FreeRDP_WTSVirtualChannelRead checks
Properly check MessageQueue_Peek return and propagate failure.
2026-03-03 17:15:11 +01:00
Armin Novak
7485c01a8c [freerdp,API] fix broken deprecation declarations 2026-03-03 17:15:10 +01:00
akallabeth
63df758f73 Merge pull request #12414 from akallabeth/wts-fix
[core,server] Improve WTS API locking
2026-03-03 16:44:52 +01:00
akallabeth
72be7a4250 Merge pull request #12413 from akallabeth/return-fixes
Return check fixes
2026-03-03 16:34:05 +01:00
Armin Novak
d079f59e55 [core,server] Improve WTS API locking
* Use InitOnceExecuteOnce to initialize global hash table for server
  handles
* Lock full function calls when manipulating global data (Hash table and
  session ID count)
2026-03-03 16:31:18 +01:00
Armin Novak
81526b19a2 [string,format] fix format strings for debug messages 2026-03-03 14:07:49 +01:00
Armin Novak
a9e0abf2ea [core,orders] improve input validation
check length before subtracting. Might underflow and be cought by the
next check, but lets be strict.
2026-03-03 13:58:52 +01:00
Armin Novak
125b1ce99e [utils,ringbuffer] fix debug log messages
Renaming some variables also in debug messages
2026-03-03 13:49:59 +01:00
Armin Novak
2b8ecce7a9 [channels,rdpecam] fix return checks 2026-03-03 13:49:59 +01:00
Armin Novak
1fb02134e6 [channels,drive] fix missing return checks 2026-03-03 13:49:58 +01:00
Armin Novak
1220c7af1c [channels,video] fix missing return checks 2026-03-03 13:49:58 +01:00
Armin Novak
ce3f205cd6 [core,gateway] fix missing return checks 2026-03-03 13:49:58 +01:00
Armin Novak
17163d3738 [winpr,collections] fix PubSub_OnEvent return checks
* proper return checks on use
* fix return on invalid input arguments
* fix return on no event registered
2026-03-03 13:49:54 +01:00
akallabeth
32b2bd22aa Merge pull request #12412 from akallabeth/more-return-checks
More return checks
2026-03-03 12:17:41 +01:00
Armin Novak
ba11055d66 [utils,smartcard] log all call failures
Use GNU expression statement to log a call failure if the compiler
supports that extension.
2026-03-03 11:59:14 +01:00
Armin Novak
75d8f295b4 [checks,return] Fix missing *_foreach return checks
* Check HashTable_Foreach return
* Check ArrayList_ForEach return
2026-03-03 11:07:59 +01:00
akallabeth
ef3e4a39c9 Merge pull request #12411 from akallabeth/multifrag-add-checks
Address various error handling inconsistencies
2026-03-03 10:31:09 +01:00
Armin Novak
b724ba546d [checks,return] fix various unchecked return values 2026-03-03 09:40:27 +01:00
Armin Novak
103e0907cc [core,caps] use getter/setter for MultifragMaxRequestSize
to ease debugging use the getter/setters instead of direct struct
access.
2026-03-03 09:30:34 +01:00
Armin Novak
4475e21b7e [codec,rfx] return nullptr if input parameters are invalid
rfx_encode_messages maxDataSize must be > 1024 or the function will
fail. Add a check to abort early if this is the case.
while this is a usage error, it is helpful to have proper error handling
in the function as well.
2026-03-03 09:30:33 +01:00
Armin Novak
23fc8778cb [channels,usb] fix possible deadlock
newly introduced error handling did lead to a deadlock. refactor to
avoid this.
2026-03-03 09:30:33 +01:00
Armin Novak
d34f41a30b [winpr,sspi] fix missing return check 2026-03-03 09:30:32 +01:00
akallabeth
fd642b57b1 Merge pull request #12410 from larsch/fix/uwac-damage-glitch
[uwac] fix rectangular glitch around surface damage regions
2026-03-03 09:21:56 +01:00
Armin Novak
eba2bf463c [channels,rail] refactor server side functions
* proper error handling
* use [[nodiscard]]
2026-03-03 09:12:35 +01:00
Lars Christensen
6bf6ac26ab [uwac] fix rectangular glitch around surface damage regions
Commit 09c12699 ("[uwac] window: fix damage region dimensions (rounding
errors)") added a 1-pixel expansion to the damage rect passed to
wl_surface_damage() (i.e. x-=1, y-=1, w+=2, h+=2) to guard against
rounding errors when a display scale factor is in use.

This causes visible rectangular glitches around every updated region.
The Wayland compositor re-composites exactly the declared damage region
from the attached buffer onto the screen. Because wlf_copy_image() only
writes pixels for the exact update rectangle, the 1-pixel border in the
buffer is never updated. With double-buffering the border pixels contain
stale data from the last time that buffer was used, and the compositor
faithfully composites those stale pixels, producing the artifact.

The floor/ceil rounding applied before the expansion is already
sufficient to cover any sub-pixel precision lost when converting from
buffer coordinates to surface coordinates. Remove the extra ±1 padding.
2026-03-02 19:42:26 +01:00
akallabeth
98eb11b253 Merge pull request #12409 from akallabeth/various-return-checks
Various return checks
2026-03-02 19:05:12 +01:00
Armin Novak
7de0a5bba4 [winpr] fix various return checks 2026-03-02 18:39:42 +01:00
Armin Novak
212d4d5e29 [client,common] add return checks 2026-03-02 18:32:38 +01:00
Armin Novak
9de3eaa037 [server,proxy] add return checks 2026-03-02 18:32:38 +01:00
Armin Novak
35831303ea [various] add proper return checks 2026-03-02 18:32:38 +01:00
Armin Novak
b1e4933c78 [winpr,clipboard] ClipboardRegisterSynthesizer return checks 2026-03-02 18:32:37 +01:00
akallabeth
9774cd06f5 Merge pull request #12408 from akallabeth/stream-return
[winpr,stream] fix Stream_Write_UTF16_String return checks
2026-03-02 18:06:14 +01:00
Armin Novak
8316bca9dd [winpr,stream] fix Stream_Write_UTF16_String return checks 2026-03-02 17:36:41 +01:00
akallabeth
a16e6a632f Merge pull request #12407 from akallabeth/return-checks
Return checks
2026-03-02 16:58:34 +01:00
Armin Novak
a694b4f22b [winpr,crypto] simplify expression 2026-03-02 16:31:09 +01:00
Armin Novak
b909b0cf1c [channels] Check HashTable_SetHashFunction return 2026-03-02 16:30:35 +01:00
Armin Novak
56f4337220 [server] Check WTSRegisterWtsApiFunctionTable return 2026-03-02 16:19:17 +01:00
akallabeth
c7622dffa0 Merge pull request #12406 from akallabeth/nodiscard-more-more
Nodiscard more more
2026-03-02 12:21:57 +01:00
Armin Novak
7d2f75881c [core,license] fix unused result warnings 2026-03-02 12:08:44 +01:00
Armin Novak
2b0637d629 [nodiscard] mark more internal functions 2026-03-02 11:57:27 +01:00
Armin Novak
be5c9bed64 [utils,pcap] check _fseeki64 return 2026-03-02 11:57:23 +01:00
akallabeth
f58eb53b98 Merge pull request #12403 from akallabeth/winpr-set-pos
Winpr set pos
2026-03-02 11:56:28 +01:00
Armin Novak
e6fca2c021 [winpr,stream] Fix Stream_SetPosition return checks 2026-03-02 11:37:20 +01:00
Armin Novak
0f46216a24 [winpr,ssl] check winpr_InitializeSSL return 2026-03-02 11:21:21 +01:00