diff options
| author | George Burgess IV <gbiv@google.com> | 2019-02-01 11:52:55 -0800 |
|---|---|---|
| committer | George Burgess <gbiv@google.com> | 2019-02-04 19:06:57 +0000 |
| commit | 20d7c96c368a551553519a3bfcfb1024944703be (patch) | |
| tree | 422e58ffe82a6acf8343b81e833b3664363e963e /tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp | |
| parent | 03f6205e3e4746dbc768a8a6ddc025f5a0a382ac (diff) | |
verity: Fix memory leaks
This CL refactors this code slightly in order to fix memory leaks
reported by the static analyzer. In general, RAII classes are
stack-allocated and moveable, rather than being heap-allocated. (If they
are heap-allocated, they should sit in a unique_ptr.)
Since it looks like this class has no children (adding `final` still
builds without issue), this devirtualizes its dtor, as well.
Finally, it looks like one instance of this class can easily be replaced
with a stack variable, so that's done, too.
Bug: None
Test: Built with the analyzer. Complaints are gone.
Change-Id: I6d284b06828afd47987534720bdaaa99e54b2c4c
Diffstat (limited to 'tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp')
0 files changed, 0 insertions, 0 deletions