Re: [PATCH] Implement -u (UNIX socket batch processing)

[Ingo Schwarze <schwarze@usta.de>]

Hi Michael,

Michael Stapelberg wrote on Sat, Jan 28, 2017 at 02:02:28PM +0100:

> Attaching revision 2 which resets ->gzip in mparse_reset(). I
> previously did not link against libz, so this issue escaped me.

independent of your added code, this part of your patch is
correct.  You found a genuine bug here, so i committed that
part of your patch, see below.

Thanks,
  Ingo


Log Message:
-----------
If an application parses multiple files with mparse_readfd(3) but 
without using mparse_open(3) to open the files, and if one of the
files includes a gzip'ed file with .so, then the gzip flag remains
set and the next main file will be expected to be gzip'ed.
Fix this by clearing the gzip flag in mparse_reset(3).

Bug found and patch provided by Michael <Stapelberg at debian dot org>.

Modified Files:
--------------
    mdocml:
        read.c

Revision Data
-------------
Index: read.c
===================================================================
RCS file: /home/cvs/mdocml/mdocml/read.c,v
retrieving revision 1.158
retrieving revision 1.159
diff -Lread.c -Lread.c -u -p -r1.158 -r1.159
--- read.c
+++ read.c
@@ -837,6 +837,7 @@ mparse_reset(struct mparse *curp)
 
 	free(curp->sodest);
 	curp->sodest = NULL;
+	curp->gzip = 0;
 }
 
 void
--
 To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv