[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: pkg/34723 (pkgsrc patch crashes links and links-gui)
On Tue, 19 Aug 2008, Christian Biere wrote:
> Christian Biere wrote:
> > wiz%netbsd.org@localhost wrote:
> > > Synopsis: pkgsrc patch crashes links and links-gui
> > >
> > > Responsible-Changed-From-To: pkg-manager->is
> > > Responsible-Changed-By: wiz%netbsd.org@localhost
> > > Responsible-Changed-When: Fri, 06 Oct 2006 07:19:07 +0000
> > > Responsible-Changed-Why:
> > > is added the patch.
> > Index: distinfo
> > ===================================================================
> > RCS file: /cvsroot/pkgsrc/www/links/distinfo,v
> > retrieving revision 1.47
> > diff -u -p -r1.47 distinfo
> > --- distinfo 21 May 2008 14:00:40 -0000 1.47
> > +++ distinfo 20 Jun 2008 21:27:43 -0000
> > @@ -1,7 +1,6 @@
> > -$NetBSD: distinfo,v 1.47 2008/05/21 14:00:40 wiz Exp $
> > +$NetBSD$
> > SHA1 (links-2.1pre36.tar.bz2) = 79ff4ff2d663de99ba590e11f42a52435a6e1809
> > RMD160 (links-2.1pre36.tar.bz2) = e34d61aa6db196d897afd4f07c1eb6f3a87dd4eb
> > Size (links-2.1pre36.tar.bz2) = 3824394 bytes
> > -SHA1 (patch-aa) = 374bd3262b4f50712cd42898d67e90ebcba72577
> > SHA1 (patch-ab) = a2d461c9d8b6300469ab6195886830fdd63be837
> I find it unbelievable that after almost TWO YEARS, this patch which
> does nothing but adding bugs, is still in pkgsrc.
> The patch can be seen here:
> Even without looking at the context of it, the patch is really poor C and has
> multiple bugs in itself.
Surely, it's broken.
The function goto_url isn't supposed to modify its argument (if you need
to modify it, create copy of the string with stracpy and free it at the
end of the function with mem_free). I should have marked the argument as
"const unsigned char *url", but at that time I didn't use "const" and I
kept this information in my head.
And the bug when the string is empty or contains only spaces...
Main Index |
Thread Index |