Description
Although guac_socket_nest() takes an index parameter, this parameter is never used and the index of the socket is left uninitialized.
This is not a problem in practice as nested sockets are no longer used (the Guacamole protocol has evolved to inherently support streaming large blocks of data and no longer needs to nest itself as a workaround). It's likely that guac_socket_nest() and the "nest" instruction should be deprecated, but this should be fixed in the meantime.
Attachments
Issue Links
- is related to
-
GUACAMOLE-662 Failing unit tests for guacamole-server not triggering build failure
- Resolved
-
GUACAMOLE-661 Deprecate "nest" instruction
- Resolved
- relates to
-
GUACAMOLE-1517 data->written is not initialized in guac_socket_nest()
- Closed