Removed INVALID_HANDLE_VALUE.

Removed INVALID_HANDLE_VALUE from types not using it.

Removed INVALID_HANDLE_VALUE from types not using it.

Fixed conversion warnings.

Fixed NULL pointer dereference.

Fixed missing checks for invalid handle.

Fixed use after free and realloc checks.

Merge pull request #2511 from hardening/fix2399

Fix segfault when the xkb layout has no variants

Fix segfault when the xkb layout has no variants

This is a fix for #2399: when there's no variants we should not try to scan them.
I have set the RDP US keyboard for the South African layout, if someone has a better
layout...
Note: we should probably set something that is not zero for other layouts

Merge pull request #2510 from akallabeth/build_date

Added version and build info functions

Merge pull request #2502 from bmiklautz/capacity

stream: Stream_Ensure*Capacity: change return type

Integrate feedback from pull request

* orders.c: remove ; at eol
* rdpdr_main.c: simplify error case
* rdpsnd_pulse.c: fix incorrect check
* rdpsnd_main.c: adapt return value according to documentation
* update.c: fix incorrect return values

Fixed return value in test.

Using sprintf_s now.

Now returning const char*

Fixed windows _snprintf.

Added test for freerdp version functions.

Added version and build info functions

Added functions to get
 * Version String
 * Build Time String
 * Git Revision String
and appropriate tests.

drdynvc: ensure stream != NULL

cliprdr: ensure stream != NULL

coding style fixes

Add missing space after if

stream: Stream_Ensure*Capacity: change return type

Change the return type of Stream_Ensure*Capacity from void to BOOL to be
able to detect realloc problems easily. Otherwise the only way to detect
this was to check if the capacity after the call was >= the required
size.
In case Stream_Ensure*Capacity fails the old memory is still available
and need to freed outside.

This commit also adds checks to most calls of Stream_Ensure*Capacity to
check if the call was successful.

Merge pull request #2492 from bmiklautz/malloc_wrap

Fix compiler warnings and code hardening

Merge pull request #2508 from nfedera/fix-2015-03-30-01

crypto: support MSFT iOS clients with faulty SNI

Merge pull request #2507 from nfedera/fix-2015-03-30-02

nego: fixed X.224 Connection Request PDU parsing

crypto: support MSFT iOS clients with faulty SNI

Microsoft iOS Remote Desktop Clients eventually send NULL-terminated
hostnames in SNI which is not allowed in the OpenSSL implementation.

Since we're not using SNI this commit adds an OpenSSL TLS extension
debug callback which modifies the SSL context in a way preventing it
from parsing this extension

nego: fixed X.224 Connection Request PDU parsing

The X.224 Connection Request PDU might contain an optional cookie or
routing token before the optional RDP Negotiation Request (rdpNegReq).
If present, both of these fields must be terminated by a 0x0D0A
two-byte sequence. It seems that until now FreeRDP has incorrectly
assumed that a token or cookie must always be present.
If that was not the case, FreeRDP was searching for 0x0D0A until it
arrived at the end of the stream which prevented the remaining data
(RDP Negotiation Request, RDP Correlation Info) from being parsed.

Merge pull request #2483 from akallabeth/feat/freerdp_set_last_error_name

Feat/freerdp last error name

Merge pull request #2498 from TaGUNil/tsmf_fix_memory_leak

Fix memory leak in TSMF video playback

tsmf: free frame data after processing

Merge pull request #2493 from llyzs/llyzs

tls: revert checking SSL_ERROR_SYSCALL which may cause deadlock.

Merge pull request #2494 from akallabeth/openh264_link_fix

Fixed OpenH264 linking

Integrate feedback from pull request (#2492)

* shell: add missing NULL checks
* thread: handle case where HAVE_EVENTFD_H isn't defined
* wlog: return NULL instead of 0
* wlog: use g_RootLog instead of WLog_GetRoot
  otherwise a new root might be initialized on uninit
* indentation and style fixes

opensl: remove unnecessary memset

Fix compiler warnings

gcc 4.7.2
* warning: value computed is not used [-Wunused-value]
* warning: unused variable ‘oss’ [-Wunused-variable]

hardening

Start to add missing checks for:
* *alloc
* *_New

gdi/gfx: remove unused variables

gdi: remove unnecessary includes

gdi: remove dead code

asn1: use enum instead of returning a fixed value

rdtk: remove redundant casts

Added openh264_dll to library name hints.

tls: revert checking SSL_ERROR_SYSCALL which may cause deadlock.

Merge pull request #2482 from llyzs/llyzs

tls: fix error handling of no error code.

Merge pull request #2489 from akallabeth/security_fixes

Security fixes

Fixed uninitialized test argument buffer.

Fixed pipe cleanup.

Merge pull request #2443 from realjiangms/fix_region

Incorrect extents calculation in region16_intersect_rect (libfreerdp/codec/region.c)

Add ascii art for norbert_test_case

resolve comments from hardening

fix tab/space style

Fixes extents of region16_intersect_rect.
See Issue #2443.
When there's more than 2 rectangles in the region structure, region16_intersect_rect would calculate extents by all 'intersected' sub rectangles.
But it always extend the extents to (0,0) because it initialize the new extents as (0,0,0,0) and union later rectangles with this empty point by simple MIN/MAX calculation.

Also fixed rectangle_is_empty although it has not been used yet. The function does not work as its name.
Reuse norbert case. That case is enough for the intersect fix, but the expected result is not correct. The test case is also fixed.
Added test case to check empty rectangle.

tls: retry for all SSL_ERROR_SYSCALL errors.

Merge pull request #2484 from dvincent-devolutions/gateway

RDP8 Gateway support

libfreerdp-core: fix rdg.c header include

libfreerdp-core: fix rdg.h header include

libfreerdp-core: add configurable TSG/RGB fallback, fix edge cases

Merge branch 'gateway' of https://github.com/dvincent-devolutions/FreeRDP into gateway

Merge branch 'master' of github.com:FreeRDP/FreeRDP into gateway

Merge pull request #2475 from hardening/winpr_moreops

Implement ReadFile() and WriteFile() as handle operation

Merge pull request #2478 from akallabeth/win_sw_gdi_fix

Fixed resize for software GDI.

libfreerdp-core: minor code corrections

Removed unused define.

fixed log message arguments.

Added error name and string functions.

Added new functions allowing to get a string representation of an error code
and a description of the error for connect and base error classes.

libfreerdp-core: added fallback to RPC.

libfreerdp-core: fix RDG http receive

libfreerdp-core: fix RDG valgrind issues

libfreerdp-core: fix some RDG valgrind issues

libfreerdp-core: fix rdg event handles

Merge branch 'gateway' of https://github.com/dvincent-devolutions/FreeRDP into gateway

libfreerdp-core: Initial RDG support.

Fixed resize for software GDI.

Implement ReadFile() and WriteFile() as handle operation

libfreerdp-codec: add support for 16 color palette image copy

libfreerdp-codec: add support for 1bpp image copy

Merge pull request #2474 from awakecoding/master

Test Fixes, BipBuffer, TSG Cleanup, SCardTransmit Fix

freerdp: fix failing tests

libfreerdp-gdi: disable broken BitBlt tests

Merge pull request #2473 from akallabeth/force_cmdline_flags

Force cmdline to ignore unknown keywords

Merge branch 'master' of github.com:FreeRDP/FreeRDP

Merge pull request #2469 from dupondje/master

libfreerdp-core: Initialize values correctly

Merge pull request #2472 from hardening/winpr_ops_rebased

Winpr ops rebased

Fixed the test for named pipe creation

All credits goes to @akallabeth.

Changes for upstream changes on pipe

Make handle operations static

This patch moves the handle operations in a static struct.

Added ignore unkown keyword flag to detection.

Added flag to ignore unknown keywords

freerdp_client_settings_parse_command_line now allows ignoring
unknown keywords.

libfreerdp-core: Initialize values correctly

Merge pull request #2468 from dupondje/master

fix size error introduced in commit 99b14815a53e556080f459a4998d94244803...

fix size error introduced in commit 99b14815a53e556080f459a4998d942448031f08

Merge pull request #2467 from akallabeth/named_pipe_fix

Named pipe fix

libwinpr-utils: improve BipBuffer

Proper default values for fds.

Fixed missing callbacks for named pipes.

libwinpr-utils: update BipBuffer

Merge branch 'master' of github.com:FreeRDP/FreeRDP

Merge pull request #2461 from ivan-83/master

audin OSS support, dyn channels fix, dev arg broken fix, add debug verbosity, build warning fix

Merge pull request #2462 from zhangzl2013/fix

Fix

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

libfreerdp: crypto: add certificate chain validation!