Merge pull request #2553 from akallabeth/leak_fixes

Leak fixes

Merge pull request #2560 from akallabeth/get_event_handle_api_fix

Add count to ```*_get_event_handles``` function

Changed listener callback arguments and return.

Fixed handle copy and index.

Fixed some type mismatches.

Add *get_event_handles handle count, return check.

Modified GetEventHandles of listener

Now using nCount as in and out argument.
When called, set nCount to the number of available handles.
This value is checked and an error returned, if not enough
handles are available.

Merge pull request #2558 from llyzs/llyzs

h264: change encoder api and add multi-thread support.

h264: change encoder api and add multi-thread support.

Merge pull request #2538 from MartinHaimberger/mh-add-transport-get-bytes

transport: added function to get bytes written

Merge pull request #2552 from akallabeth/sample_update

Sample update

Fixed reading bitmap from buffer.

Added proper bitmap and png image test.

Fixed bitmap reading bug.

Using new API for socket listener.

Moved client handler to new API.

Moved cleanup code, removed unused.

Updated sample for new API.

Merge pull request #2547 from nfedera/fix-2015-04-16-01

Fullscreen fixes and smart sizing resuscitation

xfreerdp: fixed initial smart-sizing dimensions

The window width/height variables and in turn SmartSizingWidth/Height
get automatically updated in th the configure notify event handler.
If the window is created initially the values don't differ from the
event values and therefore SmartSizing was not applied until the
window was resized.

Fixed NULL dereference.

Fixed NULL dereference and double free

Fixed NULL dereference

Fixed memory leaks and NULL dereferences.

xfreerdp: FS fixes & smart-sizing resuscitation

- make smart-sizing work again which was killed in previous commits
- removed several unnecessary/ugly workarounds
- miscellaneous small fixes
- new feature: restore previous window position when toggling out of
  fullscreen mode
- new feature: if /f is specified in combination with /smart-sizing:WxH
  we run the session in the /smart-sizing dimensions scaled to full screen

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

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