GW RPC-HTTP login fix

* f469e069dc8287f6fcfe0741961022e0cc50170f introduced a logic flip
  that broke RPC-HTTP Gateway.  Fix that.

Merge pull request #2522 from realjiangms/fix_WriteLock_leak

transport_write sometimes returns directly without cleanup.

transport_write sometimes returns directly without cleanup. Fix code path to always do cleanup job.

Merge pull request #2513 from hardening/license_and_security_retValues

License and security ret values

Merge pull request #2512 from hardening/help2353

Kill more exit() occurences

Take in account @nfedera's comments

Fix a too big Stream_EnsureRemainingCapacity()

Merge pull request #2516 from bmiklautz/cmd_fix

Fix regression introduced in 0b7f9d4

Fix regression introduced in 0b7f9d4

This fixes problems with command line parsing.

Fix a typo in server-side code

Fix the error case in mcs_send_connect_response()

Fix code style

Drop the limit on key size

Fixed missing return value

Correctly handle HMAC_Init_ex() for older versions of OpenSSL

Treat return values in license.c

This patch changes functions that should not return void and also treat the
callers of these functions.

Complement gitignore files with new generated files

Treat return values for security.c

This patch make functions in security.c return values when they should instead of
beeing void. And it also fix the callers of these functions.

Kill more exit() occurences

In general we should not have any exit() call in libfreerdp.

Merge pull request #2487 from mfleisz/nla_fix

Fix server side NLA

Merge pull request #2509 from bmiklautz/security_fixes_v2

NULL dereference fixes

Merge pull request #2505 from bmiklautz/tls_shutdown

rename and update tls_disconnect

wlog: add missing check

Integrate pull request feedback

Removed scopes.

Removed duplicated NULL pointer checks.

Corrected end of string checks.

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.

rename and update tls_disconnect

tls_disconnect shut down the ssl stream but didn't inform
the BIO(s) about this therefore could happen that a second shut down
was initiated (e.g. in bio_rdp_tls_free) causing rather long delays.

After removing the shut down from tls_disconnect the only thing the
function does is to prepare/send an alert therefore it was renamed to
tls_send_alert.

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.

Fix server side NLA

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.