summaryrefslogtreecommitdiffstats
path: root/examples
diff options
context:
space:
mode:
authordscho <dscho>2005-10-07 09:58:45 +0000
committerdscho <dscho>2005-10-07 09:58:45 +0000
commit951ec26b7cad541c030b7aebf289a21f631fc736 (patch)
tree87532ba5a1597a72d9e397b0d0ad4b81f867c0b7 /examples
parent2c177c866be95e9a2ea84021e8274cd2f60efdad (diff)
downloadlibvncserver-951ec26b7cad541c030b7aebf289a21f631fc736.zip
libvncserver-951ec26b7cad541c030b7aebf289a21f631fc736.tar.gz
The PseudoEncoding extension code was getting silly:
If the client asked for an encoding, and no enabled extension handled it, LibVNCServer would walk through all extensions, and if they promised to handle the encoding, execute the extension's newClient() if it was not NULL. However, if newClient is not NULL, it will be called when a client connects, and if it returns TRUE, the extension will be enabled. Since all the state of the extension should be in the client data, there is no good reason why newClient should return FALSE the first time (thus not enabling the extension), but TRUE when called just before calling enablePseudoEncoding(). So in effect, the extension got enabled all the time, even if that was not necessary. The resolution is to pass a void** to enablePseudoEncoding. This has the further advantage that enablePseudoEncoding can remalloc() or free() the data without problems. Though keep in mind that if enablePseudoEncoding() is called on a not-yet-enabled extension, the passed data points to NULL.
Diffstat (limited to 'examples')
-rw-r--r--examples/backchannel.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/examples/backchannel.c b/examples/backchannel.c
index 4db01a8..6a21390 100644
--- a/examples/backchannel.c
+++ b/examples/backchannel.c
@@ -33,7 +33,7 @@ typedef struct backChannelMsg {
uint32_t size;
} backChannelMsg;
-rfbBool enableBackChannel(rfbClientPtr cl, void* data, int encoding)
+rfbBool enableBackChannel(rfbClientPtr cl, void** data, int encoding)
{
if(encoding == rfbBackChannel) {
backChannelMsg msg;
OpenPOWER on IntegriCloud