xpra icon
Bug tracker and wiki

This bug tracker and wiki are being discontinued
please use https://github.com/Xpra-org/xpra instead.

Opened 9 years ago

Closed 9 years ago

Last modified 16 months ago

#240 closed defect (wontfix)

early startup errors should not leave a vfb behind

Reported by: Antoine Martin Owned by:
Priority: trivial Milestone: 0.15
Component: server Version: trunk
Keywords: Cc:


these are very rare but we should still do the right thing.

In this case, there was a bug in the codec's C code and it bombed out, IIRC it took the whole server with it but left the vfb running...

Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/xpra/server_source.py", line 1049, in data_to_packet
  File "/usr/lib64/python2.7/site-packages/xpra/window_source.py", line 624, in make_data_packet
    packet = self.make_data_packet(*data)
  File "/usr/lib64/python2.7/site-packages/xpra/window_source.py", line 755, in make_data_packet
    data, client_options = self.video_encode(wid, x, y, w, h, coding, data, rowstride, options)
  File "/usr/lib64/python2.7/site-packages/xpra/window_source.py", line 868, in video_encode
    self._video_encoder.init_context(w, h, self.encoding_options)
  File "codec.pyx", line 207, in xpra.x264.codec.Encoder.init_context (xpra/x264/codec.c:3146)
  File "codec.pyx", line 187, in xpra.x264.codec.Encoder._get_profile (xpra/x264/codec.c:2611)
ValueError: unsupported format character 'S' (0x53) at index 11

Change History (4)

comment:1 Changed 9 years ago by Antoine Martin

Resolution: wontfix
Status: newclosed

actually, we can't do anything about this: we can't just encode a dummy frame to test the encoder, and if something crashes after the initial setup we should leave the vfb running so that it can be salvaged.

comment:2 Changed 7 years ago by Antoine Martin

Note: we handle this in 0.16 a lot better: most of the codecs have selftests which will catch those errors and disable the codec. (see also #948)

comment:3 Changed 6 years ago by Antoine Martin

Milestone: 1.00.15

fix milestone: selftests were added during the 0.15.x milestone, ie: r8264 for enc_x264.

comment:4 Changed 16 months ago by migration script

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

Note: See TracTickets for help on using tickets.