Better description and formatting for update_read_icon_info

Made all internal functions static.

Fixed color conversion, log level and buffer types

Fixed rail_string_to_unicode_string size calculation.

xfreerdp: set _NET_WM_ICON to RAIL app icon

Icons on X11 windows are configured using the _NET_WM_ICON property
described in Extended Window Manager Hints. Here we implement converison
from DIB bitmaps used by RAIL to the format expected by _NET_WM_ICON,
and actually set the icon for RAIL app windows.

Both DIB format and _NET_WM_ICON (or rather, Xlib) are weird. Let's
start with RAIL's format. That's the one used in BMP and ICO formats
on Windows. It has some strange properties but thankfully FreeRDP's
freerdp_image_copy() can handle most of them for us. (With an exception
of monochrome and 16-color formats that it does not support. Sorry, but
I'm too lazy to fix them. They are not seem to be used by any real
application either.) The one thing that it can't do is to apply the
alpha transparency bitmask so we have to do it manually. This instantly
reminds us that DIB format has HISTORY: it's vertically flipped and
each must be padded to 4 bytes. Both these quirks having reasonable
(for a certain definition of 'reason') explanations. Such is life.
(Also, 8-bit images require a color palette which we must fill in.)

So okay, now comes _NET_WM_ICON. It is more sane (or rather, easier to
deal with). The bitmap is represented with a tiny [width, height] header
followed by an array of pixels in ARGB format. There is no padding, no
weird color formats. But here's a catch: you can't simply take the
output of freerdp_image_copy() and cast to (unsigned char*) of colors.
We have to allocate an array of C's longs and copy the pixels there,
because that's what Xlib expects (and this is mentioned in the spec).
Simply casting an array of bytes causes crashes on 64-bit systems.
So don't try to cheat or "optimize" and read the docs, kids.

Note that XFlush() call after XChangeProperty(). It's there because it
seems to helps see the icon quicker with Unity on Ubuntu 14.04. I don't
know why. (And Unity does not support _NET_WM_ICON officially. But it
sorta kinda works sometimes.)

Oh, and while we're here, delete some old, unused, and commented out
code that was setting window icons in the past. It's not needed anymore.

xfreerdp: add RAIL icon cache

This commit adds a cache for RAIL application icons. It is (surpisingly)
used to cache icons for remote applications. This mechanism is described
in MS-RDPERP 3.1.1.2 Icon Cache Support and related items.

Note that some (actually, most) of the icons are not cached. These are
marked with CacheId == 0xFF. In order to keep the code clean we do not
introduce a special case and instead use a 'scratch' icon and simply
pretend to return an xfRailIcon from cache.

We're going to set icons via _NET_WM_ICON property which explains
why we use "long" values to store pixel data. The icon conversion
is not implemented in this commit, it's only stubs.

However, we do implement processing of window information orders that
contain new icons or cached references to previously sent icons. Note
that it is important to not fail (i.e., to not return FALSE) if we
cannot find a window for the icon by its ID. The server occasionally
likes to be slow or something and send icon updates for nonexistent
windows. This behavior is mandated by the spec, too:

    MS-RDPERP 3.2.5.1.6 Processing Window Information Orders

    Upon receipt of a Window Information Order for an icon or
    cached icon, as specified in sections 2.2.1.3.1.2.2 and
    2.2.1.3.1.2.3, the client SHOULD locate the local RAIL window
    that corresponds to the WindowId reported in the Hdr field
    and apply the icon updates to the RAIL window. If no such
    window can be found, the client SHOULD ignore the order.

Indeed, we silently ignore such orders now.

libfreerdp-core: fix reading TS_ICON_INFO

The spec says that CbColorTable field is present when Bpp is 1, 4, 8.
Actually, bpp == 2 is not supported by TS_ICON_INFO according to the
spec (though, DIB definitely supports 16-color images).

    MS-RDPERP 2.2.1.2.3 Icon Info (TS_ICON_INFO)

    CbColorTable (2 bytes):
        This field is ONLY present if the bits per pixel (Bpp)
        value is 1, 4, or 8.

Omitting 8-bit value breaks 256-color icons which are incorrectly
read with color and alpha data mixed up.

Merge pull request #5005 from bmiklautz/nightly_build

fix [rpm/freerdp-nightly]: build dependencies

fix [rpm/freerdp-nightly]: build dependencies

Depend on libswresample-devel not libavresample-dev.
This reverts the behavior introduced in
fbe95209e72265af07dd426e0def7e3a9082fb4e.

Merge pull request #4997 from akallabeth/use_bio_free_all

Replaced BIO_free with BIO_free_all

Merge pull request #4960 from akallabeth/interleaved_fix

Fixed #4954: Check destination buffer bounds.

Make VS2010 happy, reworked UNROLL defines.

Unroll bBits loops as well.

Merge pull request #4968 from akallabeth/timezone_updates

Simplified timezone update handling:

Merge pull request #4961 from akallabeth/align_malloc_fix

Fixed #2039: Check for overflow in calculations.

Readded loop unrolling.

Fixed #2039: Check for overflow in calculations.

Fixed profiler naming in tests.

Fixed various issues with freerdp_bitmap_compress and interleaved_compress

Added a unit test for interleaved codec compression.

Merge pull request #4996 from bmiklautz/tls_security_level

new [crypto/tls]: add support to set tls security level

Replaced BIO_free with BIO_free_all

There is no point in using BIO_free with a custom recursion
to free up stacked BIOs if there is already BIO_free_all.
Using it consistently avoids memory leaks due to stacked BIOs
not being recursively freed.

fix [cmdline]: parameter parsing

use strtoul instead of strtol

new [crypto/tls]: add support to set tls security level

The newly introduced option /tls-seclevel can be used to set the tls
security level on systems with openssl >= 1.1.0 or libressl.
As default level 1 is used as higher levels might prohibit connections
to older systems.

Merge pull request #4994 from akallabeth/bio_uninit_fix

Fixed #4993: Allow uninitialized bio socket in transport_bio_simple_uninit

Fixed #4993: Allow uninitialized bio socket in transport_bio_simple_uninit

When the TLS connection is aborted early (certificate issues, ...)
the bio simple socket might not have been initialized.
Do not access the NULL pointer on cleanup.

Merge pull request #4982 from bmiklautz/ctest_asan_fixes

Fix issues with multiple tests if address sanitation is on

Simplified timezone update handling:

* Fix #4965: Adjusted the timezone update script to initialize
  structs properly.
* Updated the scripts to generate the files directly
* Added a small HOWTO
* Split zone information to separate files to allow automatic
  update without further manual editing.

Create a timezone.c file that does conform to C rules.

Simplified code, moved bounds check out of loops.

Fixed #4954: Check destination buffer bounds.

Merge pull request #4985 from informatimago/issue-4983

Fix Issue #4983 : Increased size of buffer to encrypt hash with kerbe…

Fix Issue #4983 : Increased size of buffer to encrypt hash with kerberos.

fix [libfreerdp/codec]: heap buffer overflow in TestFreeRDPCodecClear

The examples and the allocated buffers assume that the destination
starts with x 0 and y 0 and not 1/1.

fix [libfreerdp/crypto]: memory leak in Test_x509_cert_info

fix [winpr/util]: memory leak in TestCmdLine

Merge pull request #4969 from akallabeth/covscan_fix

Covscan fixes

Merge pull request #4980 from hardening/logon_infov2_fix

fix uninitialised variable

fix uninitialised variable

Otherwise we can run in troubles when sending logon_info_v2 packets

Merge pull request #4974 from Devolutions/android_timezone_fix

Remove unsupported call to fseek when file is opened with popen

Merge pull request #4977 from hardening/nego_flags

nego: add new flags

nego: add new flags

Reading the new spec there's some new flags

remove unsupported call to fseek when file is opened with popen

Merge pull request #4907 from akallabeth/transport_write_leak_fix

Transport write leak fix

Merge pull request #4962 from akallabeth/wextra_fixes

Wextra fixes

Merge pull request #4970 from hardening/uwac_fixes

uwac: change seat API to not reference disposed UwacSeat

Fixed warnings in Stream test.

Fixed casts in Data_Read macros.

uwac: change seat API to not reference disposed UwacSeat

This is a false positive returned by covscan, but that doesn't hurt to fix it.

Fixed signedness casts.

Fixed signedness of MoniorCount

Fixed sockfd/socketBio resource leak.

Fixed sockfd/socketBio resource leak.

Silence code analyzers, check correct variable for NULL.

Fixed sockfd/socketBio resource leak.

Fixed resource leak in rpc_ntlm_http_request

Fixed initialisation in DEFINE_EVENT_ENTRY define

Fixed another bunch of warnings.

Fixed -Wextra sign warnings

Fixed warnings of uninitialized struct values.

Fixed cast warnings for thread wrapper

Changed size iterator variables to size_t

Fixed default return value for fastpath_send_multiple_input_pdu

Merge pull request #4947 from akallabeth/http_gw_fix

Fixed http gateway body length read.

Merge pull request #4958 from nfedera/nf-fix-alsapulse-wrn-err

channels/client: fix alsa/pulse warning and checks

channels/client: fix alsa/pulse warning and checks

- pa_stream_writable_size() returns (size_t) -1 on error
- fix missing cast in alsa client sound channel code

Merge pull request #4957 from akallabeth/wave_2_pdu_fix

Wave 2 pdu fix

Merge pull request #4932 from hardening/stream_static

winpr: add some functions to use wStream in a static way

Cleaned up resource cleanup for rdpsnd alsa backend.

Merge pull request #4796 from akallabeth/groegera_fixes

Update X11-modifier key state on focus_in

Fixed memory leak in update_recv_surfcmd_surface_bits

Implemented rdpsnd_pulse_get_volume

fixed return value check in rdpsnd_recv_volume_pdu

Seal length in wave pdu alignment.

Align wave pdu size

Added missing length check and log message information.

Improved debug logging for sound device handling.

Removed unnecessary pulse context check.

Unified sound device reopen in a single function.

Merge pull request #4952 from eledoux/master

gateway HTTP transport - fixed partial reads

Merge pull request #4951 from nfedera/nf-fix-fnobjnew-fnstylecasts

fix issue with fnObjectFree and related casts

Merge pull request #4949 from bmiklautz/floatbar_warning

fix [x11]: compiler warnings in floatbar

Merge pull request #4950 from bmiklautz/cleanup_pr_4581p2

fix [cache/pointer]: add missing callback checks

Read byte by byte, the alignment may otherwise be broken.

Read http request in 4byte chunks until '\r\n\r\n' is found

Avoid reading too much data in a single call to BIO_read as some
implementations may return a lot more data than is part of the response.

fixed partial reads

Properly parse ContentType to find length.

fix [x11]: compiler warnings in floatbar

client/X11/xf_floatbar.c:530:27: warning: use of logical '&&' with constant
      operand [-Wconstant-logical-operand]
        if (event->xmotion.state && Button1Mask && mode > XF_FLOATBAR_MODE_DRAGGING)
                                 ^  ~~~~~~~~~~~
client/X11/xf_floatbar.c:534:32: warning: use of logical '&&' with constant
      operand [-Wconstant-logical-operand]
        else if (event->xmotion.state && Button1Mask && mode == XF_FLOATBAR_MODE_DRAGGING)
                                      ^  ~~~~~~~~~~~
clang 8.0.0-svn344413-1~exp1+0~20181012203207.819~1.gbpc91f27

fix issue with fnObjectFree and related casts

- remove unnecessary/dangerous OBJECT_xxx function-style casts
- fix -Wstrict-prototypes issue with OBJECT_NEW_FN definition

fix [cache/pointer]: add missing callback checks

Merge pull request #4948 from bmiklautz/cleanup_pr_4581

new [cache/pointer]: don't require pointer functions to be set

new [cache/pointer]: don't require pointer functions to be set

This as cleanup/rework of PR #4581.

Fixed http gateway body length read.

Merge pull request #4946 from mmattes/fix/floatbar_app_check_missing

added missing check for remote app where no floatbar exists

Merge pull request #4945 from mmattes/fix/transferEncoding

transferEncoding can be NULL, correct if happens in line 348

added missing check for remote app where no floatbar exists

Merge pull request #4942 from akallabeth/rail_fixes_2.0

Fix ORDER_TYPE_WINDOW reading

Merge pull request #4941 from nfedera/nf-remove-stale-valgrind-macro

gateway/rpc: remove stale valgrind macro

Merge pull request #4938 from akallabeth/warning_fixes_master

Warning fixes and TSG abort condition fix

transferEncoding can be NULL, verification happens in line 348

Fixed read_altsec_order ORDER_TYPE_WINDOW