aboutsummaryrefslogtreecommitdiffstats
path: root/win
Commit message (Collapse)AuthorAgeFilesLines
* Return std::string instead of dynamic allocationsPierre Ossman2023-02-0411-81/+64
| | | | | | 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.
* Use fixed size character bufferPierre Ossman2023-02-043-17/+16
| | | | | We know the needed space here, so let's keep it simple with a constant size string buffer.
* Return static char buffer from some methodsPierre Ossman2023-02-047-19/+18
| | | | | | | | | 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.
* Use string literal instead of dynamic bufferPierre Ossman2023-02-042-8/+8
| | | | | These don't need to be changed, so let's just keep it simple with a constant string literal.
* Remove unneeded CharArray:sPierre Ossman2023-02-042-10/+9
| | | | | Avoid complicating things by moving things in to a second buffer here as there is no need for it.
* Use StringParameters directlyPierre Ossman2023-02-044-19/+15
| | | | | 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.
* Get rid of TCHAR magicPierre Ossman2023-02-0463-685/+447
| | | | | We never use Windows' "UNICODE" mode anyway, so let's get rid of this complexity.
* Return std::vector instead of dynamic allocationPierre Ossman2023-02-043-19/+12
| | | | | This makes memory management more clear and robust when using these helper functions.
* Move hex conversion helpers to utilPierre Ossman2023-02-042-5/+7
| | | | | These are used here and there so let's make them more general rather than hiding them in the stream classes.
* Use std::vector for basic data arraysPierre Ossman2023-02-013-30/+31
| | | | | Avoid our own custom types in favour of what's already included with C++.
* Use stdint typesPierre Ossman2023-02-0111-41/+42
| | | | | Avoid having our own custom stuff and instead use the modern, standard types, for familiarity.
* Work around -Wcast-function-typePierre Ossman2023-01-052-10/+10
| | | | | That warning doesn't play well with Windows' GetProcAddress(), so add some extra casting to work around it.
* Fix incorrect typesPierre Ossman2023-01-051-1/+1
| | | | | These types caused an incorrect signed/unsigned behaviour, so let's make sure we use the appropriate type.
* Remove useless checksPierre Ossman2023-01-051-1/+1
| | | | These are all truisms because of the valid range of the types.
* Explicitly mark switch case fall throughPierre Ossman2023-01-051-0/+1
| | | | | This allows us to separate accidental fall through from explicit ones, which allows us to turn on such checks in the compiler.
* Explicitly mark unused parametersPierre Ossman2023-01-0416-20/+21
| | | | | This allows us to separate accidentally unused, from explicitly unused parameters, which allows us to turn on such checks in the compiler.
* Don't do access checks on early clientsPierre Ossman2022-11-071-3/+5
| | | | | They don't get access bits until they've been authenticated, so avoid doing any checks on clients in those early states.
* Avoid temporary cmake variablesPierre Ossman2022-06-071-4/+2
| | | | | Modern cmake has better support for adding source files and libraries incrementally, so let's use that to clean things up.
* Enable stream sanity checks in debug buildsPierre Ossman2022-04-061-4/+0
| | | | | | Enable this automatically for developers so we increase the chance of these problems getting caught. There is a risk of overhead though so keep them disabled for release builds.
* Remove 32-/64-bit distinctionsPierre Ossman2022-01-212-14/+4
| | | | | 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.
* Update copyright year to 2022Pierre Ossman2022-01-052-2/+2
|
* Be consistent in including config.hPierre Ossman2021-12-3039-0/+156
| | | | | | 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.
* Add support for notifying clients about pointer movementslhchavez2021-03-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | This change adds support for the VMware Mouse Position pseudo-encoding[1], which is used to notify VNC clients when X11 clients call `XWarpPointer()`[2]. This function is called by SDL (and other similar libraries) when they detect that the server does not support native relative motion, like some RFB clients. With this, RFB clients can choose to adjust the local cursor position under certain circumstances to match what the server has set. For instance, if pointer lock has been enabled on the client's machine and the cursor is not being drawn locally, the local position of the cursor is irrelevant, so the RFB client can use what the server sends as the canonical absolute position of the cursor. This ultimately enables the possibility of games (especially FPS games) to behave how users expect (if the clients implement the corresponding change). Part of: #619 1: https://github.com/rfbproto/rfbproto/blob/master/rfbproto.rst#vmware-cursor-position-pseudo-encoding 2: https://tronche.com/gui/x/xlib/input/XWarpPointer.html 3: https://hg.libsdl.org/SDL/file/28e3b60e2131/src/events/SDL_mouse.c#l804
* Update copyright year to 2021Pierre Ossman2021-01-252-2/+2
|
* Merge branch 'noblock' of https://github.com/CendioOssman/tigervncPierre Ossman2021-01-191-5/+32
|\
| * Change streams to be asynchronousPierre Ossman2020-05-211-5/+32
| | | | | | | | | | | | | | | | | | | | Major restructuring of how streams work. Neither input nor output streams are now blocking. This avoids stalling the rest of the client or server when a peer is slow or unresponsive. Note that this puts an extra burden on users of streams to make sure they are allowed to do their work once the underlying transports are ready (e.g. monitoring fds).
* | Don't clear complex objects using memset()Pierre Ossman2020-05-312-4/+2
|/ | | | This is fine for simple structs but not class based objects.
* Use standard install dir variable namesPierre Ossman2020-03-123-3/+3
| | | | | This makes our builds directly compatible with most distributions without packagers/users having to specify extra flags.
* Update copyright year to 2020Pierre Ossman2020-01-162-2/+2
|
* Clean up initialization of DIBSectionBufferPierre Ossman2020-01-034-35/+8
| | | | | | We had an unintentional conflict with PixelBuffer::setSize() here. But we can simplify this further as this initialization is only used by the subclass DeviceFrameBuffer, and only once.
* Use size_t for lengths in stream objectsPierre Ossman2019-11-152-6/+6
| | | | | | | | Provides safety against them accidentally becoming negative because of bugs in the calculations. Also does the same to CharArray and friends as they were strongly connection to the stream objects.
* Encapsulate PixelBuffer internal detailsPierre Ossman2019-11-151-25/+16
| | | | | | Don't allow subclasses to just override dimensions or buffer details directly and instead force them to go via methods. This allows us to do sanity checks on the new values and catch bugs and attacks.
* Use display polling by default in WinVNCPierre Ossman2019-09-061-1/+1
| | | | | Window hooks aren't working well on modern systems so switch the default to polling until we can fix things.
* Use UTF-8 in clipboard APIPierre Ossman2019-07-011-54/+19
| | | | | In prepartion for better clipboard extensions that can send Unicode data between the client and server.
* Improved clipboard APIPierre Ossman2019-07-014-36/+65
| | | | | Change the internal clipboard API to use a request based model in order to be prepared for more advanced clipboard transfers.
* Clean up internal clipboard handlingPierre Ossman2019-07-014-12/+12
| | | | | | We now filter incoming data, which means we can start assuming the clipboard data is always null terminated. This allows us to clean up a lot of the internal handling.
* Make sure clipboard uses \n line endingsPierre Ossman2019-07-011-14/+2
| | | | | | This is required by the protocol so we should make sure it is enforced. We are tolerant of clients that violate this though and convert incoming clipboard data.
* Add missing throws for exceptionPierre Ossman2019-03-261-1/+1
| | | | | It is not enough to create an exception object, you need to throw it as well.
* Update visible copyright years to 2019Samuel Mannehed2019-02-272-2/+2
|
* Merge branch 'exittimer' of https://github.com/CendioOssman/tigervncPierre Ossman2018-11-094-2/+15
|\
| * Get rid of SocketServer::checkTimeouts()Pierre Ossman2018-11-091-1/+2
| | | | | | | | | | | | It doesn't belong on each socket server object as timers are global. Force implementations to call the Timer system directly instead, avoiding any middle men.
| * Properly terminate server on timeoutsPierre Ossman2018-11-093-1/+13
| | | | | | | | | | | | 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.
* | Merge branch 'vncserver' of https://github.com/CendioOssman/tigervncPierre Ossman2018-11-0911-39/+304
|\|
| * Stop trying to send ListConnInfo via WM_COPYDATAPierre Ossman2018-11-092-3/+3
| | | | | | | | | | | | 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.
| * Move ListConnInfo to WinVNC directoryPierre Ossman2018-11-098-20/+253
| | | | | | | | | | It is functionality specific to WinVNC, so move the code there to make things more clear.
| * Encapsulate client handling in VNCServerSTPierre Ossman2018-10-261-0/+1
| | | | | | | | | | | | 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.
| * Remove QueryConnectionHandlerPierre Ossman2018-10-104-6/+38
| | | | | | | | | | Make things simpler by making this a part of the SDesktop interface that always needs to be implemented.
| * Force common flow of connection queriesPierre Ossman2018-10-092-11/+10
| | | | | | | | | | | | 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.
* | Merge branch 'nojava' of https://github.com/CendioOssman/tigervncPierre Ossman2018-10-2614-288/+2
|\|
| * Remove Java web serverPierre Ossman2018-10-0914-288/+2
| | | | | | | | | | Applets don't work anymore so remove everything that has to do with serving them.