xfreerdp: removed some weird/unnecessary code

xfreerdp: removed desktopWidth/desktopHeight vars

These were basically unused (from the contexts's point of view)
and only served as temporary storage

xfreerdp: removed unused variables from xf_monitor

xfreerdp: rename xf_context's width height vars

Since several contributers in the past repeatedly made the error
to treat xfc->width and xfc->height as equal with the x11 window
width and heigth I've renamed these variables to sessionWidth and
sessionHeight.

Merge pull request #2546 from akallabeth/error_fixes

Fixed some warnings

Fixed uninitialized return value.

Fixed NULL dereference.

Merge pull request #2532 from dbungert/4bytestub

GW RPC-HTTP - 2 fixes

Merge pull request #2539 from bmiklautz/fix/regression

ObjectPool: fix regression

ObjectPool: fix regression

winpr allocation checks introduced a problem that lead to zero size
allocations in ObjectPool

autodetect: debug fixed and level adjusted

Fixed two debug statements and moved
to TRACE debug level

transport: added function to get bytes written

added function to get written bytes on transport.
This is needed to get a more accurate bandwidth
management.

Merge pull request #2517 from bmiklautz/alloc

winpr allocation checks

Merge pull request #2527 from AMV007/patch-1

fixed undefined behaviour of sprintf, buffer overflow, crash

Merge pull request #2535 from akallabeth/null_dereference_fix

Fixed NULL dereference.

Fixed broken pad setting.

Update urbdrc_main.c

GW RPC-HTTP - Don't drop 4 byte stub from the pipe

* Though not frequent, it's possible to get TsProxySetupReceivePipe
  data of stublength 4 that is actual data.  This happens when
  header->common.call_id == rpc->PipeCallId &&
  !(header->common.pfc_flags & PFC_LAST_FRAG).
  This should address GW disconnects that manifest as SSL read errors.

GW RPC-HTTP login fix

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

fixed buffer overflow at sprintf

fixed undefined behaviour of sprintf, buffer overflow

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.

winpr: small fixes and cleanups

Update the pull request and integrate the latest comments and
suggestions.

* TestLibrary*: fix typo in error message
* TestPipeCreateNamedPipeOverlapped: free possibly allocated memory
* smartcard_pcsc.c: format fix
* process.c: add missing NULL check
* MessageQueue.c: delete possibly initialized critical section on error

winpr: fixes and cleanup

Integrated notes and issues for the pending pull request
* wf_info:
 * cleanup: remove unnecessary breaks
 * fix typo
 * change usage of calloc
* print.c: fix incorrect check
* makecert.c: cleanup includes

winpr: add checks for *alloc

Add missing checks if memory allocation was successful. Also adapt
caller(s) when possible.

comm/tests: set label comm

Set the comm label to all comm related test that they can be excluded
more easily.

aligned memory: fixes in _aligned_offset_recalloc

* _aligned_offset_recalloc did ignore the nmemb element
  therefore only *one* element was allocated
* in case memblock was NULL the allocated memory
  wasn't zeroed
* restructure realloc and recalloc to check if the memory was initially
  created aligned before allocating a new segment

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