Fix unsafe format strings to the logging functions

Don't shove a pointer into a 32-bit variable

Fix bad initialiser lists

Compiler doesn't like us not checking return values here

Compiler doesn't like referencing unset variables

LPARAM and WPARAM have different types on Win32 and Win64

Solve it by casting to the larger of the two, i.e. 64-bit.

WIN64 should come from the compiler or SDK, not us

Handler still needs to return something even with XDamage

No need to force modifiable strings here

Pointer truncation/expansion needs to be very explicit

Wrong array index in CPU usage calculation on Windows

Windows doesn't have %lld

overrun() needs to return the available space

Move things around slightly so the compiler can track cc

Give CharArray a printf style method to ease automatic allocation

The PAM callback is slightly different on Solaris

Add extra parentheses to avoid and/or priority ambiguity

Reorder class initialiser list to match what actually happens

Make Rect::area() signed, just like Rect::width()/height()

Makes it more sane what type to use when interacting with this class.

Fix mismatches between format string and arguments

Shorten stats from EncodeManager using SI/IEC prefixes

Also avoids %lld which isn't supported on Windows.

A switch() needs to handle every value in an enum

Switch to unsigned parameters for ExtendedDesktopSize handler

The constants used here are unsigned so it makes more sense that
the parameters are as well.

Fix bad signed/unsigned comparisons

Either by casting, or switching to a more appropriate type
for the variable.

Make sure Exceptions do not use unsafe format strings

Tag the Exception constructor as a printf function

This allows the compiler to analyse the format strings and
arguments for correctness.

Remove unused variables

Handle no useful address types from getaddrinfo()

The compiler cannot figure out if this is assigned properly

The logic is a bit complex so we're going to have to silence the
warning with this explicit assignment.

Turn on more warnings to catch potential bugs

Merge pull request #142 from bphinz/master

Updates to package builds

Build local static libpng on el5

The system libpng is too old for fltk 1.3.3, resulting in undefined reference errors when trying to link vncviewer.

Remove obsolete distro patches

Commit f0fffd4d7f0907e91dfcee02cd7fbe4f4cbe2342 makes ldnow patch unnecessary.
Commit 0dc36a501ee5e69ce1a201935a4a4a903d06cbe1 makes format-security patch unnecessary.

Use autotools to build fltk-1.3.3 on el5 instead of cmake

Static libraries are too hard to get linked correctly with cmake.

Bump version of fltk to 1.3.3 for el5, el6, & ubuntu packages

Merge pull request #141 from bphinz/java

Don't violate RFB spec by sending UTF-8

Don't violate RFB spec by sending UTF-8

Java viewer was violating the RFB specification by sending UTF-8 encoded clipboard text rather than ISO-8859-1.

Merge branch 'perf' of https://github.com/dcommander/tigervnc

Merge branches 'format-security' and 'module-ldnow-master' of https://github.com/twaugh/tigervnc

encperf, decperf: catch fopen() errors
This prevents unhandled exceptions if the file isn't found.

Link libvnc.so with '-z now' to make symbol resolution failures obvious.

Merge branch 'cmdline-crash' of https://github.com/twaugh/tigervnc

Merge branch 'xserver117-master' of https://github.com/twaugh/tigervnc

Merge branch 'perf' of https://github.com/dcommander/tigervnc

encperf: Optionally disable pixel translation
For 16-bit and 8-bit datasets, you can now run encperf with an argument of
-translate=0 to benchmark just the encoder without converting the datasets to
24-bit.  This allows the output of encperf to be directly compared with that
of the TurboVNC Benchmark Tools.

encperf: change % to %% in printf format
GCC lets this slide, but other compilers (clang) complain about using % rather
than %% to print a literal % character.

encperf: add compression level to encoding list
Explicitly add the compression level to the list of encodings so it's obvious
how to override it.

encperf: allow # of iterations to be specified

encperf: report encoded & raw-equivalent bytes
Report encoded and raw-equivalent bytes as well as compression ratio (I like
to have these statistics so I can compute overall compression ratios for
groups of datasets in a spreadsheet.)

encperf: Code formatting tweaks
Code formatting tweaks to improve readability & make the code consistent with
the rest of TigerVNC

Check for out-of-memory when installing color map.

Also remove Xalloc/Xfree definitions, using malloc/free directly
instead. Note that vfbAllocateFramebufferMemory() does not need to
check for failed allocations as it is the caller's responsibility to
do so (and they do).

The legacy OS X keyboard API is only 32-bit

Don't need to log local port when handling -via (#126).

Updated to build against xorg-x11-server 1.17.x.

Don't return without value here.

In particular, return 0 as this is a failure.

Escape FLTK menu entries

We don't want it automatically creating submenus when least
expected.

Revert "Switch to FLTK's copy() method for menus"

This reverts commit e95412569bd45ec3da43e1b9a244856e0408f6ab
and commit 52a18150a7a8f2802000136c35005bc83b5b0855.

The copy() method unfortunately has its own problems so it was not
a good replacement.

Avoid warning when compiling with gcc's -Wformat-security option.

Disable entries instead of shuffling memory around

Less risk for bugs this way.

Update copyright years

Switch to FLTK's copy() method for menus

It has less surprises in terms of magical characters.

Abort argument processing on error.

This prevents e.g. 'Xvnc -screen x' crashing.

Create a console window for help output on Windows

Add shortcuts to context menu for quick access

Stop mapping Meta to Super

A lot of X11 keyboard layouts send Meta for Shift-Alt so it's not
really a good idea to assume this is the Windows key.

Merge branch 'perf' of https://github.com/CendioOssman/tigervnc

Default compression level should be interpreted as high

Be more aggressive in finding solid rectangles

The previous FIXME was incorrect. The old code did in fact continue
looking over the entire area. This commit restores that old behaviour.

Add test to measure high level encoding performance

Move encoder statistics into EncodeManager from SMsgWriter

It knows more details so it can track things better.

Optimise fillRect()

It had regressed in performance compared to 1.3. This brings it back
up to the same speed.

Make it possible to measure CPU usage in parellel

Add test to measure decoding performance

GnuTLS needs socket libraries so include them in the static link

rfb::Region uses the Xregion library

Add stream class to feed data from a file

No need for this array to be writeable

Move reverse connection information into VNCSConnectionST

The only class that cares if connections are reversed or not is
VNCSConnectionST so let it keep track of this fact.

These methods aren't critical so no need for them to be pure

Give subclasses the ability to overwrite the reader and writer

Needed for some tests.

SConnection internals should be private, just like for CConnection

Build fixes for modern versions of binutils.

Need to explicitly list DSOs.

Use a common header file for Windows errno names.

In earlier Visual Studio and MinGW editions, BSD socket errno:s were
left undefined. This is no longer the case. This may cause build or
runtime errors. To avoid this, we are using a common header file which
corrects all definitions. This header will also be used with other
projects such as sercd, unfs3, PulseAudio etc.

Merge pull request #114 from bphinz/master

Fixes for issues caused by restructuring

Fix ubuntu patches after recent restructuring

unix/xserver/hw/vnc/xvnc.cc was renamed to xvnc.c

Fixes for build issues caused by recent restructuring

 * Add typedef for XF86OptionPtr for Xorg < 112
 * Include stddef.h to prevent missing size_t definition errors
 * Workaround for libtool convenience library issue

Fix fuzz in distro patches caused upstream changes

Make the calls to va_list logging functions more explicit

On some platforms va_list might be defined as something that causes
ambiguity and results in the wrong function being called.

Add parameters to force IPv4/IPv6 selection

Add Esperanto translations

Avoid shadowing variables

ddxInitGlobals() was removed from Xorg ages ago

ddxBeforeReset() is only conditionally included in the build

Properly define functions as not having any arguments

Remove declaration for variable already present in headers

Make sure we get prototypes for global DDX functions

Avoid complaints about unused variable in argument less requests

The REQUEST macro is used to access variables in the request data,
and if we don't then the compiler complains that the macro sets up
variables that we never use.

No need to cast these as they are already an appropriate type

Declare variables at the start of each block, as required

Use proper prototypes in headers for all our internal functions