[ccache] ccache doesn't notice gzip CRC errors
Wilson Snyder
wsnyder at wsnyder.org
Sun Oct 31 17:07:58 MDT 2010
>> GZIP files contain CRC checksums, however ccache doesn't
>> notice CRC errors because it doesn't check gzerror at the
>> end of decompression. This patch fixes it. (It's arguably
>> a bad API decision on the gz library's part, but we're stuck
>> with it.)
>...
>
>> + gzeof(gz_in);
>
>Why is the above gzeof call needed? The implementation doesn't seem to
>modify the state and the documentation doesn't imply that it could.
You're right, that was left from my debugging; remove it.
>Looks good otherwise.
Thanks.
-Wilson
More information about the ccache
mailing list