[Freeciv-Dev] [bug #16778] Client segfaults when buying while the cities report is open

2010-10-02 Thread pepeto

Update of bug #16778 (project freeciv):

  Status:None = Ready For Test 
 Planned Release: = 2.3.0  

___

Follow-up Comment #3:

Attached 2 fixes:
* the first to remove the store merging that doesn't work when the stores are
sorted (+miscellaneous cleanup);
* the second to fix the city report sorting.

Note that the crash you reported only happen when the city report is sorted.


(file #10552, file #10553)
___

Additional Item Attachment:

File name: trunk_remove_store_merging.diff Size:47 KB
File name: trunk_fix_cityrep_sorting.diff Size:0 KB


___

Reply to this item at:

  http://gna.org/bugs/?16778

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #16778] Client segfaults when buying while the cities report is open

2010-09-30 Thread Goswin von Brederlow

Additional Item Attachment, bug #16778 (project freeciv):

File name: freeciv-T0002-Y-3998-manual.sav.bz2 Size:19 KB


___

Reply to this item at:

  http://gna.org/bugs/?16778

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #16778] Client segfaults when buying while the cities report is open

2010-09-30 Thread Goswin von Brederlow

Follow-up Comment #2, bug #16778 (project freeciv):

Seems to be hapening in just any game. Attached a game anyway. Load game,
press f4, sort by Currently Building, select Michelangelos Chapel and press
buy.


___

Reply to this item at:

  http://gna.org/bugs/?16778

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #16778] Client segfaults when buying while the cities report is open

2010-09-29 Thread pepeto

Update of bug #16778 (project freeciv):

Severity:  3 - Normal = 4 - Important  


___

Reply to this item at:

  http://gna.org/bugs/?16778

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #16778] Client segfaults when buying while the cities report is open

2010-09-29 Thread pepeto

Follow-up Comment #1, bug #16778 (project freeciv):

Do you have a savegame I could reproduce the first crash?


___

Reply to this item at:

  http://gna.org/bugs/?16778

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #16778] Client segfaults when buying while the cities report is open

2010-09-28 Thread Goswin von Brederlow

URL:
  http://gna.org/bugs/?16778

 Summary: Client segfaults when buying while the cities
report is open
 Project: Freeciv
Submitted by: mrvn
Submitted on: Tuesday 09/28/2010 at 19:51
Category: client-gtk-2.0
Severity: 3 - Normal
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 2.2.99+svn18101
 Discussion Lock: Any
Operating System: GNU/Linux
 Planned Release: 

___

Details:

Opening the cities report (F4), sorting by Currently Building and then
buying something causes a segfault.

The problem seems to be that the client stores NULL entries in the gtk tree
in merge_list_stores(). This causes the compare function to be called with a
NULL string which segfaults when it tries to split it.

As a workaround I patched
int cityrepfield_compare(const char *str1, const char *str2)
to handle NULL strings. This avoids the segfault but then GTK complains
about

(freeciv-gtk2:6001): Gtk-CRITICAL **: gtk_list_store_move_after: assertion
`!GTK_LIST_STORE_IS_SORTED (store)' failed

and the list of cities shown in the report becomes incomplete. Closing and
reopening the report fixes that.

Patch attached.

MfG
   Goswin




___

File Attachments:


---
Date: Tuesday 09/28/2010 at 19:51  Name: freeciv-cities-segfault.patch  Size:
525B   By: mrvn
Prevent segfault when buying things. Does not fix the real problem.
http://gna.org/bugs/download.php?file_id=10494

___

Reply to this item at:

  http://gna.org/bugs/?16778

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev