Merge pull request #2466 from WarheadsSE/fix-cert-chain-validation-GH2446

libfreerdp: crypto: add certificate chain validation!

libfreerdp: crypto: add certificate chain validation!

Merge pull request #2463 from akallabeth/vs2010_error_fix

Fixed mixed declaration of code and variables.

Fixed mixed declaration of code and variables.

Merge pull request #2447 from ivan-83/master

Fix poor sound quality with ALSA, add OSS sound system support, FreeBSD patches

Merge pull request #2459 from akallabeth/handle_close_cb

Refactored WaitForSingleObject and CloseHandle

Merge pull request #2456 from zhangzl2013/fix-realloc

Fix incorrect usage of realloc

Merge pull request #2452 from bmiklautz/cmd_fix

settings: handle collection_add in detection case

Merge pull request #2448 from eledoux/ports2

Fixed a regression on the #2333 resolution

Merge pull request #2426 from bjcollins/master

xfreerdp fullscreen enhancements

Merge pull request #2430 from serayrosh/master

release keys when xfreerdp is unfocused to prevent stuck keys

Merge pull request #2458 from bmiklautz/cert_handling

crypto: use PEM as certificate store file format

Merge pull request #2457 from akallabeth/pubsub_fix

PubSub: Fixed missing loop exit condition.

Fixed missing return value check.

Initializing handles with calloc now.

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using new handle cleanup structure

Using handle close callback now.

Using handle close callback now.

Using handle close callback now.

Using handle close callback now.

Using handle close callback now.

Using handle close callback now.

Using handle close callback now.

Using handle close callback now.

Fixed PubSub_Unsubscribe.

crypto: use PEM as certificate store file format

Currently the certificate format expected in FreeRDPs certificate store
is DER (ASN1). On most linux/unix systems the system certificate store
default format is PEM. Which is also the more common format used by CAs
to distribute their certificates.

Changing the default format to PEM allows the usage of system
certificates or published CA certificates without the need to convert them.

This fixes a part of issue #2446.

Fixed missing loop exit condition.

wf_cliprdr: fix potential memory leak.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

libwinpr: check return value of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

fix build error.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

tool: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

libwinpr: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

codec: check return value of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

keyboard_layout: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

gdi: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

settings: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

core: fix incorrect usage of realloc.

realloc can handle the case that its first parameter is NULL.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

crypto: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

xf_cliprdr: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

android_event: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

cmdline: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

common/file: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

wf_cliprdr: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

rail: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

audin: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

tsmf: fix incorrect usage of realloc.

Signed-off-by: Zhang Zhaolong <zhangzl2013@126.com>

* OSS build fix on linux

Merge pull request #2455 from WarheadsSE/fix-hostname-matching

Fix case sensitive hostname comparison in tls

Fix case sensitive hostname comparison in tls

To do this I've swapped _strnicmp with memcmp. Seemless, but does lock it to the restrictions of that function.

Signed-off-by: Jason Plum <jplum@archlinuxarm.org>

settings: handle collection_add in detection case

Command line detection is run with dummy settings where not everything
is allocated. Collections (device, dynamic channel and static
channel) didn't handle this case properly.

* FreeBSD timezone detection patch

* remove OSS debug messages

* BSD/FreeBSD build fix

winpr-comm: fixed a regression about timeouts to block of first bytes when required. Added timeouts values in the DEBUG log.

+ some files for OSS, missed from prev commit

+ add OSS support for rdpsnd

Fix poor sound quality with ALSA
and decrease "ALSA lib pcm.c:7339:(snd_pcm_recover) underrun occurred"
+ little code cleanup

Merge pull request #2435 from llyzs/llyzs

Add openh264 encoder support.

Merge pull request #2436 from bmiklautz/warnings

Compiler warnings

build: fix compiler warnings

warning: redundant redeclaration
Tested with: 4.7.2 and 3.5.0-1~exp1

build: enable more warnings for gcc

Enable implicit-function-declaration and redundant-decls warnings when
compiling with gcc. redundant-decl useful for exported headers.

Merge pull request #2433 from abergmeier/master

Added support for finding OpenSLES with CMake + Android.

Add openh264 encoder support.

Merge pull request #2428 from akallabeth/ios_color_fix

Fixed bitmap context for 15bit color depth

Added support for finding OpenSLES with CMake + Android.

release keys when xfreerdp is unfocused to prevent stuck keys

Fixed bitmap context for 15bit color depth

Merge pull request #2422 from akallabeth/nmake_warning_fixes

Fixes warning D9025 with NMake

Remove override redirect flag for fullscreen with keyboard grab enabled. Window manager should always know about the main window.
Small cleanup of passing around decorations flag.
Limit PercentScreen to single monitor vs. entire desktop. IMO - this is better behavior in a multimonitor environment.

Handle fullscreen windows better:
1. Ensure that size hints are set to allow resizing before setting a window to fullscreen as some window managers do not behave properly.
2. Handle fullscreen toggles without destroying and recreating window.
3. Use NET_WM_STATE_FULLSCREEN Extended Window Manager Hint for fullscreen functionality
4. Use the NET_WM_FULLSCREEN_MONITORS Extended Window Manager Hint when appropriate
5. When a single monitor fullscreen is requested - use the current monitor(as determined from mouse location)
6. Handle cases where there is no local monitor at coordinate 0,0. The Windows server expect there to be a monitor at this location, so we maintain offset if necessary between our local primary monitor and the server side primary monitor located at 0,0.

Fixes warning D9025 with NMake

Merge pull request #2416 from zavadovsky/transport_dead_end_fix

libfreerdp-core: fix dead-end state in transport_read_pdu()

Merge pull request #2415 from zavadovsky/android_fixes

Android fixes

libfreerdp-core: add comments for previous fix

Merge pull request #2409 from bmiklautz/tcp_user_timeout

core/tcp: set TCP_USER_TIMEOUT if available

client/Android: fix coords calculation in previous commit

Merge pull request #2417 from austin987/master

winpr/asn1: add ASN1_CreateDecoder/ASN1_Decode stubs

winpr/asn1: add ASN1_Decode stub

winpr/asn1: add ASN1_CreateDecoder stub

libfreerdp-core: fix dead-end state in transport_read_pdu()

Situation: we have fragmented TPKT PDU without two last bytes
(or one last byte - for fast-path) in network stack.

First call to transport_read_pdu() works normally, read
available bytes and exit with status 0 - no whole PDU readed.

Before second call this missed bytes arrive.
Optionally with next PDU.

In second call header parsing code unconditionally read this
two bytes(one byte) despite this is not header bytes.
And increase stream position, so stream now contains whole PDU.

This cause (pduLength - Stream_GetPosition(s)) calculation to be 0.
So transport_read_layer_bytes()-->transport_read_layer() return 0
and transport_read_pdu() exits with "not enough data is available"
status.

If next PDU isn't available next calls to transport_read_pdu()
give same result.
If next PDU arrive - (pduLength - Stream_GetPosition(s)) will be
less than 0. Stream position will grow, grow and grow on each call.
And transport_read_pdu() never signals that PDU is readed.

Caught on Android FreeRDP client with high RDP traffic (several MBytes/s).

client/Android: optimize android_end_paint() logic

1. Don't draw to screen if server sends begin/end paint
messages without any real updates.
2. Redraw only summary region that covers really invalidated
part of screen.

client/Android: fix bad realloc

Fast and long pointer moves cause app to crash.

Merge pull request #2411 from awakecoding/master

SCardTransmit pioRecvPci Fix

channels/smartcard: fix SCardTransmit return encoding with pioRecvPci

core/tcp: set TCP_USER_TIMEOUT if available

Keepalive settings are usually (depending on the implementation) only
used if the TCP connection is idle.
If the network is interrupted/disconnected/... click or keyboard input
generates outgoing traffic therefore the connection isn't idle
anymore and keepalives might not be used causing the connection to
stay open and the client to stall.

Linux 2.6.36 added a TCP_USER_TIMEOUT TCP socket option that lets a
program specify the maximum time transmitted data may remain
unacknowledged before TCP will close the corresponding connection with
ETIMEDOUT.

Setting TCP_USER_TIMEOUT allows us to detect a network problem (like
cable disconnect) even if the connection isn't idle.

Merge pull request #2408 from dvincent-devolutions/gateway

TSGateway out channel recycling

libfreerdp-core: improve RTS PDU receiving

libfreerdp-core: cleanup channel recycling