[PATCH] D69155: [analyzer] Fix off-by-one in operator call parameter binding.

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 18 06:55:02 PDT 2019


NoQ marked an inline comment as done.
NoQ added inline comments.


================
Comment at: clang/lib/StaticAnalyzer/Core/CallEvent.cpp:325
     if (getKind() != CE_CXXAllocator)
       if (isArgumentConstructedDirectly(Idx))
         if (auto AdjIdx = getAdjustedParameterIndex(Idx))
----------------
Charusso wrote:
> What about this one? It smells the same issue.
Here `Idx` is an argument index to begin with, so it shouldn't be a problem.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69155/new/

https://reviews.llvm.org/D69155





More information about the cfe-commits mailing list