bugGnash - The GNU Flash player - Bugs: bug #21842, Memory leak due to missing...

 
 

bug #21842: Memory leak due to missing ::unload call

Submitted by:  Sandro Santilli <strk>
Submitted on:  Tue 18 Dec 2007 09:06:23 PM UTC  
 
Category: coreSeverity: 5 - Blocker
Release: NoneStatus: Fixed
Privacy: PublicAssigned to: Sandro Santilli <strk>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Mon 21 Apr 2008 02:05:58 PM UTC, comment #3:

Zou: I think we should reopen the _unloaded and _destroyed
flags discussion. See bug #22982.

I think we should re-introduce the assertion in character::destroy(), and once we're there also
allow multiple calls to ::destroy maybe (to make
calling code easier)

Sandro Santilli <strk>
Project MemberIn charge of this item.
Wed 19 Dec 2007 01:24:01 AM UTC, comment #2:

I still don't like the idea of a destroyed but not unloaded character. isUnloaded() means the character is not on the stage.
And if it's destroyed it can't be on the stage.

Sandro Santilli <strk>
Project MemberIn charge of this item.
Wed 19 Dec 2007 12:53:44 AM UTC, comment #1:

I may look the memory leaks from this view: cleanupDisplayList() forgot to clear the destroyed characters.

Zou Lunkai <zoulunkai>
Tue 18 Dec 2007 09:06:23 PM UTC, original submission:

Attached SWF and source testcase for the problem I was talking with zou about introduced by a missing setting _unloaded flag to true
from withing ::destroy.

We might want to expose some GC inspecting interfaces in MovieTester to verify the managed resources don't grow over
a given threshold...

To build create an empty.as file (the movie needs to loop for things to keep being pushed on the global instance list)
and build with:

makeswf -o leak.swf testunload.as empty.as

Sandro Santilli <strk>
Project MemberIn charge of this item.

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #14655:  leak.swf added by strk (162B - application/x-shockwave-flash)
file #14656:  testunload.as added by strk (167B - application/x-applix-spreadsheet)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by zoulunkai (Posted a comment)
  • -unavailable- added by strk (Submitted the item)
  • -unavailable- added by strk
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 9 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 29 Dec 2007 01:40:42 PM UTCstrkStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Tue 18 Dec 2007 09:29:03 PM UTCstrkStatusNone=>Ready For Test
      Assigned toNone=>strk
    Tue 18 Dec 2007 09:06:36 PM UTCstrkCategoryNone=>core
      Severity3 - Normal=>5 - Blocker
    Tue 18 Dec 2007 09:06:23 PM UTCstrkAttached File-=>Added leak.swf, #14655
      Attached File-=>Added testunload.as, #14656
      Carbon-Copy-=>Added zoulunkai

    Back to the top


    Powered by Savane 3.1-cleanup1