Addin function pointers now expect const strings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed unused variable warnings.

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed missing return value check.

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed free checks to eliminate bad free warnings.

Fixed sign-compare warnings

Fixed NSC SSE argument checks.

Merge pull request #5285 from akallabeth/urbdrc_preparations

Channel, log debug and semaphore related fixes

Merge pull request #5309 from smaeul/patch/ntlm

Fix NTLM AvPair lists

Merge pull request #5318 from akallabeth/filetime_fix

Fix #5317: Change file attributes before modifying file time.

Merge pull request #5251 from akallabeth/wayland_cursor_handling_fix

Moved wayland cursor handling to seat.

Merge pull request #5273 from akallabeth/mac_compile_warning_fix

Fixed mac compilation warnings.

Merge pull request #5319 from akallabeth/clip_file_fix

Fixed \0 terimination of clipboard file list

Fixed \0 terimination of clipboard file list

Fix #5317: Change file attributes before modifying file time.

Merge pull request #5249 from akallabeth/mac_timer_crash_fix

Mac timer crash fix

Merge pull request #5281 from akallabeth/assistance_ssl_init_fix

Fixed #5276: Enable SSL before parsing assistance file.

Merge pull request #5295 from akallabeth/stream_macro_fix

Fixed Stream_* macros, bracing arguments.

Merge pull request #5313 from m4ntis/fix-end-paint-log-level

Fix update_end_paint wrong log level

update.c: Fix update_end_paint wrong log level

Merge pull request #5307 from akallabeth/flatpak_dep_fix

Updated flatpak udev build script

Merge pull request #5311 from m4ntis/bugfix/gfx-w2s2-decode

Fix Wire to Surface 2 pixel format decode on client

client/rdpgfx_main.c: Fix wire to surface 2 pixel format decode

Close file handle correctly. (#5310)

Fix NTLM AvPair lists

There were two main issues here: First, the `ntlm_av_pair_add` and
`ntlm_av_pair_add_copy` were not adding a new `MsvAvEOL` to the end of
the list to replace the one they overwrote. This caused the second call
to one of those functions to fail (since it couldn't find the
terminator), which was the source of the test failure. It also caused
`ntlm_av_pair_list_length` and `ntlm_print_av_pair_list` to read out of
bounds until they happened to find the right word.

Second, several bounds checks were wrong or missing. For example,
`ntlm_av_pair_add` does not ensure that the value fits inside the list.
And `ntlm_av_pair_get_len` and `ntlm_av_pair_get_value_pointer` can
return error codes or NULL, but those error returns were ignored, and
the values used anyway (such as in `ntlm_av_pair_add_copy`).

This fixes the list handling code to have the invariant that all
functions returning `NTLM_AV_PAIR*` only return non-`NULL` if the entire
returned `AvPair` is within bounds. This removes the need for the length
parameter in functions that only operate on a single `AvPair`. This
check is performed by the new `ntlm_av_pair_check` helper, which is
added in some new places and used to simplify the code in others.

Other issues fixed along the way include:
 - `ntlm_av_pair_list_length` did not cast to `PBYTE`, so it was
   returning the number of `NTLM_AV_PAIR`-sized chunks (which was
   possibly not even an integer) instead of the number of bytes
 - I removed an impossible check for `offset <= 0` in
   `ntlm_av_pair_get_next_pointer`
 - The assertion that `Value != NULL` and the call to `CopyMemory` are
   only necessary if `AvLen` is nonzero
 - `ntlm_av_pair_get_next_pointer` (renamed to `ntlm_av_pair_next`)
   could be declared `static`

With this commit, TestNTLM now passes on powerpc64.

```
$ ./Testing/TestSspi TestNTLM
NTLM_NEGOTIATE (length = 40):
NTLM_CHALLENGE (length = 168):
NTLM_AUTHENTICATE (length = 352):
$ echo $?
0
```

Fixes #5250

Fixed flatpak pcsd build

Updated flatpak udev build script

server/rdpgfx_main.c: Fixed caps advertise PDU recv (#5304)

Merge pull request #5297 from akallabeth/nla_server_functions

Exposing NLA functions to impersonate and revert context.

Exposing NLA functions to impersonate and revert context.

Fixed Stream_* macros, bracing arguments.

Merge pull request #5294 from akallabeth/gfx_temporary_disable_new

Temporarily disable GFX version >= 10.3

client/Windows/wf_event.c: add missing "break" (#5293)

found by cppcheck

[client/Windows/wf_event.c:464] -> [client/Windows/wf_event.c:468]: (warning) Variable 'xNewPos' is reassigned a value before the old one has been used. 'break;' missing?

Temporarily disable GFX version >= 10.3

Merge pull request #5252 from akallabeth/big_endian_fix

Fixed endianess issue with GETPIXEL16 and GETPIXEL32

Merge pull request #5289 from akallabeth/gfx_crash_fix

Disabled RDPGFX_CAPVERSION_106 for good.

Disabled RDPGFX_CAPVERSION_106 for good.

Merge pull request #5027 from akallabeth/wlog_off_fix

Fix #5024: Update inherited log level for all child loggers when setting a new one.

Fixed winpr_HexLogDump line length calculation.

Merge pull request #5288 from akallabeth/rdp_spec_updates

Rdp spec updates

Better bounds checks for winpr_HexLogDump

Fixed EGFX capability parsing, respect length fields.
Disable RDPGFX_CAPVERSION_106 as we currently do not support scaled
outputs.

Updated EGFX support to 10.6

Fixed version magic mismatch with [MS-RDPBCGR]

Fixed ReleaseSemaphore, respect lReleaseCount

Signal the semapore lReleaseCount times.

Initialize ppKeys to NULL in HashTable_GetKeys

Ensure ppKeys=NULL in case of a function failure or empty table.

Added a logger dump function operating on a preallocated logger.

Force close channel, use defines instead of magic numbers

* When senging a close request for a dynamic channel close the channel
  immediately. A response from the server is not guaranteed.
* Use the *_PDU defines instead of magic numbers when creating
  dynamic channel messages.

potential fix for ENTER (modifiers) stuck after CTRL-ALT-ENTER (toggle fullscreen), clearing the buffer in xk_keyboard_update_modifier_keys. (#5280)

* xk_keyboard_update_modifier_keys fixed to clear the buffer.

* removed the xf_keyboard_clear(xfc) fix comments

Fixed #5276: Enable SSL before parsing assistance file.

The assistance file requires primitives from the ssl wrapper.
Enable these before parsing the file.
Additionally split the FIPS mode enablement from the one time
initializer to avoid ignoring that flag.

Fixed TS_INFO_PACKET (#5275)

Those fields exclude the length of the mandatory null terminator:
* cbDomain
* cbUserName
* cbPassword
* cbAlternateShell
* cbWorkingDir

Merge pull request #5274 from chipitsine/master

libfreerdp/codec/include/bitmap.c: resolve possible NULL pointer arithmetic

libfreerdp/codec/include/bitmap.c: resolve possible NULL pointer arithmetic
found by cppcheck

[libfreerdp/codec/include/bitmap.c:112] -> [libfreerdp/codec/include/bitmap.c:95]: (warning) Either the condition '!pbSrcBuffer' is redundant or there is pointer arithmetic with NULL pointer.
[libfreerdp/codec/include/bitmap.c:112] -> [libfreerdp/codec/include/bitmap.c:96]: (warning) Either the condition '!pbDestBuffer' is redundant or there is pointer arithmetic with NULL pointer.

Merge pull request #5271 from akallabeth/create_event_exa_scanbuild_fix

Create event exa scanbuild fix

Fixed mac compilation warnings.

Merge pull request #5272 from akallabeth/ssl_api_warning_fixes

Ssl api warning fixes

Silenced unused-const-variables for test cases.

Fixed cast warnings from freerdp_load_channel_addin_entry

Silenced function pointer cast warnings for BIO_callback_ctrl

Ignore INVALID_HANDLE_VALUE during clang scanbuild runs.

The value INVALID_HANDLE_VALUE could in theory be a valid memory address,
so the analyzer is confused and thinks either we have a memroy leak
or we try to free a fixed address.

Respect return value from SetEvent and abort creation.

Merge pull request #5270 from akallabeth/warning_fixes_more_more_more

Warning fixes more more more

Fixed conversion warnings.

Range checks and type casts to remove all warnings in printer channel.

Fixed all integer conversion warnings for OpenH264.

Added range checks and casts to integer where necessary
to remove warnings.

Fixed xrealloc argument type

Using a void* as arugment instead of char* removes warnings when
allocating something beside a string.

Merge pull request #5269 from anbukart/master

Embedded window client

Embedded window local fix

Client width and height getting set as 0 when embedded using ParentWindowId setting

Merge pull request #5265 from akallabeth/wayland_mouse_wheel_adjust

Fixed #5262: Vertical mouse wheel direction and whell step

Fixed #5262: Vertical mouse wheel direction and whell step

* Vertical wheel uses the inverted direction of horizontal wheel.
* The wheel step is now calculated from the value provided by wayland.

Use safe iteration for seat removal.

Merge pull request #5248 from akallabeth/naming_fixes

Replace hard coded xfreerdp program name

Fixed endianess issue with GETPIXEL16 and GETPIXEL32

Moved wayland cursor handling to seat.

Merge pull request #5246 from wintersandroid/master

Resolve Gfx mode missing surface updates from cache.

Fixed #5234: Added timer invalidate on application willresignactive

Replaced hardcoded xfreerdp program name

Replaced hardcoded xfreerdp program name

Replaced hardcoded xfreerdp program name

Replaced hardcoded xfreerdp program name

Merge pull request #5245 from akallabeth/uwac_zero_init

Fix 5243: Allocate buffers zero in uwac.

Resolve Gfx mode missing surface updates from cache.

Fix 5243: Allocate buffers zero in uwac.

Merge pull request #5219 from akallabeth/wayland_smart_sizing

Wayland smart sizing

Destroying released cursor buffers.

Don't destroy wayland buffer attached to a surface.

Added support for /smart-sizing:<width>x<height> syntax in wayland client.

Fixed duplicate log tag definition in wayland client.

Fixed conversion range checks and cast for use with cairo

Fixed missing initializer.

Removed unused variables.

Fixed color format for wayland mouse pointer.

Added mouse pointer scaling.