On Tue, 2016-03-08 at 00:30 +0100, Rémi Cardona wrote: > Le 07/03/2016 23:20, Julien Cristau a écrit : > > Hi, > > > > prompted by Adam's recent fixes for unchecked AddResource, I've been > > looking at the failure paths, and found a number of places where it > > looks like we're freeing things twice (once as part of AddResource > > itself calling the destroy function for the resource, and once in the > > caller). Plus more unchecked AddResource calls in xvmc and dri3. > > > > Cheers, > > Julien > > Caveat emptor: I didn't check that CreateNewResourceType was called with > a correct deleteFunc. With that said, the patches look correct to me: > > Reviewed-by: Rémi Cardona <[email protected]>
remote: I: patch #76082 updated using rev 119d5c0e2f800737c949ef760c5fe25d963200bf. remote: I: patch #76083 updated using rev ac97fb2b804809c39b12fe0032d96fb076657258. remote: I: patch #76078 updated using rev 59b9c3d5e4bf05aeaaac2ee4ea12c301a67aae2c. remote: I: patch #76080 updated using rev 164753f158e78f615f903467bfd234d7c58244ef. remote: I: patch #76079 updated using rev acf263df81ad6813e0233033610fb44521cab1b4. remote: I: patch #76085 updated using rev d0c1a5bc61a3d151f2234aa3820862f16c0f00c7. remote: I: patch #76084 updated using rev 054f80717812d4781741cd05393623fe6f6c627f. remote: I: patch #76086 updated using rev 4217db89ecd480fda2ee74fecba06c6713c2a0f0. remote: I: patch #76081 updated using rev 2be527b1d4ce2b0412c4484539a8c9607645ec6d. remote: I: 9 patch(es) updated to state Accepted. To ssh://git.freedesktop.org/git/xorg/xserver a2c3c34..2be527b master -> master - ajax _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel
