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 627420 - ODF import crazy format
ODF import crazy format
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: import/export OOo / OASIS
git master
Other All
: Normal normal
: ---
Assigned To: Andreas J. Guelzow
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2010-08-19 19:40 UTC by Morten Welinder
Modified: 2010-08-20 05:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Corrupted ods file (15.76 KB, application/vnd.oasis.opendocument.spreadsheet)
2010-08-19 19:40 UTC, Morten Welinder
Details

Description Morten Welinder 2010-08-19 19:40:18 UTC
Created attachment 168313 [details]
Corrupted ods file

(gdb) p *state->cur_format.accum
$4 = {str = 0x7fff6d86e010 "mm\":\"ss.", '0' <repeats 192 times>..., len = 978054072, allocated_len = 1073741824}

That's a mighty long format!

OO_DATE_SECONDS_PRINT_SECONDS should limit the number of digits to (say) 6.
And it should use go_string_append_c_n.
Comment 1 Morten Welinder 2010-08-19 20:06:48 UTC
The comment of limits and go_string_append_c_n applies here too:

	while (min_n_digits-- > 0)
		g_string_append_c (state->cur_format.accum, '0');

(Multiple occurrences)
Comment 2 Andreas J. Guelzow 2010-08-20 03:53:12 UTC
go_string_append_c_n ? I didn't even know that existed.
Comment 3 Andreas J. Guelzow 2010-08-20 05:00:43 UTC
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.