diff options
| author | Douglas Gregor <dgregor@apple.com> | 2011-02-01 15:50:11 +0000 |
|---|---|---|
| committer | Douglas Gregor <dgregor@apple.com> | 2011-02-01 15:50:11 +0000 |
| commit | d880f52be3e4a8ccad92ac31932eeae5e0870a93 (patch) | |
| tree | 97af0d083661709907c9ab8374ce6c1d92d877b9 /lib/Sema/SemaExprCXX.cpp | |
| parent | f9997a0834e0e0298b04ef044ad2699c727a7979 (diff) | |
Implement access checking for the "delete" operator. Fixes PR9050,
from Alex Miller!
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@124663 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Sema/SemaExprCXX.cpp')
| -rw-r--r-- | lib/Sema/SemaExprCXX.cpp | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/lib/Sema/SemaExprCXX.cpp b/lib/Sema/SemaExprCXX.cpp index 82e818fba0..7bc9af1191 100644 --- a/lib/Sema/SemaExprCXX.cpp +++ b/lib/Sema/SemaExprCXX.cpp @@ -1687,7 +1687,15 @@ Sema::ActOnCXXDelete(SourceLocation StartLoc, bool UseGlobal, MarkDeclarationReferenced(StartLoc, OperatorDelete); - // FIXME: Check access and ambiguity of operator delete and destructor. + // Check access and ambiguity of operator delete and destructor. + if (const RecordType *RT = PointeeElem->getAs<RecordType>()) { + CXXRecordDecl *RD = cast<CXXRecordDecl>(RT->getDecl()); + if (CXXDestructorDecl *Dtor = LookupDestructor(RD)) { + CheckDestructorAccess(Ex->getExprLoc(), Dtor, + PDiag(diag::err_access_dtor) << PointeeElem); + } + } + } return Owned(new (Context) CXXDeleteExpr(Context.VoidTy, UseGlobal, ArrayForm, |