f0c90dfcd8
Changed the warning message: - **From**: 'Attempt to free released memory' **To**: 'Attempt to release already released memory' - **From**: 'Attempt to free non-owned memory' **To**: 'Attempt to release non-owned memory' - **From**: 'Use of memory after it is freed' **To**: 'Use of memory after it is released' All connected tests and their expectations have been changed accordingly. Inspired by [this PR](https://github.com/llvm/llvm-project/pull/147542#discussion_r2195197922)
32 lines
1.1 KiB
C++
32 lines
1.1 KiB
C++
// RUN: %clang_analyze_cc1 -std=c++14 -w -analyzer-checker=core,cplusplus -analyzer-output=text -verify %s
|
|
|
|
namespace no_crash_on_delete_dtor {
|
|
// We were crashing when producing diagnostics for this code, but not for the
|
|
// report that it currently emits. Instead, Static Analyzer was thinking that
|
|
// p.get()->foo() is a null dereference because it was dropping
|
|
// constraints over x too early and took a different branch next time
|
|
// we call .get().
|
|
struct S {
|
|
void foo();
|
|
~S();
|
|
};
|
|
|
|
struct smart_ptr {
|
|
int x;
|
|
S *s;
|
|
smart_ptr(S *);
|
|
S *get() {
|
|
return (x || 0) ? nullptr : s; // expected-note{{Field 'x' is 0}}
|
|
// expected-note@-1{{Left side of '||' is false}}
|
|
// expected-note@-2{{'?' condition is false}}
|
|
// expected-warning@-3{{Use of memory after it is released}}
|
|
// expected-note@-4{{Use of memory after it is released}}
|
|
}
|
|
};
|
|
|
|
void bar(smart_ptr p) {
|
|
delete p.get(); // expected-note{{Memory is released}}
|
|
p.get()->foo(); // expected-note{{Calling 'smart_ptr::get'}}
|
|
}
|
|
} // namespace no_crash_on_delete_dtor
|