diff options
| author | Gary Benson <gbenson@redhat.com> | 2018-10-11 10:19:26 +0100 |
|---|---|---|
| committer | Gary Benson <gbenson@redhat.com> | 2018-10-11 10:19:26 +0100 |
| commit | fbe61a3661b083a666e6550b3b0c2de364e6d4a6 (patch) | |
| tree | 081ad1be61ca54cdef6de1de3641afb91062c86a /gdb/interps.c | |
| parent | a9597defaf39e2277c4c5ff510e708f226f54fbc (diff) | |
| download | gdb-fbe61a3661b083a666e6550b3b0c2de364e6d4a6.zip gdb-fbe61a3661b083a666e6550b3b0c2de364e6d4a6.tar.gz gdb-fbe61a3661b083a666e6550b3b0c2de364e6d4a6.tar.bz2 | |
Fix interp::m_name resource leak found by Coverity
This commit fixes a resource leak found by Coverity, where interp's
constructor allocated memory for m_name that interp's destructor did
not free.
gdb/ChangeLog:
* interps.h (interp::m_name): Make private and mutable.
* interps.c (interp::~interp): Free m_name.
Diffstat (limited to 'gdb/interps.c')
| -rw-r--r-- | gdb/interps.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/gdb/interps.c b/gdb/interps.c index 6fe4c74..883e042 100644 --- a/gdb/interps.c +++ b/gdb/interps.c @@ -84,7 +84,9 @@ interp::interp (const char *name) } interp::~interp () -{} +{ + xfree (m_name); +} /* An interpreter factory. Maps an interpreter name to the factory function that instantiates an interpreter by that name. */ |