Merge pull request #2727 from akallabeth/leak_fixes_reformat

Fixed leaks, NULL dereferences and broken init.

Merge pull request #2719 from bmiklautz/pull/2481

OSS, tsmf, usb and BSD fixes and improvements

Merge pull request #2736 from akallabeth/windows_client_fix

Fixed windows client pre_connect return.

Merge pull request #2734 from giox069/master

Fixes for software GDI issues #2732 and #2137

Fixed windows client pre_connect return.

Merge pull request #2735 from akallabeth/clipboard_temp_file_name_conversion_fix

[Clipboard] Fixed out of bound array access.

Merge pull request #2729 from akallabeth/win_reg_key_by_vendor_product_define

Replaced hard coded registry keys with cmake defines.

Fixed out of bound array access.

Returing OK when there is nothing to draw after clipping, fixes #2732

Signed glyph offset calculation, fixes #2137

Merge pull request #2731 from akallabeth/mutex_cleanup_fix

Fixed mutex trylock in MutexCloseHandle.

Fixed mutex trylock in MutexCloseHandle.

Merge pull request #2728 from bmiklautz/unsigned

Fix possible endless loops on cleanup.

clipboard: fix of by one error on cleanup

Integrate pull request feedback

Use while instead of for with additional variable where appropriate.

Fixed cleanup of resources, only in error case now.

Replaced registry keys with cmake defines.

Fix possible endless loops on cleanup.

Some cleanup code possibly create endless loops because an unsigned
type was used as run variable but the check was >= 0 in the for loop.

Fixed cleanup of MachineAddresses and MachinePorts.

Fixed loop condition.

Fixed memory leak.

Fixed sizeof misuse.

Fixed a resource leak.

Fixed a resource leak.

Fixed a resource leak.

Fixed a resource leak.

Merge pull request #2717 from realjiangms/shadow_encoder_fps

server/shadow: shadow encoder related enhancement/fix.

Fixed leaks, NULL dereferences and broken init.

Merge pull request #2725 from akallabeth/warning_fixes_more_more_more

Warning fixes more more more

Removed unused variable and incorrect comment.

ifdef variables used with specific build options.

Merge pull request #2721 from akallabeth/warning_fixes_some_more

Some warnings fixed

Fixed memory leak.

Added missing argument checks.

Fixed malloc with 0 size.

Fixed uninitialized value.

Fixed uninitialized value.

Fixed grabage return value.

Fixed NULL pointer dereference.

tsmf_oss: fix compiler warning

Merge pull request #2708 from bmiklautz/dupdup

alloc and strdup checks

core: use error instead of debug

When the function would return with an error print an error message
instead of a debug message

nla and cmdline: integrated feedback

* fix possible problems with 0 size lengths
* add return value checks

server/shadow: shadow encoder related enhancement/fix.
1. Export fps related API so that subsystem implementation no longer need to know about details in encoder structure.
2. Discard frameList dictionary.
The 'value' in this dictionary is never used and not properly free'ed when client is disconnected.
The dictionary was used to calculate 'inflight' frame count. Once an ACK is received from client, an item in the dictionary is removed.
We then calculate 'inflight' frame by the count of the items in the dictionary.
However, some rdp clients (win7 mstsc) skips frame ACK if it is inactive, ACK of some frame would actually never arrive.
We actually don't need the dictionary. We only need to record the latest acknowledged frame id, and the difference between last sent frame id is the inflight frame count.
3. Minor fix in default fps calculation. encoder->frameAck is wrongly used as integer while it's actually bool flag.

oss: fix formating

windows: fix compilation and warnings

misc: integrate pull request feedback

client/locale: properly check popen return value

popen returns NULL if an error occurs and not < 0.

client/android: reset debug level to default

* WLOG debug level was hard coded to DEBUG which generates a lot of
  noise
* remove indecent and unneeded debug message

sspi/ntlm: integrate pull request comments

cmd line: add missing checks

* strdup
* some allocs

cache: fix regression

Fix a regression in cache_new introduced in commit
b4f147e242ea396ef92082d29866e5ab7d041b4e

codec/planar: regression - fix conflicting types

Update planar.h to fix the conflicting types of
freerdp_bitmap_planar_delta_encode_planes.

Fixes for malloc / calloc + other fixes

This patch contains:

* checks for malloc return value + treat callers;
* modified malloc() + ZeroMemory() to calloc();
* misc fixes of micro errors seen during the code audit:
** some invalid checks in gcc.c, also there were some possible
integer overflow. This is interesting because at the end the data are parsed
and freed directly, so it's a vulnerability in some kind of dead code (at least
useless);
** fixed usage of GetComputerNameExA with just one call, when 2 were used
in misc places. According to MSDN GetComputerNameA() is supposed to return
an error when called with NULL;
** there were a bug in the command line parsing of shadow;
** in freerdp_dynamic_channel_collection_add() the size of array was multiplied
by 4 instead of 2 on resize

A malloc() pass on WinPR

This patch treats remaining non-checked mallocs. And changes to calloc in places
where it makes sense

client/android: misc updates and fixes

* increase minimum required SDK to 14
* change functions from void to BOOL where required to have proper
  return values
* update the JAVA classes and JNI accordingly
* add return value checks for *alloc and strdup

android: fix misc compiler warnings

with gcc version arm-linux-androideabi-gcc (GCC) 4.8

winpr/wtsapi: disable tests that block on windows

winpr/thread: fix test compilation on windows

Use DWORD instead of QWORD.

Fix compiler warnings:

gcc 4.7.2

client/X11/xf_keyboard.c: In function ‘xf_keyboard_action_script_init’:
client/X11/xf_keyboard.c:48:6: warning: unused variable ‘exitCode’
[-Wunused-variable]

gcc 4.9

client/X11/xf_client.c: In function ‘xf_client_thread’:
client/X11/xf_client.c:1537:3: warning: ‘inputThread’ may be used
uninitialized in this function [-Wmaybe-uninitialized]

client/ios: fix unchecked strdups

Fix unchecked strdups

* add missing checks
* adapt function return values where necessary
* add initial test for settings

Merge pull request #2686 from bmiklautz/ios/openssl

ios: update openssl build script

Merge pull request #2705 from akallabeth/sspi_fix_v2

Sspi fix v2 - Fixes static build

Merge pull request #2697 from realjiangms/shadow_audio

Shadow server: fix channel disposal and add audio support. fix message mechanism in client

Merge pull request #2698 from realjiangms/shadow_stand_alone

server/shadow: Decouple shadow_subsystem and subsystem implementation detail.

Merge pull request #2699 from realjiangms/shadow_connect_disconnect

server/shadow: add connect/disconnect anchor in subsystem callback

Merge pull request #2701 from akallabeth/upn_syntax_support

Allowing UPN syntax for user argument.

Merge pull request #2684 from Skool/master

Add bépo layout

Merge pull request #2636 from xhaakon/master

Fix crashes in shadow server

Merge pull request #2713 from bmiklautz/divbyzero

core/metrics: prevent division by zero

Merge pull request #2702 from akallabeth/nla_null_identity_support

NLA Use current users credentials if none provided.

Merge pull request #2711 from nfedera/nf-fix-rdprecvmessagechannelpdu

core: message channel pdu broken with rdp security

core/metrics: prevent division by zero

core: message channel pdu broken with rdp security

rdp_recv_message_channel_pdu always read the rdp security header
even if it was already previously read (which is the case if rdp
security is active)

This caused malfunctions and disconnects when heartbeat or bandwidth
autodetect packets were sent/received in rdp security mode.

Credit goes to @MartinHaimberger for identifying the broken code
part.

Setting hostname as default domain now.

Fixed typo and invalid domain name.

Fixed domain setting for windows client.

UPN syntax fix: domain must be empty, not NULL.

Added user setting if no domain provided.

Fixed variable parsed.

Using last @ for domain split.
Command line parser splits username after
parsing all options.

Merge pull request #2706 from nfedera/nf-fix-reset-event

winpr/synch: fix ResetEvent & improve CTest

Merge pull request #2675 from akallabeth/path_make_path

Added PathMakePath function.

winpr/synch: fix ResetEvent & improve CTest

Unless there is an invalid parameter or internal error, ResetEvent
must return TRUE, even if the object is in nonsignaled state.

Also fixed and improved the CTest TestSynchEvent to test for the
expected/correct SetEvent and ResetEvent return values.

Removed duplicate function tables, respecting WITH_NATIVE_SSPI

Using InitSecurityInterfaceEx(0) now.

Fixed cmake warning.

Merge pull request #2703 from akallabeth/unified_sspi

Let SSPI module choose which implementation to use.

server/shadow: integrate comment from @hardening
Return directly at subsystem->ClientConnect
Also change to argument to accept shadow_client instead of freerdp_peer

server/shadow: Integrate comment from @hardening: use switch to handle different message id.

Let SSPI module choose which implementation to use.

NLA identity now a pointer, allowing NULL.

When NLA identity is NULL, the current user context
is used on windows.

Allowing UPN syntax for user.

server/shadow: Decouple shadow_subsystem and subsystem implementation detail.
Make shadow_subsystem standalone so that others can implement customized shadow server with shadow library.

server/shadow: add connect/disconnect anchor in subsystem callback
Other implementation of subsystem may want to track connect/disconnect of each client

This is originally to add audio support. Several fixes included:
1. Introduce message queue in shadow client. No longer use subsytem->MsgPipe->out to deliver message to clients.
We used to use subsytem->MsgPipe->out for messages which need to be sent to client. But it's not correct. Only one client would get the message if multiple client exists
This problem make the fix in PR #2643 incomplete.
Introduced reference count based solution to release resource taken by the message.
Also added APIs for client message delivery.
Also fixed msg pipe in subsystem to clean resource when destroyed.
2. Discard unused StopEvent in client. We actually use quit message instead.
3. Enhance disposal of channels.
Free context for remdesk and encomsp channels. The original fix only stop the threads, but doesn't release resource.
Dispose channels earlier. The channels are built on client->vcm. Disposing channels after client->vcm is closed cause unknown behavior.
Original fix is #2644
4. Start to add audio support.

Merge pull request #2687 from akallabeth/known_hosts_v2

Store SSL fingerprints with host and port

Merge pull request #2692 from nfedera/add-16bit-cursor-conversion

codec/color: support for 8bpp and 16bpp color pointer

Merge pull request #2688 from hardening/fix_packaging

Correct the included files to have a working freerdp-nightly-dev package