Hardening [Wed, 20 May 2015 19:19:20 +0000 (21:19 +0200)]
Merge pull request #2632 from akallabeth/client_bandwidth_measurement_callback
Added client bandwidth measurement result callback.
Armin Novak [Wed, 20 May 2015 08:47:24 +0000 (10:47 +0200)]
Added client bandwidth measurement result callback.
Marc-André Moreau [Wed, 20 May 2015 13:41:18 +0000 (09:41 -0400)]
Merge pull request #2633 from clivest/progressive_codec_new_surface_fix
Fix rounding error in progressive codec surface new
Marc-André Moreau [Wed, 20 May 2015 13:40:24 +0000 (09:40 -0400)]
Merge pull request #2635 from clivest/gdi_gfx_leaked_regions_fix
Fix two memory leaks in gdi/gfx
Clive Stevens [Wed, 20 May 2015 11:35:24 +0000 (12:35 +0100)]
Fix two cases of leaked regions in gdi/gfx
Clive Stevens [Wed, 20 May 2015 11:26:58 +0000 (12:26 +0100)]
Fix rounding error in progressive codec
The grid is composed of 64x64 blocks and should not be smaller
than the surface. If width or height were not a multiple of 64
the previous rounding resulted in a grid smaller than the surface.
Bernhard Miklautz [Tue, 19 May 2015 09:07:42 +0000 (11:07 +0200)]
Merge pull request #2627 from nfedera/fix-x11-polyregion
client/x11: fix + simplify invalidate_poly_region
Norbert Federa [Mon, 18 May 2015 23:07:51 +0000 (01:07 +0200)]
client/x11: fix + simplify invalidate_poly_region
Marc-André Moreau [Mon, 18 May 2015 12:55:48 +0000 (08:55 -0400)]
Merge pull request #2624 from kwitaszczyk/master
QueryContextAttributes shouldn't return an error when a user or a domain is empty.
Konrad Witaszczyk [Mon, 18 May 2015 07:42:17 +0000 (09:42 +0200)]
QueryContextAttributes shouldn't return an error when a user or a domain
is empty.
Hardening [Sun, 17 May 2015 19:40:38 +0000 (21:40 +0200)]
Merge pull request #2623 from nfedera/fix-xfclient-if-semicolons
client/x11: fixed if statements with empty body
Norbert Federa [Sun, 17 May 2015 16:24:11 +0000 (18:24 +0200)]
client/x11: fixed if statements with empty body
Marc-André Moreau [Fri, 15 May 2015 12:37:10 +0000 (08:37 -0400)]
Merge pull request #2622 from kwitaszczyk/master
Pass valid context to CompleteAuthToken function in Negotiate Security Package.
Konrad Witaszczyk [Fri, 15 May 2015 11:02:40 +0000 (13:02 +0200)]
Pass valid context to CompleteAuthToken function in Negotiate Security Package.
Bernhard Miklautz [Fri, 15 May 2015 09:59:12 +0000 (11:59 +0200)]
Merge pull request #2621 from nfedera/fix-mac-async-transport
transport/mfreerdp: fix async transport
Norbert Federa [Thu, 14 May 2015 19:54:09 +0000 (21:54 +0200)]
transport/mfreerdp: fix async transport
- handle WAIT_TIMEOUT result as error in async transport thread
if an INFINITE timeout was specified in WaitForMultipleObjects
- fix mfreerdp's async transport handling to not use
freerdp_get_event_handles/freerdp_check_event_handles if async
transport is activated
Hardening [Thu, 14 May 2015 19:13:12 +0000 (21:13 +0200)]
Merge pull request #2620 from nfedera/fix-gdi-cache
gdi: fix nullification of instance->context->cache
Norbert Federa [Thu, 14 May 2015 18:33:21 +0000 (20:33 +0200)]
gdi: fix nullification of instance->context->cache
Hardening [Wed, 13 May 2015 06:24:35 +0000 (08:24 +0200)]
Merge pull request #2611 from nfedera/fix-async-transport-thread
transport: fixed multiple errors in async thread
Hardening [Tue, 12 May 2015 14:18:08 +0000 (16:18 +0200)]
Merge pull request #2613 from oshogbo/gethostname
At least at FreeBSD - the gethostname(3) function can fail.
Hardening [Tue, 12 May 2015 13:45:55 +0000 (15:45 +0200)]
Merge pull request #2612 from oshogbo/style
Function return bool, so lets return FALSE if 0 or TRUE if 1.
Mariusz Zaborski [Tue, 12 May 2015 13:33:42 +0000 (15:33 +0200)]
At least at FreeBSD - the gethostname(3) function can fail.
Check returned value.
Mariusz Zaborski [Tue, 12 May 2015 13:31:23 +0000 (15:31 +0200)]
Function return bool, so lets return FALSE if 0 or TRUE if 1.
Norbert Federa [Mon, 11 May 2015 19:54:36 +0000 (21:54 +0200)]
transport: fixed multiple errors in async thread
MartinHaimberger [Mon, 11 May 2015 14:59:32 +0000 (16:59 +0200)]
Merge pull request #2605 from nfedera/fix-2015-05-08-01
fixed multiple missing gdi return value checks
MartinHaimberger [Mon, 11 May 2015 08:14:30 +0000 (10:14 +0200)]
Merge pull request #2607 from nfedera/remove_free_null_checks
free can handle NULL perfectly fine
Norbert Federa [Mon, 11 May 2015 07:07:39 +0000 (09:07 +0200)]
free can handle NULL perfectly fine
Norbert Federa [Fri, 8 May 2015 19:39:23 +0000 (21:39 +0200)]
fixed multiple missing gdi return value checks
mainly gdi_Create* functions
Bernhard Miklautz [Fri, 8 May 2015 11:44:26 +0000 (13:44 +0200)]
Merge pull request #2594 from baruchsiach/fix-math-2
Don't disable C99 long double math on non Unix platforms
Norbert Federa [Fri, 8 May 2015 09:17:33 +0000 (11:17 +0200)]
Merge pull request #2577 from hardening/listen_from_socket
Add the ability to have listener from an existing socket
David FORT [Fri, 8 May 2015 08:07:29 +0000 (10:07 +0200)]
Return FALSE under Win32
David FORT [Fri, 8 May 2015 08:05:39 +0000 (10:05 +0200)]
Test the number of listening handles
David FORT [Tue, 7 Apr 2015 13:18:07 +0000 (15:18 +0200)]
Add the ability to have listener from an existing socket
Bernhard Miklautz [Thu, 7 May 2015 22:49:41 +0000 (00:49 +0200)]
Merge pull request #2596 from akallabeth/wlog_default_out_fix
Wlog console output to stderr
Hardening [Thu, 7 May 2015 17:06:59 +0000 (19:06 +0200)]
Merge pull request #2601 from nfedera/fix-2015-05-07-01
Fix some unchecked create calls
Norbert Federa [Thu, 7 May 2015 16:42:15 +0000 (18:42 +0200)]
Merge pull request #2597 from clivest/bitstream_read_fix
Fix bitstream reading bounds check
Norbert Federa [Thu, 7 May 2015 16:24:47 +0000 (18:24 +0200)]
Merge pull request #2598 from clivest/stream_leak_fix
Fix two cases of stream resource leak
Norbert Federa [Thu, 7 May 2015 16:15:01 +0000 (18:15 +0200)]
Merge pull request #2599 from hardening/fix_wayland_colors
Fix colors with the wayland client
Norbert Federa [Thu, 7 May 2015 16:13:09 +0000 (18:13 +0200)]
Merge pull request #2603 from hardening/rail_empty_title_fix
Fix empty window title
Norbert Federa [Thu, 7 May 2015 16:11:38 +0000 (18:11 +0200)]
Merge pull request #2604 from hardening/rail_fix
Fix remoteApps
Norbert Federa [Thu, 7 May 2015 15:42:01 +0000 (17:42 +0200)]
server/shadow: added missing error message
David FORT [Thu, 7 May 2015 13:08:00 +0000 (15:08 +0200)]
Fix empty window title
When the size of a RAIL string was 0, we were reporting an OOM error.
David FORT [Thu, 7 May 2015 12:36:07 +0000 (14:36 +0200)]
Fix remoteApps
The assert that have been removed were preventing RAIL from working when you
have an intermediary screen before seeing the application.
Norbert Federa [Tue, 5 May 2015 17:45:34 +0000 (19:45 +0200)]
Fix unchecked CreateDirectory calls
Norbert Federa [Tue, 5 May 2015 15:32:06 +0000 (17:32 +0200)]
Fix unchecked CreateSemaphore calls
Norbert Federa [Tue, 5 May 2015 15:25:17 +0000 (17:25 +0200)]
Fix unchecked CreateMutex calls
David FORT [Wed, 6 May 2015 19:54:39 +0000 (21:54 +0200)]
Fix colors with the wayland client
Clive Stevens [Wed, 6 May 2015 14:54:23 +0000 (15:54 +0100)]
Fix whitespace
Clive Stevens [Wed, 6 May 2015 14:26:06 +0000 (15:26 +0100)]
Fix bitstream reading
The bounds checks where previously incorrect
Clive Stevens [Wed, 6 May 2015 14:23:37 +0000 (15:23 +0100)]
Fix two cases of potentially leaked streams
Armin Novak [Wed, 6 May 2015 13:36:54 +0000 (15:36 +0200)]
Fixed broken indent.
Armin Novak [Wed, 6 May 2015 08:58:57 +0000 (10:58 +0200)]
Not writing messages if level is WLOG_OFF.
Armin Novak [Wed, 6 May 2015 08:56:30 +0000 (10:56 +0200)]
Console logging to stdout and stderr
With this patch the default for the console logger
changes to writing to stdout for TRACE, DEBUG and INFO
messages and stderr for WARNING, ERROR and FATAL messages.
Martin Fleisz [Wed, 6 May 2015 08:16:26 +0000 (10:16 +0200)]
Merge pull request #2592 from nfedera/fix-2015-05-05-01
Fix unchecked CreateThread calls and misc fixes
Baruch Siach [Tue, 5 May 2015 18:27:17 +0000 (21:27 +0300)]
Don't disable C99 long double math on non Unix platforms
CMakeLists.txt may enable HAVE_MATH_C99_LONG_DOUBLE only on Unix platforms.
Limit the effect of this symbol accordingly.
Marc-André Moreau [Tue, 5 May 2015 14:56:12 +0000 (10:56 -0400)]
Merge pull request #2587 from floppym/armfix
Remove the ARM_FP_ABI option
Marc-André Moreau [Tue, 5 May 2015 14:47:03 +0000 (10:47 -0400)]
Merge pull request #2589 from baruchsiach/fix-math
Don't use unavailable C99 long double math functions
Norbert Federa [Tue, 5 May 2015 11:55:48 +0000 (13:55 +0200)]
Fix unchecked CreateThread calls and misc fixes
MartinHaimberger [Mon, 4 May 2015 08:14:27 +0000 (10:14 +0200)]
Merge pull request #2582 from nfedera/fix-2015-04-28-01
Fix unchecked CreateEvent calls and misc fixes
Baruch Siach [Sun, 3 May 2015 17:46:22 +0000 (20:46 +0300)]
Don't use unavailable C99 long double math functions
uClibc variants do not provide the C99 long double math functions like ceill,
powl, etc.. For future compatibility use check_symbol_exists() to check
whether these functions are available, and keep the result in
HAVE_MATH_C99_LONG_DOUBLE. Use that instead of the fragile Cygwin version
check in triodef.h.
Fixes build failures under uClibc(-ng) like:
../../libwinpr/utils/libwinpr-utils.so.0.1.0: undefined reference to `powl'
../../libwinpr/utils/libwinpr-utils.so.0.1.0: undefined reference to `fmodl'
../../libwinpr/utils/libwinpr-utils.so.0.1.0: undefined reference to `ceill'
../../libwinpr/utils/libwinpr-utils.so.0.1.0: undefined reference to `log10l'
../../libwinpr/utils/libwinpr-utils.so.0.1.0: undefined reference to `floorl'
collect2: error: ld returned 1 exit status
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
Mike Gilbert [Fri, 1 May 2015 21:35:21 +0000 (17:35 -0400)]
Remove the ARM_FP_ABI option
It's unclear why this option would be necessary, and it causes problems
when people do not match it to their toolchain and CFLAGS.
To set the float abi, either use a toolchain with an appropriate default
or set the float-abi option in the CFLAGS environment variable.
This should resolve #2586.
Bernhard Miklautz [Thu, 30 Apr 2015 12:20:33 +0000 (14:20 +0200)]
Merge pull request #2584 from baruchsiach/fix-eventfd-2
Fix build with newer uclibc versions
Norbert Federa [Thu, 30 Apr 2015 07:20:34 +0000 (09:20 +0200)]
Increase API version to 1.2.3
Baruch Siach [Tue, 28 Apr 2015 06:30:45 +0000 (09:30 +0300)]
Fix build with newer uclibc versions
uClibc master branch, as well as uclibc-ng add eventfd_{read,write}
definitions. Instead of testing for __UCLIBC__, have cmake explicitly check
for the existence of eventfd_read and save the result in
WITH_EVENTFD_READ_WRITE.
Fixes build errors like:
.../winpr/libwinpr/synch/event.c:120:12: error: static declaration of 'eventfd_read' follows non-static declaration
static int eventfd_read(int fd, eventfd_t* value)
^
In file included from .../winpr/libwinpr/synch/event.c:39:0:
.../usr/include/sys/eventfd.h:37:12: note: previous declaration of 'eventfd_read' was here
extern int eventfd_read (int __fd, eventfd_t *__value);
^
.../winpr/libwinpr/synch/event.c:125:12: error: static declaration of 'eventfd_write' follows non-static declaration
static int eventfd_write(int fd, eventfd_t value)
^
In file included from .../winpr/libwinpr/synch/event.c:39:0:
.../usr/include/sys/eventfd.h:40:12: note: previous declaration of 'eventfd_write' was here
extern int eventfd_write (int __fd, eventfd_t __value);
Norbert Federa [Tue, 28 Apr 2015 15:00:41 +0000 (17:00 +0200)]
Fix unchecked CreateEvent calls and misc fixes
1)
Added missing checks for CreateEvent which also required the
following related changes:
- changed freerdp_context_new API to BOOL
- changed freerdp_peer_context_new API to BOOL
- changed pRdpClientNew callback to BOOL
- changed pContextNew callback to BOOL
- changed psPeerAccepted callback to BOOL
- changed psPeerContextNew callback to BOOL
2)
Fixed lots of missing alloc and error checks in the
changed code's neighbourhood.
3)
Check freerdp_client_codecs_prepare result to avoid segfaults
caused by using non-initialized codecs.
4)
Fixed deadlocks in x11 caused by missing xf_unlock_x11() calls
in some error handlers
5)
Some fixes in thread pool:
- DEFAULT_POOL assignment did not match TP_POOL definition
- don't free the pool pointer if it points to the static DEFAULT_POOL
- added error handling and cleanup in InitializeThreadpool
Norbert Federa [Wed, 29 Apr 2015 16:04:59 +0000 (18:04 +0200)]
Merge pull request #2579 from akallabeth/mutex_debug_log
winpr mutex unlock on destroy
Armin Novak [Wed, 29 Apr 2015 06:48:07 +0000 (08:48 +0200)]
Added EBUSY to success states.
Armin Novak [Tue, 28 Apr 2015 15:29:07 +0000 (17:29 +0200)]
Better error handling on mutex destroy.
Hardening [Tue, 28 Apr 2015 12:43:30 +0000 (14:43 +0200)]
Merge pull request #2578 from jonathankap/master
Fix issue when numWindowIds is 0
Armin Novak [Tue, 28 Apr 2015 07:42:46 +0000 (09:42 +0200)]
Ensuring mutex is unlocked on destroy.
Jonathan Kaplan [Tue, 28 Apr 2015 07:21:38 +0000 (00:21 -0700)]
Fix issue when numWindowIds is 0
Armin Novak [Tue, 28 Apr 2015 06:55:26 +0000 (08:55 +0200)]
Added mutex debug flag.
When mutex debugging is enabled now a stack trace
is logged, if a mutex is locked on destruction.
Norbert Federa [Mon, 27 Apr 2015 07:34:13 +0000 (09:34 +0200)]
Merge pull request #2575 from bmiklautz/gdi_fix
gdi: fix problem with gdi_SelectObject check(s)
Bernhard Miklautz [Sun, 26 Apr 2015 20:28:49 +0000 (22:28 +0200)]
gdi: fix problem with gdi_SelectObject check(s)
When a hdc is created no initial or default objects are created
therefore can the first call of gdi_SelectObject return NULL.
Because of this checking the return value of gdi_SelectObject failed
for newly create hdc causing errors (disconnects).
Since all types of HGDIOBJECT are handled and the return value of
gdi_SelectObject isn't used the recently added checks were removed
again.
Marc-André Moreau [Fri, 24 Apr 2015 17:36:27 +0000 (13:36 -0400)]
Merge pull request #2568 from nfedera/fix-2015-04-23-01
codec/rfx: error checking and various fixes
Norbert Federa [Fri, 24 Apr 2015 15:54:49 +0000 (17:54 +0200)]
codec/rfx: simplification, segfault/malloc fixes
rfx_process_message_sync:
- simplified the check if the header messages got processed
rfx_process_message_tileset:
- ObjectPool_Take result was not checked
- fail if TS_RFX_TILE block type is not CBT_TILE
- CreateThreadpoolWork result was not checked
- post decoding loop code segfaulted in error case
rfx_decoder_tile_new:
- missing malloc check
rfx_message_free:
- segfault protection
rfx_write_message_tileset:
- segfault protection
Bernhard Miklautz [Fri, 24 Apr 2015 15:38:02 +0000 (17:38 +0200)]
Merge pull request #2572 from bceverly/OpenBSD-2015-04-24
Added support for manpage construction for OpenBSD
Bryan Everly [Fri, 24 Apr 2015 12:38:35 +0000 (08:38 -0400)]
Added support for manpage construction for OpenBSD
Norbert Federa [Thu, 23 Apr 2015 13:42:21 +0000 (15:42 +0200)]
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
Bernhard Miklautz [Wed, 22 Apr 2015 16:44:25 +0000 (18:44 +0200)]
Merge pull request #2566 from bceverly/OpenBSD-2015-04-22
Further cleanup on the OpenBSD build
Bryan Everly [Wed, 22 Apr 2015 13:50:50 +0000 (09:50 -0400)]
Removed compiler warning about obsolete header file in OpenBSD
Bryan Everly [Wed, 22 Apr 2015 13:19:04 +0000 (09:19 -0400)]
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.
Bernhard Miklautz [Wed, 22 Apr 2015 12:42:31 +0000 (14:42 +0200)]
Merge pull request #2564 from akallabeth/pthread_mutex_timedjoin_detection
Fixed pthread_mutex_timedjoin detection.
Hardening [Wed, 22 Apr 2015 11:33:12 +0000 (13:33 +0200)]
Merge pull request #2555 from akallabeth/wayland_warning_fix
Updated wayland client to new API.
Armin Novak [Wed, 22 Apr 2015 09:05:29 +0000 (11:05 +0200)]
Updated to new freerdp_get_event_handles API.
Armin Novak [Mon, 20 Apr 2015 14:20:16 +0000 (16:20 +0200)]
Updated wayland client to new API.
Norbert Federa [Wed, 22 Apr 2015 09:04:37 +0000 (11:04 +0200)]
Merge pull request #2554 from akallabeth/event_fix
Fix for transport event reset
Hardening [Wed, 22 Apr 2015 09:01:27 +0000 (11:01 +0200)]
Merge pull request #2544 from bmiklautz/boolenization
change return types of callbacks to BOOL
Bernhard Miklautz [Wed, 22 Apr 2015 08:23:25 +0000 (10:23 +0200)]
Increase API version to 1.2.2
Armin Novak [Wed, 22 Apr 2015 07:30:50 +0000 (09:30 +0200)]
Fixed pthread_mutex_timedjoin detection.
Hardening [Wed, 22 Apr 2015 07:14:29 +0000 (09:14 +0200)]
Merge pull request #2562 from bceverly/OpenBSD-2015-04-11
Open bsd 2015 04 11
Bryan Everly [Tue, 21 Apr 2015 20:14:56 +0000 (16:14 -0400)]
Modified code to use bulitin pthread functions on OpenBSD
Bryan Everly [Tue, 21 Apr 2015 18:44:02 +0000 (14:44 -0400)]
Necessary changes to get latest branch working on OpenBSD 5.6
Bryan Everly [Tue, 21 Apr 2015 18:42:06 +0000 (14:42 -0400)]
Necessary changes to get latest branch working on OpenBSD 5.6
Bernhard Miklautz [Tue, 21 Apr 2015 14:52:32 +0000 (16:52 +0200)]
Merge pull request #2537 from MartinHaimberger/mh-autodetect-fix
autodetect: debug fixed and level adjusted
Bernhard Miklautz [Tue, 21 Apr 2015 14:49:02 +0000 (16:49 +0200)]
Merge pull request #2553 from akallabeth/leak_fixes
Leak fixes
Bernhard Miklautz [Tue, 21 Apr 2015 14:34:39 +0000 (16:34 +0200)]
Merge pull request #2560 from akallabeth/get_event_handle_api_fix
Add count to ```*_get_event_handles``` function
Armin Novak [Tue, 21 Apr 2015 13:57:25 +0000 (15:57 +0200)]
Changed listener callback arguments and return.
Armin Novak [Tue, 21 Apr 2015 13:10:17 +0000 (15:10 +0200)]
Fixed handle copy and index.
Bernhard Miklautz [Tue, 21 Apr 2015 12:25:59 +0000 (14:25 +0200)]
TestStream: set function parameters to void