[clang] Fix null dereference on return in lambda attribute statement expr (#66643)

clang was crashing on a lambda attribute with a statement expression
that contained a `return`.
It attempted to access the lambda type which was unknown at that point.

Fixes https://github.com/llvm/llvm-project/issues/48527
This commit is contained in:
Piotr Fusik 2023-09-19 04:02:04 +02:00 committed by GitHub
parent 7db4a6f278
commit c724ac9330
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 20 additions and 0 deletions

View File

@ -301,6 +301,10 @@ Bug Fixes to C++ Support
makes an invalid call to an immediate function.
(`#66324 <https://github.com/llvm/llvm-project/issues/66324>`_)
- Fix crash for a lambda attribute with a statement expression
that contains a `return`.
(`#48527 <https://github.com/llvm/llvm-project/issues/48527>`_)
Bug Fixes to AST Handling
^^^^^^^^^^^^^^^^^^^^^^^^^
- Fixed an import failure of recursive friend class template.

View File

@ -3577,6 +3577,8 @@ StmtResult Sema::ActOnCapScopeReturnStmt(SourceLocation ReturnLoc,
CapturingScopeInfo *CurCap = cast<CapturingScopeInfo>(getCurFunction());
QualType FnRetType = CurCap->ReturnType;
LambdaScopeInfo *CurLambda = dyn_cast<LambdaScopeInfo>(CurCap);
if (CurLambda && CurLambda->CallOperator->getType().isNull())
return StmtError();
bool HasDeducedReturnType =
CurLambda && hasDeducedReturnType(CurLambda->CallOperator);

View File

@ -0,0 +1,10 @@
// RUN: %clang_cc1 -fsyntax-only -verify %s
int main() { // expected-note {{to match this '{'}}
auto a = [](void)__attribute__((b(({ // expected-note {{to match this '('}}
return 0;
} // expected-error 3 {{expected ')'}} \
// expected-error {{expected ';' at end of declaration}}
// expected-error@+2 {{expected ')'}}
// expected-error@+1 {{expected body of lambda expression}}
// expected-error {{expected '}'}}

View File

@ -714,3 +714,7 @@ void foo() {
// CHECK-NEXT: ConstantExpr
// CHECK-NEXT: value: Int 2
}
void GH48527() {
auto a = []()__attribute__((b(({ return 0; })))){}; // expected-warning {{unknown attribute 'b' ignored}}
}