xpra icon
Bug tracker and wiki

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#491 closed defect (fixed)

webp is leaking memory

Reported by: Antoine Martin Owned by: Antoine Martin
Priority: major Milestone: 0.13
Component: client Version:
Keywords: Cc:

Description (last modified by Antoine Martin)

See ticket:457#comment:30 - this is a client side leak only (I think), which affects all platforms.

Probably keeping references to some buffers somewhere (create_string_buffer?)

Fix it so we can actually use webp (and fix #487), and use it for more things. (ie: #419)

r5167 adds a big warning for anyone trying to use webp, will need to be removed if/when we figure this out

Change History (6)

comment:1 Changed 6 years ago by Antoine Martin

Description: modified (diff)
Owner: changed from Antoine Martin to Antoine Martin
Status: newassigned

comment:2 Changed 6 years ago by Antoine Martin

Description: modified (diff)

comment:3 Changed 6 years ago by Antoine Martin

Milestone: 0.120.13

re-scheduling all webp stuff

comment:4 Changed 6 years ago by Antoine Martin

  • leak fixed in r5945
  • remove leak warnings and re-add webp as a valid lossless encoding to switch to in r5951
  • refuse to use webp unless the client tell us it is not going to leak in r5955

All will need to be backported.

webp support via PIL was added in:

  • r5937 for encoding
  • r5936 for decoding (r5953 ensures we don't use Pillow for decoding webp for now - we can lift this restriction with a version check once upstream fixes the bug)


Pillow also leaks like we used to! (see r5952)

Bug reported upstream: Pillow issue 586

Last edited 6 years ago by Antoine Martin (previous) (diff)

comment:5 Changed 6 years ago by Antoine Martin

Resolution: fixed
Status: assignedclosed

Backports for v0.12.x in r5956. Closing.

comment:6 Changed 6 years ago by Antoine Martin

Note: the new webp cython encoder in 0.13 was also leaking memory, this is fixed in r6505.

Note: See TracTickets for help on using tickets.