bugFreeciv - Bugs: bug #21553, "Island-based" generator...

 
 
Show feedback again

bug #21553: "Island-based" generator memory leak

Submitted by:  pepeto <pepeto>
Submitted on:  Wed 29 Jan 2014 12:39:15 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Priority: 5 - NormalStatus: Fixed
Assigned to: pepeto <pepeto>Open/Closed: Closed
Release: S2_3,S2_4,S2_5,trunkOperating System: None
Planned Release: 2.4.3,2.5.0,2.6.0

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

Mon 12 May 2014 12:04:56 PM UTC, SVN revision 24880:

Free correctly the terrain selection lists for the island generator, not only
their elements. Also use genlist features to free the elements.

Reported by me (with the help of valgrind)

See bug #21553

(Browse SVN revision 24880)

pepeto <pepeto>
Project MemberIn charge of this item.
Mon 12 May 2014 12:04:52 PM UTC, SVN revision 24879:

Free correctly the terrain selection lists for the island generator, not only
their elements. Also use genlist features to free the elements.

Reported by me (with the help of valgrind)

See bug #21553

(Browse SVN revision 24879)

pepeto <pepeto>
Project MemberIn charge of this item.
Mon 12 May 2014 12:04:47 PM UTC, SVN revision 24878:

Free correctly the terrain selection lists for the island generator, not only
their elements. Also use genlist features to free the elements.

Reported by me (with the help of valgrind)

See bug #21553

(Browse SVN revision 24878)

pepeto <pepeto>
Project MemberIn charge of this item.
Wed 29 Jan 2014 12:39:15 AM UTC, original submission:

Terrain selection lists are never freed. Fix attached.

Valgrind output:

pepeto <pepeto>
Project MemberIn charge of this item.

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #19886:  mapgen_mem_leak.diff added by pepeto (3kB - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by pepeto (Submitted the item)
  •  

    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 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 12 May 2014 12:05:11 PM UTCpepetoStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Wed 07 May 2014 09:25:07 PM UTCpepetoAssigned toNone=>pepeto
      Planned Release=>2.4.3,2.5.0,2.6.0
    Wed 29 Jan 2014 12:39:15 AM UTCpepetoAttached File-=>Added mapgen_mem_leak.diff, #19886
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup