After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 358415 - crash in gantt view
crash in gantt view
Status: RESOLVED FIXED
Product: planner
Classification: Other
Component: General
0.14.x
Other Windows
: Normal blocker
: ---
Assigned To: planner-maint
planner-maint
: 360071 425796 451125 456248 476966 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-09-30 02:21 UTC by Michael Tedder
Modified: 2007-09-15 06:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Planner file which causes the crash (29.29 KB, application/octet-stream)
2006-09-30 02:25 UTC, Michael Tedder
  Details
one-line patch to prevent crashing (668 bytes, patch)
2006-10-24 12:43 UTC, Arthur Petitpierre
none Details | Review
One-line patch to fix Gantt chart crash. (305 bytes, patch)
2006-10-24 14:13 UTC, mdpoole
none Details | Review

Description Michael Tedder 2006-09-30 02:21:33 UTC
After creating resources and tasks in Planner 0.14, attempting to use the Gantt view to specify predecessors caused Planner to crash.  Upgrading to Planner 0.14.1 improved this, but after adding a fairly complete predecessor hierarchy, even 0.14.1 still continues to crash -- so much that the Gantt view is completely unusable at any zoom level.

Planner 0.13.90 does not exhibit any crash problems.

Attached is the .planner file I am using.  I have already wiped the email addresses, and changed some of the task names to make this .planner file available for public use.
Comment 1 Michael Tedder 2006-09-30 02:25:36 UTC
Created attachment 73685 [details]
Planner file which causes the crash
Comment 2 mdpoole 2006-10-18 14:32:56 UTC
It seems likely that this is related to bug #345517 -- cur_unit and cur_cal are used similarly in gantt_row_draw(), but only cur_unit was initialized to fix that crash.  valgrind complains that cur_cal is used uninitialized by map_get_next_mrp_ival().  Adding "cur_cal = (GList *)NULL;" quashes that warning from Valgrind but I haven't yet run the patched version long enough to be sure the crash is entirely fixed.
Comment 3 Arthur Petitpierre 2006-10-24 12:43:05 UTC
Created attachment 75296 [details] [review]
one-line patch to prevent crashing

little patch that seems to fix this crash.
Arthur Petitpierre.
Comment 4 mdpoole 2006-10-24 14:13:37 UTC
Created attachment 75306 [details] [review]
One-line patch to fix Gantt chart crash.

This patch actually compiles and follows the pattern for the previous crash fix.  (It is what I use as my local workaround.)
Comment 5 Daniel Holbach 2006-10-27 11:22:42 UTC
The patches both look good. Does any planner maintainer care to comment?
Comment 6 Kurt Maute 2006-12-12 23:32:12 UTC
Patches applied and included in Planner v0.14.2
Comment 7 Pascal Terjan 2007-03-30 09:41:11 UTC
*** Bug 360071 has been marked as a duplicate of this bug. ***
Comment 8 Pascal Terjan 2007-04-03 22:33:10 UTC
*** Bug 425694 has been marked as a duplicate of this bug. ***
Comment 9 Pascal Terjan 2007-04-03 22:34:30 UTC
*** Bug 425796 has been marked as a duplicate of this bug. ***
Comment 10 Pascal Terjan 2007-06-28 15:32:55 UTC
*** Bug 451125 has been marked as a duplicate of this bug. ***
Comment 11 Pascal Terjan 2007-08-27 09:14:39 UTC
*** Bug 456248 has been marked as a duplicate of this bug. ***
Comment 12 Pascal Terjan 2007-09-15 06:27:37 UTC
*** Bug 476966 has been marked as a duplicate of this bug. ***