On Tue, 4 April 2006 16:56:51 +0100, Richard Purdie wrote:
> On Tue, 2006-04-04 at 17:33 +0200, Jörn Engel wrote:
> > On Tue, 4 April 2006 16:18:08 +0100, Richard Purdie wrote:
> > > Upgrade the zlib_inflate implementation in the kernel from a patched
> > > version 1.1.3 to a patched 1.2.3.
> >
> > s/1.1.3/1.1.4/
> >
> > I once pulled all the bugfixes between the versions into the kernel.
>
> That's not what the header says :)
Well, the complete patch from 1.1.3 to 1.1.4 was rather big. Every
single copyright line got updated to a current date, without any other
changes in those files. I found it rather silly and took the bugfixes
only. ;)
> > > +#if 0
> > > +int zlib_inflatePrime(z_streamp strm, int bits, int value)
> >
> > Was this code dead in 1.2.3 as well?
>
> This code was commented out in the kernel by Adrian Bunk as it has no
> users and I just followed that example when updating I'm sure if
> someone needed it, it could be enabled.
My thinking was rather that we could remove it. #if 0 has an
advantage in case you made a mistake - easy to revert. But after a
while...
> > > +#ifndef PKZIP_BUG_WORKAROUND
> >
> > For the kernel, we can remove compat code against DOS compilers, etc.
> > In this particular case, I believe we can just consider data
> > compressed with PKZIP to be illegal and throw an error.
>
> Agreed, there are places it can be tidied up a bit further. I was trying
> not to make too many changes so any future updates against zlib are
> easier.
Fair enough.
Jörn
--
Data expands to fill the space available for storage.
-- Parkinson's Law
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
