[PATCH] Remove static MAXSCREENS limit from Xext/shm.c.
Keith Packard
keithp at keithp.com
Thu Oct 1 15:55:09 PDT 2009
Excerpts from Jamey Sharp's message of Thu Oct 01 15:26:47 -0700 2009:
> Hmm. I think that's what I did?
Wow. 3 hours of sleep is clearly not good here. Yes, it looks quite
reasonable on actual review (and a couple of hours of napping on the plane).
> But ProcPanoramiXShmGetImage seems to need an actual array with at least
> as many entries as the number of active screens, which it uses as a
> temporary.
I'd allocate/free at each call. (and, no, not using alloca).
> > You then wrap CloseScreen (in the usual way) and free your private
> > structure there.
>
> I missed that step. I wondered whether this stuff was supposed to get
> freed somewhere...
heh. server 'regen' has quite a number of leaks of this nature. I
found a bunch a week ago or so, but it's nice to catch them early :-)
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: not available
Url : http://lists.x.org/archives/xorg-devel/attachments/20091001/d4378040/attachment.pgp
More information about the xorg-devel
mailing list