diff options
author | Zhongxing Xu <xuzhongxing@gmail.com> | 2010-02-14 06:49:48 +0000 |
---|---|---|
committer | Zhongxing Xu <xuzhongxing@gmail.com> | 2010-02-14 06:49:48 +0000 |
commit | 181cc3df6e0046a154a8a174d551d57af4561998 (patch) | |
tree | 9fdc3ea74c806ab365303f692473030c1415c3a7 | |
parent | 6dd66ed959b7f60749dd0040507b3f304183a1b6 (diff) |
Fix pr6293. If ptr is NULL, no operation is preformed.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@96154 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Checker/MallocChecker.cpp | 5 | ||||
-rw-r--r-- | test/Analysis/malloc.c | 4 |
2 files changed, 9 insertions, 0 deletions
diff --git a/lib/Checker/MallocChecker.cpp b/lib/Checker/MallocChecker.cpp index 3be2e0299b..4ff98642e1 100644 --- a/lib/Checker/MallocChecker.cpp +++ b/lib/Checker/MallocChecker.cpp @@ -172,6 +172,11 @@ void MallocChecker::FreeMem(CheckerContext &C, const CallExpr *CE) { const GRState *MallocChecker::FreeMemAux(CheckerContext &C, const CallExpr *CE, const GRState *state) { SVal ArgVal = state->getSVal(CE->getArg(0)); + + // If ptr is NULL, no operation is preformed. + if (ArgVal.isZeroConstant()) + return state; + SymbolRef Sym = ArgVal.getAsLocSymbol(); assert(Sym); diff --git a/test/Analysis/malloc.c b/test/Analysis/malloc.c index 518ab82ff1..3cce1b0d2e 100644 --- a/test/Analysis/malloc.c +++ b/test/Analysis/malloc.c @@ -57,3 +57,7 @@ void pr6069() { char *buf = doit2(); free(buf); } + +void pr6293() { + free(0); +} |