[PATCH] dix/window: catch illegal window reconfiguration for composited windows. (v2)
airlied at gmail.com
Wed Jun 2 21:44:04 PDT 2010
On Thu, Jun 3, 2010 at 2:18 PM, Keith Packard <keithp at keithp.com> wrote:
> On Thu, 3 Jun 2010 09:45:31 +1000, Dave Airlie <airlied at gmail.com> wrote:
>> From: Dave Airlie <airlied at redhat.com>
>> If a windows has been redirected, then it has to respect pixmap size limits,
>> 32kx32k. This checks before resizing or changing border width if the window
>> is redirected and the pixmap size limits are being hit, and returns
> I don't think this is a reasonable plan; you're only catching really
> obvious mistakes and leaving any actual allocation failures unreported
> to the client. Given your patch which keeps the server from crashing on
> CreatePixmap failure, I'd say this patch isn't all that useful.
I think its the only possible fix without breaking ABI, which is a
concern for anyone shipping this in a server that is released, which
is everyone who probably cares.
If we break ABI then return a Bool from ConfigNotify might work and
I'll investigate that.
More information about the xorg-devel