https://issues.apache.org/ooo/show_bug.cgi?id=120040

Armin Le Grand <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #3 from Armin Le Grand <[email protected]> ---
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.

Reply via email to