Task #2501 (closed)
Opened 9 years ago
Closed 9 years ago
Bug: memory leak in omero/client.cpp
| Reported by: | jamoore | Owned by: | jamoore |
|---|---|---|---|
| Priority: | minor | Milestone: | OMERO-Beta4.2 |
| Component: | OmeroCpp | Version: | n.a. |
| Keywords: | n.a. | Cc: | |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | 0.0d |
| Sprint: | 2010-06-10 (11) |
Description
From Bernhard:
For your information, Paavo just ran a valgrind check over our application and found mem leaks in client.cpp. Those are not causing problems, I just wanted to let you know they are there (I guess they can easily be solved using smart pointers). ==21634== 5,700 (240 direct, 5,460 indirect) bytes in 5 blocks are definitely lost in loss record 162 of 187 ==21634== at 0x4C2506C: operator new(unsigned long) (in /usr/lib64/valgrind/amd64-linux/vgpreload_memcheck.so) ==21634== by 0x112CB968: omero::client::closeSession() (client.cpp:392) ==21634== by 0x10EAA779: ccinterface::CCInterface::~CCInterface() (CCInterface.h:1023) ==21634== by 0x10EB021A: Ncolumbus_conductor::OmeroConnection::~OmeroConnection() (columbus_conductor_blitz.cpp:58) It seems the culprit is line 392 in omero/client.cpp: __previous = new Ice::InitializationData(); which seems to be never deleted. Also the next line (393 in omero/client.cpp) is reported as producing memory leaks: (*__previous).properties = oldIc->getProperties()->clone();
Change History (2)
comment:1 Changed 9 years ago by jmoore
- Owner set to jmoore
- Status changed from new to assigned
comment:2 Changed 9 years ago by jmoore
- Remaining Time set to 0
- Resolution set to fixed
- Status changed from assigned to closed
Note: See
TracTickets for help on using
tickets.
You may also have a look at Agilo extensions to the ticket.
(In [7278]) Fix memory leak per createSession in OmeroCpp. Fix #2501