* src/buffer.c: Fix struct zip_magic to avoid problems with pedantic

[chaz/tar] / ChangeLog

diff --git a/ChangeLog b/ChangeLog
index 2f1ba59d45c219dd5af9d7785292528bbbefd379..41a025adab939332b0948af466b24b100a0621b2 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,24 @@
+2005-04-04  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * src/buffer.c (struct zip_magic): Use char arrays, not pointers.
+       The unsigned char * pointer ran afoul of pedantic C compilers, and
+       we didn't need pointers anyway.  Put the size field before the
+       data to avoid unnecessary padding.  All uses changed.
+       (magic) Make it const, since it doesn't change.  All used changed.
+
+2005-04-02  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * src/xheader.c (decode_record): Don't dump core when given
+       a corrupted extended header.  Problem reported by Jim Meyering.
+       Also, check for other ways that the header might be invalid,
+       e.g., missing newline at end.  Do not allow keys with nulls.
+       Allow blanks before and after length, as POSIX requires.
+       Do not allow leading "-" in length.  Check for length overflow.
+       (xheader_decode, xheader_decode_global): Let decode_record
+       check for exhaustion of record.
+       (xheader_read): Null-terminate the extended record;
+       decode_record relies on this.
+
 2005-03-21  Paul Eggert  <eggert@cs.ucla.edu>
 
        * bootstrap (TP_URL): Change from