| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
We mostly use classical C strings, but the memory management around them
can get confusing and error prone. Let's use std::string for the cases
where we need to return a newly allocated string.
|
|
|
|
|
|
|
|
|
| |
This mimics how some system functions (like inet_ntop()) work, and
avoids complexity around ownership of the returned string buffer.
The downside is that the string must be consumed directly as it will be
overwritten on the next call, but that is not an issue with the current
usage.
|
|
|
|
|
| |
These don't need to be changed, so let's just keep it simple with a
constant string literal.
|
|
|
|
|
| |
We don't need to make extra copies of the string in most cases, so let's
simplify the code and access the string directly when we can.
|
|
|
|
|
| |
We never use Windows' "UNICODE" mode anyway, so let's get rid of this
complexity.
|
|
|
|
|
| |
That warning doesn't play well with Windows' GetProcAddress(), so add
some extra casting to work around it.
|
|
|
|
|
| |
This allows us to separate accidental fall through from explicit ones,
which allows us to turn on such checks in the compiler.
|
|
|
|
|
| |
This allows us to separate accidentally unused, from explicitly unused
parameters, which allows us to turn on such checks in the compiler.
|
|
|
|
|
| |
They don't get access bits until they've been authenticated, so avoid
doing any checks on clients in those early states.
|
|
|
|
|
| |
The migration from 32-bit to 64-bit has long passed. Let's get rid of
all distinctions as native applications it the norm now.
|
| |
|
|
|
|
|
|
| |
The generally recommended way is to include it from source files, not
headers. We had a mix of both. Let's try to be consistent and follow the
recommended way.
|
| |
|
|
|
|
|
| |
This makes our builds directly compatible with most distributions
without packagers/users having to specify extra flags.
|
| |
|
| |
|
|\ |
|
| |
| |
| |
| |
| |
| | |
Do a proper cleanup when one of the termination timeouts trigger
rather than just exiting on the spot. This makes sure we don't leave
stray stuff around, e.g. unix socket files.
|
|\| |
|
| |
| |
| |
| |
| |
| | |
Windows tries to do a copy of it, and it is too complex to survive
that. Just reference the object directly as we know the source of
the message here.
|
| |
| |
| |
| |
| | |
It is functionality specific to WinVNC, so move the code there
to make things more clear.
|
| |
| |
| |
| |
| |
| | |
Removed the last parts of VNCSConnectionST's back door in to
VNCServerST and let the parent class fully handle coordination of
clients, and access to the desktop.
|
| |
| |
| |
| |
| | |
Make things simpler by making this a part of the SDesktop interface
that always needs to be implemented.
|
| |
| |
| |
| |
| |
| | |
Force queryConnection() to always call back to approveConnection()
rather than return special values. This makes the flow easier to
follow as it will be the same in all cases.
|
|\| |
|
| |
| |
| |
| |
| | |
Applets don't work anymore so remove everything that has to do
with serving them.
|
|/
|
|
| |
Our main web site now supports https, so update all references.
|
|\ |
|
| |
| |
| |
| |
| | |
Makes the code more general and not directly tied to specifically
TCP sockets.
|
|/
|
|
|
| |
We use polymorphic exception objects, so catching by value invokes
the copy constructor and stuff that we don't really want.
|
| |
|
| |
|
| |
|
|
|
|
| |
It was only used by WinVNC, so push it there instead.
|
| |
|
| |
|
| |
|
|
|
|
| |
Use the platform independent primitives instead.
|
|
|
|
|
|
| |
We've changed the service name a couple of times for various
reasons. Make sure the old versions are removed when the new one
is registered.
|
| |
|
| |
|
|
|
|
|
| |
We have lots of code that deals with Windows versions that we no
longer support anyway. Clean out all of this old cruft.
|
|
|
|
|
| |
The identifier is used to compute the registry path, and should
ideally be kept short and without spaces.
|
|
|
|
|
|
| |
The argument named "desc" was not actually the description, but
rather the short "display name". Add handling for the actual
description to reduce confusion.
|
| |
|
|
|
|
|
|
| |
It is easier to control object life time and avoid magical socket
duplication by having a single TcpListener object to pass around.
We have to be more careful about deleting the object though.
|
|
|
|
|
| |
Avoids conflicts with the original RealVNC implementation, or any
other forks of that code.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
It is needed by WinVNC, but got removed in 892d10a70. Also fix
a couple of issues:
- Use getnameinfo() in order to be compatible with Windows XP
- Make it static since it doesn't use a specific socket
- Respect UseIPv4 and UseIPv6
- Flags for getaddrinfo() that match binding code
- Dummy service value for Windows compatibility
|