Subject: Re: pkg/28889: bug in graphics/gdk-pixbuf
To: None <georg.schwarz@freenet.de>
From: Quentin Garnier <cube@cubidou.net>
List: pkgsrc-bugs
Date: 01/11/2005 01:32:53
--WplhKdTI2c8ulnbP
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Jan 06, 2005 at 08:48:00PM +0000, georg.schwarz@freenet.de wrote:
> >Number:         28889
> >Category:       pkg
> >Synopsis:       bug in graphics/gdk-pixbuf
> >Confidential:   no
> >Severity:       non-critical
> >Priority:       low
> >Responsible:    pkg-manager
> >State:          open
> >Class:          sw-bug
> >Submitter-Id:   net
> >Arrival-Date:   Thu Jan 06 20:48:00 +0000 2005
> >Originator:     Georg Schwarz
> >Release:        current pkgsrc
> >Organization:
> >Environment:
> IRIX lorenz 5.3 08031225 IP20 mips
> >Description:
> Since gdk-pixbuf 0.22 seems to be unmaintained by the Gnome project, mayb=
e pkgsrc can look into and fix, if necessary, the following issue:
> http://bugzilla.gnome.org/show_bug.cgi?id=3D156186
>=20
> In gdk-pixbuf io-gif.c line 620 reads
>   if (w > 0 && h > 0)
>   Shouldn=C5=BDt that be rather
>   if (*w > 0 && *h > 0)
>=20
> On my system at least I have to modify io-gif.c in order for compiling no=
t to fail.

It's obviously an error, as comparing pointers to 0 don't make much
sense, and we already know they're not NULL since the previous
chunk of code assumes they aren't.

I don't think it would harm to change to the obvious intended
behaviour.  However, it might break things for some unknown reason.
Meanwhile, I think it's safe to commit the fix, and we can always
failback to a simple avoidance of the test, as it always true.

How does it fail to compile, by the way?

--=20
Quentin Garnier - cube@cubidou.net - cube@NetBSD.org
"Commala-come-five! / Even when the shadows rise!
To see the world and walk the world / Makes ya glad to be alive."
Susannah's Song, The Dark Tower VI, Stephen King, 2004.

--WplhKdTI2c8ulnbP
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.6 (NetBSD)

iQEVAwUBQeMetdgoQloHrPnoAQJbZwf+PwGgnuZY9NKRsy9EMsqDkZSPqCyVXuZ1
sPWNKKxVkn7a7xO3eHer1pP5N0I1vUw6nxdnA4RS0XDA3EXX+cAEYQ404ee8nXaL
2Ay60Hw4aQZqN5PeL7eSYH2/soHFudlWdrf9bnOgsC/YpggNdV4tcEEUCUljCL5D
zoIRDiwpd7Wy2+z2EZgW4rB6BrbB2/+5+4JhIPfNMsgavFfHFpM+FJz6zcMlwczN
ekr346Dy8dgqdrEg/br42Xo75w2i+TvXFAQvPumvaOnxv3gGsyPTNZkesnVsX+CF
uTQa73BD1qtyX27QF0dVgTKdKXbGXoky6pi8PhtlSi4zMPKQKbcXoA==
=PUho
-----END PGP SIGNATURE-----

--WplhKdTI2c8ulnbP--