Re: gzip 1.3.6 released

[Paul Eggert]

Santiago Vila <address@hidden> writes:

> (It seems as if the "incomplete literal tree" were some sort of
> debug message which perhaps should not be shown at all).

Yes, that sounds right to me as well.  Thanks for reporting the problem.
I installed this fix.

2006-11-26  Paul Eggert  <address@hidden>

        * inflate.c (inflate_dynamic, inflate): Don't send output to
        stderr unless DEBUG is defined and 'verbose'.  Problem reported by
        Santiago Vila.

--- inflate.c   20 Nov 2006 08:40:33 -0000      1.4
+++ inflate.c   27 Nov 2006 00:51:01 -0000
@@ -856,7 +856,7 @@ int inflate_dynamic()
   if ((i = huft_build(ll, nl, 257, cplens, cplext, &tl, &bl)) != 0)
   {
     if (i == 1) {
-      fprintf(stderr, " incomplete literal tree\n");
+      Trace ((stderr, " incomplete literal tree\n"));
       huft_free(tl);
     }
     return i;                   /* incomplete code set */
@@ -865,7 +865,7 @@ int inflate_dynamic()
   if ((i = huft_build(ll + nl, nd, 0, cpdist, cpdext, &td, &bd)) != 0)
   {
     if (i == 1) {
-      fprintf(stderr, " incomplete distance tree\n");
+      Trace ((stderr, " incomplete distance tree\n"));
 #ifdef PKZIP_BUG_WORKAROUND
       i = 0;
     }
@@ -976,8 +976,6 @@ int inflate()
 
 
   /* return success */
-#ifdef DEBUG
-  fprintf(stderr, "<%u> ", h);
-#endif /* DEBUG */
+  Trace ((stderr, "<%u> ", h));
   return 0;
 }