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 170812 - Two blank layers not saved correctly into a winicon
Two blank layers not saved correctly into a winicon
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: Plugins
2.2.x
Other Linux
: Normal minor
: 2.2
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2005-03-18 16:34 UTC by Ari Pollak
Modified: 2008-01-15 12:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Ari Pollak 2005-03-18 16:34:30 UTC 
From Debian bug report http://bugs.debian.org/300140

1. Open the attached .xcf file in gimp.
2. Save it as a .ico with the default settings. Already there are a few warnings:
** (winicon:29273): CRITICAL **: ico_create_palette: assertion `cmap != NULL' failed

** (winicon:29273): CRITICAL **: ico_create_palette: assertion `cmap != NULL' failed
3. Now try opening the saved .ico file. The winicon plugin crashes:
GLib-ERROR **: gmem.c:158: failed to allocate 4294967292 bytes
aborting...

(gimp:29188): Gimp-Plug-In-WARNING **: gimp: plug_in_flush(): error: Broken pipe
Comment 1 weskaggs 2005-03-18 20:17:02 UTC 
The basic problem is happening around line 870 of winicon/icosave.c -- an
attempt to convert a temporary image to indexed mode is producing a result with
a NULL colormap.  I am wondering whether this might turn out, by a remarkable
coincidence, to be at root the same problem described in bug #170825.
Comment 2 Sven Neumann 2005-03-21 14:22:09 UTC 
Fixed in both branches:

2005-03-21  Sven Neumann  <sven@gimp.org>

	* plug-ins/winicon/icosave.c (ico_create_palette): fixed parameter
	check, NULL is valid for an empty palette. Fixes bug #170812.