From 136526ce2d2ea86450f4205e20f3fda24ac16a5b Mon Sep 17 00:00:00 2001 From: Victor Hernandez Date: Fri, 16 Oct 2009 18:08:17 +0000 Subject: [PATCH] Invert isSafeToGetMallocArraySize check because we return NULL when we don't know the size. Thanks to Duncan Sands for noticing this bug. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@84261 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Analysis/MallocHelper.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/Analysis/MallocHelper.cpp b/lib/Analysis/MallocHelper.cpp index 41fdab9012f..e7bb41eeec2 100644 --- a/lib/Analysis/MallocHelper.cpp +++ b/lib/Analysis/MallocHelper.cpp @@ -234,7 +234,7 @@ static bool isConstantOne(Value *val) { /// determined. Value* llvm::getMallocArraySize(CallInst* CI, LLVMContext &Context, const TargetData* TD) { - if (isSafeToGetMallocArraySize(CI, Context, TD)) + if (!isSafeToGetMallocArraySize(CI, Context, TD)) return NULL; // Match CreateMalloc's use of constant 1 array-size for non-array mallocs.