You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@openoffice.apache.org by bu...@apache.org on 2014/04/08 01:00:37 UTC

[Issue 124623] New: Memory leak with extension's Options page

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

          Issue ID: 124623
        Issue Type: DEFECT
           Summary: Memory leak with extension's Options page
           Product: General
           Version: OOo 3.3 or older
          Hardware: All
                OS: All
            Status: CONFIRMED
          Severity: normal
          Priority: P3
         Component: code
          Assignee: issues@openoffice.apache.org
          Reporter: arielch@apache.org

Steps "to reproduce it":

- Create a C++ extension that adds Options pages to OpenOffice's Options
dialog, see
https://wiki.openoffice.org/wiki/Documentation/DevGuide/Extensions/Options_Dialog

- The extension has to provide a component implementing
com.sun.star.awt.XContainerWindowEventHandler, see
https://wiki.openoffice.org/wiki/Documentation/DevGuide/Extensions/Saving_and_Reading_Data_for_the_Options_Page

- The lifetime of this C++ UNO object is controlled by reference count. The
application terminates without the object being deleted, with a reference count
> 0.

-- 
You are receiving this mail because:
You are the assignee for the issue.
You are watching all issue changes.

[Issue 124623] Memory leak with extension's Options page

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=124623

--- Comment #1 from Ariel Constenla-Haile <ar...@apache.org> ---
The extension's event handler is kept alive by the
dlgprov::DialogEventsAttacherImpl and its dlgprov::DialogUnoScriptListenerImpl
holding a reference to it. These in turn are kept alive because they are
listening at the toolkit controls. The toolkit controls are in turn kept alive
because the one that creates the container window never disposes it.

Disposing m_xPage in ExtensionsTabPage::~ExtensionsTabPage() fixes these memory
leaks.

-- 
You are receiving this mail because:
You are the assignee for the issue.
You are watching all issue changes.

[Issue 124623] Memory leak with extension's Options page

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=124623

--- Comment #2 from SVN Robot <sv...@dev.null.org> ---
"arielch" committed SVN revision 1585624 into trunk:
i124623 - Dispose extension's Options page to avoid memory leaks

-- 
You are receiving this mail because:
You are the assignee for the issue.
You are watching all issue changes.

[Issue 124623] Memory leak with extension's Options page

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=124623

brinzing <ol...@gmx.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |oliver.brinzing@gmx.de

--- Comment #4 from brinzing <ol...@gmx.de> ---
.

-- 
You are receiving this mail because:
You are watching all issue changes.

[Issue 124623] Memory leak with extension's Options page

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=124623

Ariel Constenla-Haile <ar...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
             Latest|---                         |4.1-dev
    Confirmation on|                            |
         Resolution|---                         |FIXED
           Assignee|issues@openoffice.apache.or |arielch@apache.org
                   |g                           |
   Target Milestone|---                         |4.2.0

--- Comment #3 from Ariel Constenla-Haile <ar...@apache.org> ---
Fixed with commit from comment 2.
Verified running AOO under valgrind.

-- 
You are receiving this mail because:
You are the assignee for the issue.
You are watching all issue changes.