Re: Dropping 'fringe' pixbuf loaders

On Tue, 2015-09-22 at 01:20 +0200, Bastien Nocera wrote:
On Mon, 2015-09-21 at 18:38 -0400, Matthias Clasen wrote:

On Mon, Sep 21, 2015 at 5:10 PM, Cosimo Cecchi <cosimoc gnome org>

On Mon, Sep 21, 2015 at 1:01 PM, Owen Taylor <otaylor redhat com>
Do we trust this code or not? If not, we should either a)
it or b) delete it.

Moving less-trusted loaders into a separate repo is a blame-
user or blame-the-os-vendor move, depending on who installs
onto the system.
The only way to prevent the blame game you mention in a typical
distribution where everything is installed through packages would
be to stop supporting out of tree modules entirely, if I
your concern correctly.

My point is that as long as that's the case, at least maintaining
them in a central location gives people an aggregation point for

But they are not being maintained by anybody, and the fixes have
been aggregating... every few years some security researchers
to have a look at image loaders, and then we get a bunch of
and corruptions reported, and either me of Benjamin grudgingly fix
them. And both of us are tired of doing that.

I would argue that at least I have taken care of some of that work at
the end of 2014. I didn't get to see coverity scans or cppchecks, but
this isn't the most complicated code to fix up and review.

If removing some of those loaders helps lighten the number of
bugs, sure, go for it.

As for removing those loaders, I'd double-check whether GIMP has
support for those (not through a gdk-pixbuf loader), so that at least
some modicum of support is left for those, making it less likely that
we'll crash when somebody has a duff file in their file manager.

I tested all those formats, and I think we can remove the PCX and TGA
loaders, and leave those to be loaded by GIMP, in the very short term.

For QTIF, either support in GIMP or in GStreamer (it's usually only
used along with QuickTime) would be fine.

ANI files really need to be supported by GIMP before we remove the
support for it.

In short, remove all the mentioned modules, except ANI, and file bugs
against GIMP/GStreamer for the unsupported ones.


Supported by GIMP:
PCX [2]
TGA [2]

Unsupported by GIMP:
ANI [1]
QTIF [3]
WBMP [4] (display broken in gdk-pixbuf)
RAS [5] (broken/unsupported in gdk-pixbuf)

[1]: Tests in shared-mime-info and gdk-pixbuf
[2]: Tests in shared-mime-info

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]