Fixed resource cleanup.

Fixed formatting.

Fixed leaks and formatting.

Fixed color for drawing rectangles.

Fixed missing function return check.

Fixed missing parameter checks.

Fixed uninitialized variables.

Fixed missing error check.

Fixed uninitialized arguments.

Fixed warnings and formatting.

Functions static, warnings fixed.

Fixed formatting and warnings.

Fixed dead store warning.

Removed unused argument.

Fixed missing function return check.

Fixed multiple warnings in parser

Fixed compile warnings.

Fixed warnings and test return values.

Fixed NULL arguments and compile warnings.

Fixed dead store.

Merge pull request #3940 from akallabeth/custom_help_arguments

Added a new function to allow printing help with additional arguments.

Merge pull request #4235 from akallabeth/avcodec_encode_video_fix

Added encoder path for libavcodec versions <1.0

Allow printing of custom arguments in help.

Added encoder path for libavcodec versions <1.0

Merge pull request #4233 from akallabeth/kerberos_rebased

Kerberos rebased

Merge pull request #4232 from akallabeth/ffmpeg_compat

Added compat define for missing format.

Fix logic in nla_read_ts_credentials

Fixed uninitialized values and leaks.

fix Kerberos flavour's detection (MIT/Heimdal) and double free for MIT<1.13

fix typo

fix undeclared identifier on Windows

use SSIZE_T instead of ssize_t

handle missing ssize_t on Windows

remove useless includes

fix: GSS API init, enterprise name management, variable names and format code

fix undefined symbol references at linking stage

initial commit for kerberos support

Added compat define for missing format.

Merge pull request #4122 from ben-cohen/sshagent

Forward ssh-agent data between ssh-agent and RDP

Merge pull request #4212 from SriRamanujam/ffmpeg_encoder_fixes

Fix libavcodec encoding errors and set tunables.

Fallback #ifdefs for older versions of libavcodec

Remove underscores from define in sshagent_main.h

Fix comments at the start of these files and adjust copyrights

Add server side plugin [UNTESTED]

This is based on xrdpapi/xrdp-ssh-agent.c from xrdp PR #867.

Add rdpcontext so read thread can report channel error

Also fix copyright dates...

Remove EAGAIN/EWOULDBLOCK from blocking read/write

Fix socket fd leak and other changes

1. In connect_to_sshagent() if connect() fails, the socket agent_fd is
   leaked.  It needs to be closed before returning.

2. Fix copyright messages.

3. Make if statement with call to CreateThread() clearer to read.

Forward ssh-agent data between ssh-agent and RDP

Add the sshagent plugin to forward the ssh-agent protocol over an RDP
dynamic virtual channel, just as the normal ssh-agent forwards it over
an SSH channel.  Add the "/ssh-agent" command line option to enable it.
Usage:

Run FreeRDP with the ssh-agent plugin enabled:

   xfreerdp /ssh-agent ...

In the remote desktop session run xrdp-ssh-agent and evaluate the output
in the shell as for ssh-agent to set the required environment variables
(specifically $SSH_AUTH_SOCK):

   eval "$(xrdp-ssh-agent -s)"

This is the same as for the normal ssh-agent.  You would typically do
this in your Xsession or /etc/xrdp/startwm.sh.

Limitations:

1. Error checking and handling could be improved.

2. This is only tested on Linux and will only work on systems where
clients talk to the ssh-agent via Unix domain sockets.  It won't
currently work on Windows but it could be ported.

Merge pull request #4225 from krisztian-kovacs-balabit/use-redirection-pdu-password-on-reconnect

core/connection: use redirection password when reconnecting

Merge pull request #4226 from krisztian-kovacs-balabit/open-x509-keyfile-readonly

libfreerdp/core/certificate: open key file for reading only

Fix libavcodec encoding errors and set tunables.

libfreerdp/core/certificate: open key file for reading only

There's no point in writing the key file for read-write, and it makes it
impossible to run the shadow server with the key file being read only.

Merge pull request #4224 from krisztian-kovacs-balabit/nsc-memory-corruption-fix

codec/nsc: fix memory corruption in case of chroma subsampling

Merge pull request #4163 from rjcorrig/tzscripts

winpr: Convert time zone generator programs to C# scripts

Merge pull request #4151 from ondrejholy/manpages-update-master

Manpage and /help update

core/connection: use redirection password when reconnecting

According to MS-RDPBCGR the server might send a password in the Redirection PDU
that then must be sent by the client to the server as password.

Since the field either contains a password string (unicode) or a binary cookie,
we try to convert the password from unicode and use it only if conversion
succeeds.

codec/nsc: fix memory corruption in case of chroma subsampling

For odd number of rows, the memory copy operation was broken: after exiting
the loop, yplane points to the end of the last row data, and thus (yplane +
rw) points *after* the end of the last row.

Merge pull request #4223 from akallabeth/mac_client_crash_fix

Added missing pointer callback.

Added missing pointer callback.

Merge pull request #4219 from akallabeth/various_fixes

Various fixes

Merge pull request #4211 from akallabeth/silence_duplicate_warnings

Silence WLog_ERR messages if last error is set.

Merge pull request #4214 from hardening/serial_fixes

Serial fixes

Fixed compilation errors with old FFMPEG versions.

Fixed NLA default error to FREERDP_ERROR_AUTHENTICATION_FAILED

serial redirection: implement event char

The signotec signature device requires the eventChar support to work properly in
serial redirection mode. This implementation is basic but does the job for this
device.

Sponsored by: Rangee GmbH (http://www.rangee.de)

Disambiguate USB error messages and fix a typo

Added some checks for the serial redirection channel

Sponsored by: Rangee GmbH (http://www.rangee.de)

Merge pull request #4210 from akallabeth/nla_errors_extended

Added additional NLA error mappings.

Silence WLog_ERR messages if last error is set.

Merge pull request #4053 from akallabeth/ffmpeg_encoder

Implemented FFMPEG based encoder.

Merge pull request #4200 from cedrozor/upstream-master

Windows client: refresh only updated region(s) instead of whole screen

Merge pull request #4187 from hardening/multimon_fix

Multimonitor fix

Added additional NLA error mappings.

Merge pull request #4206 from akallabeth/x_window_title

X11 window title fix

X11 window title fix

Merge pull request #4205 from akallabeth/WIN32_FIND_DATA

Fixed #4199: ConvertFindDataAToW string length

Fixed #4199: ConvertFindDataAToW string length

Merge pull request #4159 from aleksei-klv/dex

Optimized aFreeRDP for Samsung DeX

Merge pull request #4179 from akallabeth/android_cleanup_fix

[android] Ensure native session is disconnected before resource free.

Merge pull request #4204 from lunixoid/master

Fixed twice assigned values

Fixed twice assigned values

Merge pull request #4203 from floppym/mkdir-error

winpr: _IoCreateDeviceEx: fix mkdir error check

winpr: _IoCreateDeviceEx: fix mkdir error check

The mkdir(2) function returns 0 on success, and -1 on error.

This resolves an error in TestIoDevice when /tmp/.device/ does not
exist.

Bug: https://bugs.gentoo.org/635838

Merge pull request #4201 from kakaroto/listener-unlink-bugfix

Do not delete the listener socket right after creating it.

Merge pull request #4202 from lunixoid/master

Static analyser issues

Update semaphore.c

Fixed Expression 'pComm != NULL' is always true

pComm was checked earlier in 1363 string:
`if (pComm == NULL)`

Fixed duplicated assigne

Do not delete the listener socket right after creating it.

The listener server socket file needs to be deleted before we bind it
otherwise it's an "address already in use" error. But it was getting
deleted after the bind, causing the file to disappear, and preventing
anyone from connecting to the listener socket since the socket stops
existing.

This is caused by commit 884e87fde4ece132c3b7bdf947de044ecb3d4b14

fixed hdc initialization into wf_begin_paint so the region invalidation into wf_end_paint applies to the same hdc
the objective is that only the updated region(s) are refreshed instead of the whole screen
it's not much a problem for normal FreeRDP usage but becomes a serious issue if the display is processed, forwarded and rendered remotely, as done by web gateways such as Guacamole, FreeRDP-WebConnect and Myrtille (I'm the author of the latter)

remove temporary hack

Merge pull request #4198 from akallabeth/nla_error_fix

Added better error mapping for NEGO results.

Added better error mapping for NEGO results.

Fixed possible race condition on resource cleanup.

Merge pull request #4195 from nfedera/fix-sse3-yuv444pixel

primitives/yuv: fix endianess issue

primitives/yuv: fix endianess issue

Merge pull request #4190 from akallabeth/media_foundation_fix_win7

Removed unused function pointer

Removed unused function pointer

The function was breaking windows 7 compatibility.

Merge pull request #4189 from nfedera/fix-h264-init

codec/h264: fix potential segfault and mf_init

codec/h264: fix potential segfault and mf_init