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)
38 lines
902 B
C++
38 lines
902 B
C++
// RUN: %clang_analyze_cc1 \
|
|
// RUN: -analyzer-checker=core,unix.Malloc \
|
|
// RUN: -verify %s
|
|
|
|
// expected-no-diagnostics: We do not model Integer Set Library's retain-count
|
|
// based allocation. If any of the parameters has an
|
|
// '__isl_' prefixed macro definition we escape every
|
|
// of them when we are about to 'free()' something.
|
|
|
|
#define __isl_take
|
|
#define __isl_keep
|
|
|
|
struct Object { int Ref; };
|
|
void free(void *);
|
|
|
|
Object *copyObj(__isl_keep Object *O) {
|
|
O->Ref++;
|
|
return O;
|
|
}
|
|
|
|
void freeObj(__isl_take Object *O) {
|
|
if (--O->Ref > 0)
|
|
return;
|
|
|
|
free(O); // Here we notice that the parameter contains '__isl_', escape it.
|
|
}
|
|
|
|
void useAfterFree(__isl_take Object *A) {
|
|
if (!A)
|
|
return;
|
|
|
|
Object *B = copyObj(A);
|
|
freeObj(B);
|
|
|
|
A->Ref = 13;
|
|
// no-warning: 'Use of memory after it is released' was here.
|
|
}
|