incubator-ooo-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject [Bug 120040] There is a memory leak in function SvBaseLink::~SvBaseLink()
Date Wed, 20 Jun 2012 14:14:09 GMT

Armin Le Grand <> changed:

           What    |Removed                     |Added
                 CC|                            |

--- Comment #3 from Armin Le Grand <> ---
ALG: SvBaseLink has three constructors, and only in two of them pImpl gets
allocated and thus initialized. Thus, in the 3rd constructor, pImpl will point
to a random address. Freeing it in the destructor could crash the office.

A minimal fix would have to init pImpl to zero and check before construction.
But even then, the implementation makes use of pImpl without tests, so maybe in
the 3rd constructor it should also be allocated. All in all, the whole class
would need more rework, all members initialized in all constructors, and pImpl
decided for 3rd constructor.

You are receiving this mail because:
You are the assignee for the bug.

View raw message