bugFreeciv - Bugs: bug #18975, in unit_virtual_destroy()...

 
 
Show feedback again

bug #18975: in unit_virtual_destroy() [unit.c::1732]: assertion '!unit_transported(punit)' failed.

Submitted by:  Jacob Nevins <jtn>
Submitted on:  Fri 11 Nov 2011 01:24:36 AM UTC  
 
Category: clientSeverity: 3 - Normal
Priority: 5 - NormalStatus: Fixed
Assigned to: Matthias Pfafferodt <syntron>Open/Closed: Closed
Release: S2_4 r20538Operating System: GNU/Linux
Planned Release: 2.4.0,2.5.0

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

Fri 09 Dec 2011 07:14:23 PM UTC, SVN revision 20639:

fix destroying of units if they are transported or transporter

- move some asserts after the cleanup routines
- fix some access to NULL pointers (ptrans in
unit_transport_unload())

see gna bug #18975
reported by Jacob Nevins

(Browse SVN revision 20639)

Matthias Pfafferodt <syntron>
Project MemberIn charge of this item.
Fri 09 Dec 2011 07:13:38 PM UTC, SVN revision 20638:

fix destroying of units if they are transported or transporter

- move some asserts after the cleanup routines
- fix some access to NULL pointers (ptrans in
unit_transport_unload())

see gna bug #18975
reported by Jacob Nevins

(Browse SVN revision 20638)

Matthias Pfafferodt <syntron>
Project MemberIn charge of this item.
Wed 16 Nov 2011 09:59:58 PM UTC, comment #2:

Confirm that trunk r20575 with this patch quits quieter than without, thanks. (Didn't try S2_4.)

Jacob Nevins <jtn>
Project Administrator
Sun 13 Nov 2011 11:32:26 PM UTC, comment #1:

Could you test the attached patch?

(file #14550)

Matthias Pfafferodt <syntron>
Project MemberIn charge of this item.
Fri 11 Nov 2011 01:24:36 AM UTC, original submission:

Latest S2_4 (r20538). Also seen with trunk.

  1. Load attached savegame with freeciv-gtk2.
  2. Quit the client immediately.

On quitting, the client throws these assertion failures:

Backtrace from one of them with gdb:

Jacob Nevins <jtn>
Project Administrator

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #14513:  freeciv-T0265-Y01780-auto.sav.bz2 added by jtn (66kB - application/x-bzip - savefile generating assertion failures)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by syntron (Updated the item)
  • -unavailable- added by jtn (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 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 09 Dec 2011 07:14:57 PM UTCsyntronStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Sun 13 Nov 2011 11:32:26 PM UTCsyntronAttached File-=>Added 0006-fix-destroying-of-units-if-they-are-transported-or-t.patch, #14550
      StatusNone=>Ready For Test
      Assigned toNone=>syntron
    Fri 11 Nov 2011 01:24:36 AM UTCjtnAttached File-=>Added freeciv-T0265-Y01780-auto.sav.bz2, #14513
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup