[cfe-commits] r61943 - /cfe/trunk/lib/AST/DeclObjC.cpp

[Douglas Gregor]

On Jan 8, 2009, at 12:49 PM, Ted Kremenek wrote:

> Author: kremenek
> Date: Thu Jan  8 14:49:27 2009
> New Revision: 61943
>
> URL: http://llvm.org/viewvc/llvm-project?rev=61943&view=rev
> Log:
> Fix ObjCInterfaceDecl::Destroy and ObjCProtocolDecl::Destroy to  
> iterate and destroy all contained ObjCMethodDecls in one sweep.   
> This fixes a use-after-free error found by valgrind.

I don't think this is right. DeclContext should own the  
ObjCMethodDecls, and they should be destroyed by  
DeclContext::DestroyDecls.

	- Doug