bugGnome Chemistry Utils - Bugs: bug #30495, GChemPaint crashes when changing...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #30495: GChemPaint crashes when changing the text font

Submitted by:  Julian Sikorski <belegdol>
Submitted on:  Tue 20 Jul 2010 06:16:05 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: Jean Bréfort <jean_brefort>
Open/Closed: Closed

(Jump to the original submission Jump to the original submission)

Mon 26 Jul 2010 05:08:03 PM UTC, comment #16:

Patch fixed.

Jean Bréfort <jean_brefort>
Project AdministratorIn charge of this item.
Mon 26 Jul 2010 04:58:15 PM UTC, comment #15:

The new patch does not build.

Julian Sikorski <belegdol>
Project Member
Mon 26 Jul 2010 09:51:57 AM UTC, comment #14:

Just removed some printf debug from the latest patch.

(file #21070)

Jean Bréfort <jean_brefort>
Project AdministratorIn charge of this item.
Mon 26 Jul 2010 08:26:22 AM UTC, comment #13:

All patches commited. Fixed. Thanke for the bug report and the help for fixing it. Anyway, I don't intend to release now. I'll wait at least until the goffice issue is fixed.

Jean Bréfort <jean_brefort>
Project AdministratorIn charge of this item.
Sun 25 Jul 2010 09:53:54 PM UTC, comment #12:

Indeed, the problem seems to be gone now.

Julian Sikorski <belegdol>
Project Member
Sun 25 Jul 2010 09:27:49 PM UTC, comment #11:

This one should fix the issue. I'll commit tomorrow.

(file #21063)

Jean Bréfort <jean_brefort>
Project AdministratorIn charge of this item.
Sun 25 Jul 2010 09:25:34 PM UTC, comment #10:

This is all on Fedora 13, goffice 0.8.1. The backtrace with the new patch is now shorter:
With valgrind, going to preferences generates this:
and upon 2nd change of font the following appears:
Valgrind still seems to prevent the crash.

Julian Sikorski <belegdol>
Project Member
Sun 25 Jul 2010 08:37:49 PM UTC, comment #9:

Please try this new patch and tell me if something changes.

(file #21062)

Jean Bréfort <jean_brefort>
Project AdministratorIn charge of this item.
Sun 25 Jul 2010 07:11:33 PM UTC, comment #8:

OK, more info: Back to Fedora 13, I tried running gchempaint cluelessly under valgrind, and I could change the font as many times as I wanted. Here is what the console did show:

Julian Sikorski <belegdol>
Project Member
Sun 25 Jul 2010 07:03:20 PM UTC, comment #7:

On Fedora rawhide it actually behaves differently: it crashed on the 2nd font, with a different backtrace. When I installed all debuginfos, it started crashing on the 3rd font too. Backtrace is the following:

Julian Sikorski <belegdol>
Project Member
Sun 25 Jul 2010 06:58:26 PM UTC, comment #6:

The fonts do not seem to matter, this time I tried Liberation Mono, Liberation Sans and Liberation Sans Narrow.
I am not sure about the line number, the only patch the package contains is the one from comment #2, which adds 2 lines to view.cc.

Julian Sikorski <belegdol>
Project Member
Sun 25 Jul 2010 10:35:48 AM UTC, comment #5:

I can't reproduce that.

#5 seems to occur at line 409, while the call is actually at line 407 unless the file has been patch (might be a gdb issue).

Which fonts do you select to get the crash? Do they all have at least one face (that would be very strange if no face was available for a font).

Jean Bréfort <jean_brefort>
Project AdministratorIn charge of this item.
Sun 25 Jul 2010 09:14:00 AM UTC, comment #4:

pango-debuginfo was missing, here is the complete trace:

Julian Sikorski <belegdol>
Project Member
Sun 25 Jul 2010 09:08:10 AM UTC, comment #3:

With the patch you can change the font twice, and the third time it will kill the program.

Julian Sikorski <belegdol>
Project Member
Sun 25 Jul 2010 08:37:26 AM UTC, comment #2:

Fixed using attached patch.

(file #21058)

Jean Bréfort <jean_brefort>
Project AdministratorIn charge of this item.
Sat 24 Jul 2010 09:29:15 AM UTC, comment #1:

This is still present in 0.12.2:

Julian Sikorski <belegdol>
Project Member
Tue 20 Jul 2010 06:16:05 AM UTC, original submission:

Originally reported as https://bugzilla.redhat.com/show_bug.cgi?id=614417
If you go to Edit -> Preferences and try to change the text font, GChemPaint will crash. It also happens with 0.12.1. The backtrace is as follows:

Julian Sikorski <belegdol>
Project Member

 

Attached Files
file #21074:  fonts.patch added by jean_brefort (2KiB - text/x-patch)
file #21058:  30495.patch added by jean_brefort (375B - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by belegdol (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 13 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 08 Aug 2010 10:35:41 AM UTCjean_brefortOpen/ClosedOpen=>Closed
    Mon 26 Jul 2010 05:23:33 PM UTCjean_brefortAttached File-=>Added fonts.patch, #21074
    Mon 26 Jul 2010 05:07:48 PM UTCjean_brefortAttached File#21070=>Removed
    Mon 26 Jul 2010 09:51:57 AM UTCjean_brefortAttached File-=>Added fonts.patch, #21070
    Mon 26 Jul 2010 09:51:12 AM UTCjean_brefortAttached File#21062=>Removed
    Mon 26 Jul 2010 09:51:02 AM UTCjean_brefortAttached File#21063=>Removed
    Mon 26 Jul 2010 08:26:22 AM UTCjean_brefortStatusConfirmed=>Fixed
    Sun 25 Jul 2010 09:27:49 PM UTCjean_brefortAttached File-=>Added fonts.patch, #21063
    Sun 25 Jul 2010 08:37:49 PM UTCjean_brefortAttached File-=>Added fonts.patch, #21062
    Sun 25 Jul 2010 09:08:10 AM UTCbelegdolStatusFixed=>Confirmed
    Sun 25 Jul 2010 08:37:26 AM UTCjean_brefortAttached File-=>Added 30495.patch, #21058
      StatusNone=>Fixed
      Assigned toNone=>jean_brefort

    Back to the top


    Powered by Savane 3.1-cleanup1