xserver icon indicating copy to clipboard operation
xserver copied to clipboard

dix: resolved not free after ClientSleep (suspended status)

Open GermanAizek opened this issue 8 months ago • 2 comments

However, I'm not sure for sure, only resolved needs release memory or other pointers too?

GermanAizek avatar Jun 13 '25 10:06 GermanAizek

Are you really certain ? This one of the complex beasts where static analysis can easily be wrong. (I didn't dare to touch it yet, before 100% understanding it first)

For example, we need to check that the buffer isn't somehow passed to the closure. The whole client-sleep logic isn't entirely trivial. It processes the request up to certain point, then puts the client asleep (processing others in the meantime) and later, on wakeup, calling into a callback (which gets passed the closure) to finish the request.

Let's take this thing apart after the release.

metux avatar Jun 13 '25 11:06 metux

@GermanAizek ping

metux avatar Jul 23 '25 12:07 metux