From 80c8c3178276d0988c0129fb235f5b08a2e4f66d Mon Sep 17 00:00:00 2001 From: Alisdair Meredith Date: Thu, 9 Jul 2009 17:26:16 +0000 Subject: [PATCH] Resolve undefined behaviour when ManagedStatic is instantiated with a fixed-length array type. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@75149 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/Support/ManagedStatic.h | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/include/llvm/Support/ManagedStatic.h b/include/llvm/Support/ManagedStatic.h index 4fc648319ad..b8e223587fb 100644 --- a/include/llvm/Support/ManagedStatic.h +++ b/include/llvm/Support/ManagedStatic.h @@ -27,10 +27,12 @@ void* object_creator() { /// object_deleter - Helper method for ManagedStatic. /// -template -void object_deleter(void *Ptr) { - delete (C*)Ptr; -} +template struct object_deleter { + static void call(void * Ptr) { delete (T*)Ptr; } +}; +template struct object_deleter { + static void call(void * Ptr) { delete[] (T*)Ptr; } +}; /// ManagedStaticBase - Common base class for ManagedStatic instances. class ManagedStaticBase { @@ -62,28 +64,28 @@ public: C &operator*() { void* tmp = Ptr; if (llvm_is_multithreaded()) sys::MemoryFence(); - if (!tmp) RegisterManagedStatic(object_creator, object_deleter); + if (!tmp) RegisterManagedStatic(object_creator, object_deleter::call); return *static_cast(Ptr); } C *operator->() { void* tmp = Ptr; if (llvm_is_multithreaded()) sys::MemoryFence(); - if (!tmp) RegisterManagedStatic(object_creator, object_deleter); + if (!tmp) RegisterManagedStatic(object_creator, object_deleter::call); return static_cast(Ptr); } const C &operator*() const { void* tmp = Ptr; if (llvm_is_multithreaded()) sys::MemoryFence(); - if (!tmp) RegisterManagedStatic(object_creator, object_deleter); + if (!tmp) RegisterManagedStatic(object_creator, object_deleter::call); return *static_cast(Ptr); } const C *operator->() const { void* tmp = Ptr; if (llvm_is_multithreaded()) sys::MemoryFence(); - if (!tmp) RegisterManagedStatic(object_creator, object_deleter); + if (!tmp) RegisterManagedStatic(object_creator, object_deleter::call); return static_cast(Ptr); } -- 2.11.0