Changes between Initial Version and Version 1 of Ticket #12286, comment 11


Ignore:
Timestamp:
Aug 6, 2015, 11:35:29 AM (9 years ago)
Author:
ttcoder

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #12286, comment 11

    initial v1  
    2323Is it worth filing another ticket for that leaked thread ?
    2424
     25
     26'''EDIT:''' for cinfirming the leak/or not, might be worth mentionning more data: the second leak item is this
     27
     28{{{
     29allocation: base: 0x45e1fc8; size: 56; thread: 2308; alignment: 1
     30  (..)
     31        <libroot.so> __builtin_new + 0x29
     32        <libbe.so> BPrivate::MouseDownThread<BMenuField>::TrackMouse(BMenuField *, void (BMenuField::*)(BMenuField *, BPoint), void (BMenuField::*)(BMenuField *, BPoint, unsigned long), long long) + 0x1e
     33}}}
     34
     35And looking at the code, there is indeed an operator-new [http://code.metager.de/source/xref/haiku/headers/private/shared/Thread.h#349 here] but the corresponding delete in Track() is never reached because of the kill_thread() [http://code.metager.de/source/xref/haiku/headers/private/shared/Thread.h#369 here] ; so we do leak the MouseDownThread object whenever the menu is opened and closed, right ?
     36
     37
    2538Once the above is dealt with I'll return to the supposed BMenu leak..
    2639Or maybe before embarking on that, it would be interesting compile libbe.so with [http://cgit.haiku-os.org/haiku/tree/src/kits/interface/Menu.cpp#n50 this define] commented out and see if the port leak disappears?