CertC++-DCL54ΒΆ
Overload allocation and deallocation functions as a pair in the same scope
Required inputs: IR
Allocation and deallocation functions can be overloaded at both global and class scopes.
If an allocation function is overloaded in a given scope, the corresponding deallocation function must also be overloaded in the same scope (and vice versa).
Failure to overload the corresponding dynamic storage function is likely to violate rules such as MEM51-CPP. Properly deallocate dynamically allocated resources. For instance, if an overloaded allocation function uses a private heap to perform its allocations, passing a pointer returned by it to the default deallocation function will likely cause undefined behavior. Even in situations in which the allocation function ultimately uses the default allocator to obtain a pointer to memory, failing to overload a corresponding deallocation function may leave the program in an unexpected state by not updating internal data for the custom allocator.
It is acceptable to define a deleted allocation or deallocation function
without its corresponding
free
store function. For instance, it is a common practice to define a deleted
non-placement allocation or deallocation function as a class member function
when the class also defines a placement
new function. This prevents accidental allocation via calls
to
new for that class type or deallocation via calls to
delete on pointers to an object of that class type. It is
acceptable to declare, but not define, a private allocation or deallocation
function without its corresponding free store function for similar reasons.
However, a definition must not be provided as that still allows access to the
free store function within a class member function.
Noncompliant Code Example
In this noncompliant code example, an allocation function is overloaded at global scope. However, the corresponding deallocation function is not declared. Were an object to be allocated with the overloaded allocation function, any attempt to delete the object would result in undefined behavior in violation of MEM51-CPP. Properly deallocate dynamically allocated resources.
#include <Windows.h>
#include <new>
void *operator new(std::size_t size) noexcept(false) {
static HANDLE h = ::HeapCreate(0, 0, 0); // Private, expandable heap.
if (h) {
return ::HeapAlloc(h, 0, size);
}
throw std::bad_alloc();
}
// No corresponding global delete operator defined.
Compliant Solution
In this compliant solution, the corresponding deallocation function is also defined at global scope.
#include <Windows.h>
#include <new>
class HeapAllocator {
static HANDLE h;
static bool init;
public:
static void *alloc(std::size_t size) noexcept(false) {
if (!init) {
h = ::HeapCreate(0, 0, 0); // Private, expandable heap.
init = true;
}
if (h) {
return ::HeapAlloc(h, 0, size);
}
throw std::bad_alloc();
}
static void dealloc(void *ptr) noexcept {
if (h) {
(void)::HeapFree(h, 0, ptr);
}
}
};
HANDLE HeapAllocator::h = nullptr;
bool HeapAllocator::init = false;
void *operator new(std::size_t size) noexcept(false) {
return HeapAllocator::alloc(size);
}
void operator delete(void *ptr) noexcept {
return HeapAllocator::dealloc(ptr);
}
Noncompliant Code Example
In this noncompliant code example,
operator new() is overloaded at class scope, but
operator delete() is not similarly overloaded at class scope.
Despite that the overloaded allocation function calls through to the default
global allocation function, were an object of type
S to be allocated, any attempt to delete the object would result
in leaving the program in an indeterminate state due to failing to update
allocation bookkeeping accordingly.
#include <new>
extern "C++" void update_bookkeeping(void *allocated_ptr, std::size_t size, bool alloc);
struct S {
void *operator new(std::size_t size) noexcept(false) {
void *ptr = ::operator new(size);
update_bookkeeping(ptr, size, true);
return ptr;
}
};
Compliant Solution
In this compliant solution, the corresponding
operator delete() is overloaded at the same class scope.
#include <new>
extern "C++" void update_bookkeeping(void *allocated_ptr, std::size_t size, bool alloc);
struct S {
void *operator new(std::size_t size) noexcept(false) {
void *ptr = ::operator new(size);
update_bookkeeping(ptr, size, true);
return ptr;
}
void operator delete(void *ptr, std::size_t size) noexcept {
::operator delete(ptr);
update_bookkeeping(ptr, size, false);
}
};
Exceptions
DCL54-CPP-EX1: A placement deallocation function may be elided
for a corresponding placement allocation function, but only if the object
placement allocation and object construction are guaranteed to be
noexcept(true). Because placement deallocation functions are
automatically invoked when the object initialization terminates by throwing an
exception, it is safe to elide the placement deallocation function when
exceptions cannot be thrown. For instance, some vendors implement compiler
flags disabling exception support (such as -fno-cxx-exceptions in
Clangand /EHs-c- in
Microsoft
Visual Studio), which has
implementation-defined
behavior when an exception is thrown but generally results in program
termination similar to calling
abort().
Risk Assessment
Mismatched usage of
new and
delete could lead to a
denial-of-service
attack.
| Rule | Severity | Likelihood | Remediation Cost | Priority | Level |
|---|---|---|---|---|---|
| DCL54-CPP | Low | Probable | Low | P6 | L2 |
Related Guidelines
| SEI CERT C++ Coding Standard | MEM51-CPP. Properly deallocate dynamically allocated resources |
Bibliography
| [ ISO/IEC 14882-2014] | Subclause 3.7.4, "Dynamic Storage Duration" Subclause 5.3.4, "New" Subclause 5.3.5, "Delete" |
Possible Messages
Key |
Text |
Severity |
Disabled |
|---|---|---|---|
array_delete_missing |
Scope contains operator new[] but not operator delete[] |
None |
False |
array_mixed_visibility |
Scope contains both new[] and delete[] operators but the visibility differs |
None |
False |
array_new_missing |
Scope contains operator delete[] but not operator new[] |
None |
False |
delete_missing |
Scope contains operator new but not operator delete |
None |
False |
mixed_visibility |
Scope contains both new and delete operators but visibility differs |
None |
False |
new_missing |
Scope contains operator delete but not operator new |
None |
False |
Options
This rule shares the following common options: exclude_in_macros, exclude_messages_in_system_headers, excludes, extend_exclude_to_macro_invocations, includes, justification_checker, languages, post_processing, provider, report_at, severity
The following places define options that affect this rule: Stylechecks, Analysis-GlobalOptions
This rule has no individual options.