[scan-build] fix logic error warning emitted on compiler-rt code base

[apelete]

Summary

Fix a "called c++ object pointer is null" warning emitted by Clang
Static Analyzer on the following file:

• lib/asan/asan_suppressions.cc.

Signed-off-by: Apelete Seketeli <[e-mail]>

[Eugene.Zelenko]

lib/asan/asan_suppressions.cc

21 ↗

(On Diff #55329)

Please use cassert for C++ code. See http://clang.llvm.org/extra/clang-tidy/checks/modernize-deprecated-headers.html.

[apelete]

[scan-build] fix logic error warning emitted on compiler-rt code base

Changes since last revision:

• replace deprecated <assert.h> library by C++ <cassert> equivalent.

[apelete]

lib/asan/asan_suppressions.cc

21 ↗

(On Diff #55379)

If the changes are good for you, could you please check this in the main repository for me ?

Thanks for your review.

[kcc]

This code does not use *any* system headers. You may use CHECK

[apelete]

[scan-build] fix logic error warning emitted on compiler-rt code base

Changes since last revision:

• Replace assert() with CHECK() since the code do not rely on any system header.

[kcc]

LGTM

[apelete]

In D19627#417229, @kcc wrote:

LGTM

Could you please check this in the main repository for me ?

Thanks.

compiler-rt/trunk/lib/asan/asan_suppressions.cc

if (suppression_ctx->HasSuppressionType(kInterceptorViaLibrary)) {
       // Match "interceptor_via_lib" suppressions.
       if (const char *module_name = symbolizer->GetModuleNameForPc(addr))
         if (suppression_ctx->Match(module_name, kInterceptorViaLibrary, &s))
           return true;
     }
 
     if (suppression_ctx->HasSuppressionType(kInterceptorViaFunction)) {
       SymbolizedStack *frames = symbolizer->SymbolizePC(addr);
+      CHECK(frames);
       for (SymbolizedStack *cur = frames; cur; cur = cur->next) {
         const char *function_name = cur->info.function;
         if (!function_name) {
           continue;
         }
         // Match "interceptor_via_fun" suppressions.
         if (suppression_ctx->Match(function_name, kInterceptorViaFunction,
                                    &s)) {