Xpra: Ticket #671: Server crashes if lz4 is missing

Hi.

If the server does not have lz4 installed, connecting to the server via proxy fails. The proxy log gives the following error message:

2014-09-05 16:51:31,629 internal error: error in network packet reading/parsing
Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/xpra/net/protocol.py", line 587, in _read_parse_thread_loop
    self.do_read_parse_thread_loop()
  File "/usr/lib64/python2.7/site-packages/xpra/net/protocol.py", line 762, in do_read_parse_thread_loop
    self._process_packet_cb(self, packet)
  File "/usr/lib64/python2.7/site-packages/xpra/server/proxy_instance_process.py", line 526, in process_server_packet
    packet[8] = compressed_wrapper("cursor", pixels, zlib=zlib, lz4=lz4, lzo=lzo, can_inline=False)
  File "/usr/lib64/python2.7/site-packages/xpra/net/compression.py", line 184, in compressed_wrapper
    assert use_lz4, "cannot use lz4"
AssertionError: cannot use lz4

This immediately terminates the connection. I guess the server should either negotiate with the client not to use lz4 or lz4 should be considered mandatory in "setup.py". Error is reproducable with 0.14.4 (Windows client, Linux proxy+server).

Kind regards, Stephan



Fri, 05 Sep 2014 15:25:54 GMT - Antoine Martin: owner changed

Oops sorry!

Does r7521 fix things for you? (there was a big FIXME note near that code...)


Fri, 05 Sep 2014 16:05:27 GMT - sschnitzer:

Fixed, thanks.


Sat, 06 Sep 2014 09:38:06 GMT - Antoine Martin: status changed; resolution set

Backport in r7524.

Forgot to mention that you really should have lz4 installed... (lzo at the very least)


Sat, 23 Jan 2021 05:02:33 GMT - migration script:

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/671