codec/rfx: error checking and various fixes

- removed some unneeded null checks for free()
- fixed a memory leak in shadow_client
- removed rfx_compose_message_header from API

Changed the following functions to BOOL, check the result
where they are called and handle failures:
- rfx_compose_message
- rfx_compose_message_header
- rfx_write_tile
- rfx_write_message_tileset
- rfx_write_message_frame_begin
- rfx_write_message_region
- rfx_write_message_frame_end
- rfx_write_message

rfx_process_message:
- check memory allocation failures
- verify protocol-conform order of data messages to prevents memory
  leaks caused by repeated allocations
- verify that header messages were parsed/received before the
  data messages
- treat unknown rlgr mode as error
- fixed/added error handling
- fixed all callers to check/handle result

rfx_encode_message:
- fixed incorrect usage of realloc
- missing malloc check
- missing check of CreateThreadpoolWork
- correct cleanup on failure (threadpool, memory)
- check rfx_encode_message result

rfx_encode_messages:
- check rfx_split_message result
- correct cleanup on failure
- prevent memory leak on failure

rfx_write_message_context:
- fixed invalid channelId value (must be 0xFF for WBT_CONTEXT)

rfx_process_message_codec_versions:
- fixed invalid read size of codec_version (it is 16bit)

rfx_process_message_channels:
- verify protocol conform channelId value

rfx_process_message_region:
- replaced invalid reallocs with malloc
- read and verify regionType and numTileSets from stream

rfx_process_message_tileset:
- check allocation results
- fixed incorrect usages of realloc

setupWorkers:
- fixed incorrect usages of realloc

rfx_split_message:
- removed dead code
- missing malloc check

rfx_compose_message:
- fixed a memory leak
- check/handle rfx_encode_message result

Merge pull request #2566 from bceverly/OpenBSD-2015-04-22

Further cleanup on the OpenBSD build

Removed compiler warning about obsolete header file in OpenBSD

Moved declaration of OPENBSD to higher in the file to clean some things up.  Also, removed -Wredundant-decls from OpenBSD because the OpenBSD standard include files have some redundant declarations that generate a large number of compiler warnings.

Merge pull request #2564 from akallabeth/pthread_mutex_timedjoin_detection

Fixed pthread_mutex_timedjoin detection.

Merge pull request #2555 from akallabeth/wayland_warning_fix

Updated wayland client to new API.

Updated to new freerdp_get_event_handles API.

Updated wayland client to new API.

Merge pull request #2554 from akallabeth/event_fix

Fix for transport event reset

Merge pull request #2544 from bmiklautz/boolenization

change return types of callbacks to BOOL

Increase API version to 1.2.2

Fixed pthread_mutex_timedjoin detection.

Merge pull request #2562 from bceverly/OpenBSD-2015-04-11

Open bsd 2015 04 11

Modified code to use bulitin pthread functions on OpenBSD

Necessary changes to get latest branch working on OpenBSD 5.6

Necessary changes to get latest branch working on OpenBSD 5.6

Merge pull request #2537 from MartinHaimberger/mh-autodetect-fix

autodetect: debug fixed and level adjusted

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.

TestStream: set function parameters to void

Fix formating

gdi: fix broken return type checks

Integrates comments from pull request.

Fix GDI return values and other fixes

* top level GDI functions return 0 on error and != 0 otherwise but the
  low level functions (16bpp.c, 8bpp.c 32bpp.c) which are called did it
exactly the other way around. Those were adapted.
* change gdi_InvalidateRegion to BOOL and check calls where appropriate
* integrate comments from pull request

change return types of callbacks to BOOL

* change all client/server callbacks to BOOL
* update all clients accordingly
* add multiple return value checks
* small fixes

TestStream: fix compiler warning

Remove unnecessary argument from read function call.

ios: updates to build with xcode6+

* update tool chain to use clang instead of gcc
* add compatibility function for fwrite

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.

Using auto reset event now.

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

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.