Fix for #4757

Merge pull request #4754 from akallabeth/expose_cert_redirection_flag

Expose redirection flag for certificate.

Expose redirection flag for certificate.

Merge pull request #4745 from akallabeth/redirect_fix_follow_up

Fixed channel reconnect after redirect.

Fixed channel reconnect after redirect.

Merge pull request #4742 from mfleisz/fix_wave2

rdpsnd: Fix sending of wave2 PDU

rdpsnd: Fix sending of wave2 PDU

According to MS-RDPEA 2.2.3.10 the Wave2 PDU's header BodySize field
should be equal the size of the PDU minus the header (4 Bytes).

Merge pull request #4738 from akallabeth/connect_timeout_retry

Automatic timeout reconnect after callbacks

Fixed PEM certificate reading.

Moved automatic reconnect after timeout to freerdp_connect.

Added automatic reconnect if freerdp_connect fails due to timeout.

Made internal functions static to help compiler optimize.

Updated reconnect to handle cases where PostConnect was not called

freerdp_reconnect might be called after a freerdp_connect failed due
to a TCP timeout waiting for user input.
In such cases we need to know if PostConect was already called and
do that if not.

Remember accepted PEM cert to avoid unnecessary user input.

Fixed async input return value check.

Merge pull request #4727 from akallabeth/new_error

Added no or missing credentail error.

Added no or missing credentail error.

Merge pull request #4726 from akallabeth/bitmap_cache_fix

Fix #4725: Need to copy data.

Fix #4725: Need to copy data.

Merge pull request #4530 from akallabeth/order_refactor

[leak fixes] Refactored order updates

Fixed indentation.

Merge pull request #4724 from akallabeth/wave2_server

Wave2 server side support

Fixed remarks

Merge pull request #4645 from akallabeth/mac_ports_removed

Fix #2617: Removed MacPorts from CMake (default) search path.

Merge pull request #4700 from informatimago/rdpsettings-script

Rdpsettings script

Applied autoformat.sh manually to settings.h

Use tabs to indent the comment in rdp_settings too.

Merge pull request #4706 from khvMX/master

X11: fixed Right-Ctrl ungrab feature (PR #3622)

Keep extra data of audio format locally cached.

Added target dependencies.

Fixed empty lib.

Added automatic format support filter.

Fixed warning.

Added support for WAVE2 PDU in server side audio channel.

Merge pull request #4681 from akallabeth/wording_consistent

Fixed #4636: Consistent wording for boolean options depending on default

Merge pull request #4710 from akallabeth/reconnect_fixes

Reconnect fixes

Added enum for client connection state.

Fixed missing variable type

Merge pull request #4711 from akallabeth/command_line_cleanup_rework

Command line cleanup rework

Fixed memory leak in command line value parsing.

Fixed memory leak in command line value parsing.

Merge remote-tracking branch 'origin/pr/4701' into reconnect_fixes

Follow up fix for #4631

Remember the callback state to avoid calling reerdp_channels_post_connect
before the corresponding client callback has benn called.
This might happen during redirection and reconnection.

Removed duplicate resource free.

X11: fixed Right-Ctrl ungrab feature (PR #3622)

FIX: moved declarations above the statements for strict C 90 compliance.

Corrected update-rdpSettings according to PR comments.
Updated settings.h with it.

Cleaned up const char** -> char** for argv, since we definitely do modify the argv!
(we overwrite the password and pin arguments).
This implies changes in the argument parsing tests that now must pass a mutable argv
(copied from the statically declared test argvs).
Some other const inconsistency have been dealt with too.

Created script to update the #define from rdp_settings fields.

Created scripts/update-rdpSettings script to update the
rdpSettings #defines from the rdpSettings structure.

Aligned columns in rdpSettings structure declaration.

Merge pull request #4694 from akallabeth/mic_crash_fix

Fixed #4693: Fix integer underflow in encoder loop.

Fixed #4693: Fix integer underflow in encoder loop.

Merge pull request #4683 from akallabeth/ffmpeg_found_fix

Nightly build fix

Fixed case of variable names.

Merge pull request #4677 from akallabeth/build_dep_fix

Fix #4672: Check if FFMPEG and OpenH264 detected

Fix #4672: Check if FFMPEG and OpenH264 detected

When FFMPEG or OpenH264 was requested but not detected abort
the cmake run with an error.

Merge pull request #4665 from FreeRDP/akallabeth-patch-1

Update issue templates

Update issue templates

Fixed #4636: Consistent wording for boolean options depending on default.

primitives: ensure primitives_get() returns a populated struct

Fixes: https://github.com/FreeRDP/FreeRDP/issues/4658

Merge pull request #4654 from akallabeth/alsa_latency_patch

Return real alsa latency if buffer is filled.

Return real alsa latency if buffer is filled.

Merge pull request #4653 from rjcorrig/kb4093753

winpr: Updates time zone data to April 2018

Update time zone data to April 2018

Merge pull request #4651 from akallabeth/loadepng_memleak_fix

Fixed loadepng memory leak.

Merge pull request #4648 from akallabeth/nsc_free_fix

Fixed #4647: nsc_context_free must not access possibly uninitialized fields

Fix missing sound arrival time for wave2 PDU

Fixed loadepng memory leak.

Merge pull request #4637 from akallabeth/win_ninja_support

VS2017 support: set pdb output directory for Ninja generator.

Merge pull request #4631 from akallabeth/santizer_fixes

Fix #4628: CommandLineFindNextArgumentA must check the current argument

Fixed #4647: nsc_context_free must not access possibly uninitialized fields.

Merge pull request #4638 from akallabeth/known_hosts_comment_support

Added comment support for known_hosts format.

Added comment support for known_hosts format.

Fix #2617: Removed MacPorts from CMake (default) search path.

Fixed invalid function argument for ntlm_compute_message_integrity_check

Fixed compiler warnings (unused)

Fixed #4629: Only call freerdp_channels_post_connect when it was connected.

In rdp_client_redirect or rdp_client_reconnect freerdp_channels_post_connect must
be called if the channels were connected previously.
This might not be the case, skip that call then.

VS2017 support: set pdb output directory for Ninja generator.

Fix #4628: CommandLineFindNextArgumentA must check the current argument

When determining if there is a next argument first check the current one
for abort criteria.

Merge pull request #4610 from akallabeth/cmd_strtol_fix

Fix #4597: Do string argument checks before a possible strtol

Merge pull request #4623 from akallabeth/mac_sound_backend_fix

Fixed mac sound backend initialization.

Merge pull request #4627 from akallabeth/clang_warning_fixes

Clang warning fixes

Fixed NULL dereferences.

Fixed uninitialized return and early resource cleanup.

Fixed uninitialized value.

Fixed missing NULL pointer checks.

Merge pull request #4624 from akallabeth/pth_fix

Fixed /pth: Consistently treat the hash offset to password length.

Fixed memory leak.

Do not set password to identity if pth is used.

Merge pull request #4611 from akallabeth/argument_warnings

Argument warnings

Merge pull request #4625 from dualmoon/fix-hisher-pronouns

First person "they" pronouns in logoff message

Fixed formats supported by backend.

Added error log messages.

First person "they" pronouns in logoff message

Fixed /pth: Consistently treat the hash offset to password length.

Fixed mac sound backend initialization.

Merge pull request #4620 from akallabeth/bio_ctrl_fix

Bio ctrl fix

Fixed formatting.

Fixed #4476: broken casts/variable sizes for custom BIO calls.

Merge pull request #4453 from akallabeth/sound_channel_refactor

Sound channel refactoring