Fixed unicode key down and up messages.

Fixed dead store warnings

Fixed dead store warnings

Fixed dead store warnings

Fixed dead store warnings

Fixed sign-compare warnings.

Fixed unused store warnings.

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed unchecked return warning.

Fixed unused variable warnings.

Fixed sign-compare warnings, removed internal functions from API

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed _config_size return

Fixed sign-compare warnings

Fixed const free warning.

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Cast SEC_* return values to HRESULT (Same as WIN32 API)

Case SCARD_* return values to HRESULT (Same as WIN32 API)

Cast E_* errors to HRESULT (same as defined in WIN32 API)

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.