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 497191 - Style leaks with derivatives.gnumeric
Style leaks with derivatives.gnumeric
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: import/export MS Excel (tm)
git master
Other All
: Normal normal
: ---
Assigned To: Jody Goldberg
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2007-11-15 21:10 UTC by sum1
Modified: 2007-11-19 14:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description sum1 2007-11-15 21:10:20 UTC 
Version: r16144
OS: Ubuntu Gutsy

Steps to reproduce:
- From gnumeric/src, convert derivatives.gnumeric to Excel (biff8) format:

./ssconvert --export-type=Gnumeric_Excel:excel_biff8 ../samples/derivatives.gnumeric /tmp/foo.xls

or

./ssconvert --export-type=Gnumeric_Excel:excel_dsf ../samples/derivatives.gnumeric /tmp/foo.xls

Note: I do not see any leak warnings with the Gnumeric_Excel:excel_biff7 exporter.


Console output:

Leaking style-color at 0x8092898 [0000:0000:0000].
Leaking style-color at 0x8090030 [ffff:ffff:ffff].
Leaking style-color at 0x81fcb98 [0000:0000:ffff].
Leaking style at 0x8070840.

...

** (/home/s/cvs/gnumeric/src/.libs/lt-ssconvert:12351): WARNING **: Leaked 40 nodes from style pool.
Leaking string [Sans] with ref_count=40.
Leaking GOFormat at 0x821e040 [0.00].
Leaking GOFormat at 0x8092aa8 [General].
Comment 1 Jody Goldberg 2007-11-16 03:27:39 UTC 
It's the hyperlinks.
We collect them but never free the style list.
Comment 2 Jody Goldberg 2007-11-16 03:47:53 UTC 
I've got a patch for the leak, and will look at the real problem that we do not export the links.

*** This bug has been marked as a duplicate of 81718 ***
Comment 3 Morten Welinder 2007-11-16 03:48:47 UTC 
Ooops.  I just plugged these leaks.
Comment 4 Jody Goldberg 2007-11-19 14:11:54 UTC 
81718 is now only about the HYPERLINK function, rather than export.
This is no long a dup, it is fixed in it's own right.