bugFreeciv - Bugs: bug #22216, trunk server crashes in...

Show feedback again

bug #22216: trunk server crashes in execute_orders()

Submitted by:  pepeto <pepeto>
Submitted on:  Wed Jun 18 09:44:32 2014  
Category: generalSeverity: 5 - Blocker
Priority: 5 - NormalStatus: Fixed
Assigned to: pepeto <pepeto>Open/Closed: Closed
Release: trunk r25193Operating System: None
Planned Release: 2.6.0Contains string changes: None

Add a New Comment (Rich MarkupRich Markup):

You are not logged in

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


Wed Aug 13 08:39:49 2014, SVN revision 25889:

Fix a typo which was causing the server to crash when loading road unit orders
from a 2.5.x savegame.

See gna bug #22216

(Browse SVN revision 25889)

pepeto <pepeto>
Project MemberIn charge of this item.
Thu Jul 31 11:34:17 2014, comment #4:

I won't be able to commit the patch before a while. Feel free to manage this ticket.

pepeto <pepeto>
Project MemberIn charge of this item.
Thu Jul 24 11:50:06 2014, comment #3:

Fixed typo for bug #21143.

(file #21540)

pepeto <pepeto>
Project MemberIn charge of this item.
Thu Jul 24 06:40:58 2014, comment #2:

See also bug #22364.

pepeto <pepeto>
Project MemberIn charge of this item.
Thu Jun 19 00:07:29 2014, comment #1:

The savegame in question has been saved from freeciv-2.5, so it doesn't (and is not expected to) contain target extras. I'm assuming I missed handling of orders when implementing compatibility with pre-2.6 savegames.

Marko Lindqvist <cazfi>
Project Administrator
Wed Jun 18 09:44:32 2014, original submission:

To reproduce:
1) load file #20920 (for bug #22136) ;
2) press turn done.

It appear the extra of the order is NULL. I suppose this is problem when loading the savegame because I didn't encountered this segmentation fault playing normally.

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):

Attached Files
file #21540:  load_S2_5_unit_orders.patch added by pepeto (620B - text/x-patch)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by cazfi (Posted a comment)
  • -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.


    Error: not logged in



    Follow 9 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed Aug 13 08:39:59 2014pepetoOpen/ClosedOpen=>Closed
    Wed Aug 13 08:39:58 2014pepetoStatusReady For Test=>Fixed
    Mon Aug 11 08:42:09 2014pepetoAssigned toNone=>pepeto
    Thu Jul 31 11:34:17 2014pepetoStatusNone=>Ready For Test
      Assigned topepeto=>None
    Wed Jul 30 22:55:59 2014cazfiAssigned tocazfi=>pepeto
    Thu Jul 24 11:50:06 2014pepetoAttached File-=>Added load_S2_5_unit_orders.patch, #21540
    Thu Jun 19 00:07:29 2014cazfiCategoryNone=>general
      Assigned toNone=>cazfi
    Show feedback again

    Back to the top

    Powered by Savane 3.1-cleanup