[all-commits] [llvm/llvm-project] a663e7: [clang-tidy] Add recursion protection in Exception...

Piotr Zegar via All-commits all-commits at lists.llvm.org
Thu Feb 13 08:51:52 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a663e78a6eb6bbd20c0c74b3e6a78e24ee423544
      https://github.com/llvm/llvm-project/commit/a663e78a6eb6bbd20c0c74b3e6a78e24ee423544
  Author: Piotr Zegar <me at piotrzegar.pl>
  Date:   2025-02-13 (Thu, 13 Feb 2025)

  Changed paths:
    M clang-tools-extra/clang-tidy/utils/ExceptionSpecAnalyzer.cpp
    M clang-tools-extra/test/clang-tidy/checkers/performance/noexcept-move-constructor.cpp

  Log Message:
  -----------
  [clang-tidy] Add recursion protection in ExceptionSpecAnalyzer (#66810)

Normally endless recursion should not happen in ExceptionSpecAnalyzer,
but if AST would be malformed (missing include), this could cause crash.

I run into this issue when due to missing include constructor argument
were parsed as FieldDecl.
As checking for recursion cost nothing, why not to do this in check just
in case.

Fixes #111436



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list