<div dir="ltr"><div dir="ltr">Hi,  <div><br></div><div>There was probably a misunderstanding. I sent an email: <a href="https://mailman.astron.com/pipermail/file/2022-September/000856.html">https://mailman.astron.com/pipermail/file/2022-September/000856.html</a> with patch.</div><div>If I understand correctly this commit: <a href="https://github.com/file/file/commit/18f0474f698ef8f7247d61fc12a46c49dd7f7ed7">https://github.com/file/file/commit/18f0474f698ef8f7247d61fc12a46c49dd7f7ed7</a> is a response to the mentioned email.</div><div>With your fix we now ignore any errors that might occur in gzip decompression. For example:</div><div>$ touch x y</div><div>$ zip two_empty_files x y</div><div><div>$ gzip -cd two_empty_files.zip</div><div>   gzip: two_empty_files.zip.zip has more than one entry--rest ignored</div><div>$ echo $?</div><div>   2</div></div><div>$ file -z two_empty_files.zip</div><div>   two_empty_files.zip: empty (Zip archive data, at least v1.0 to extract, compression method=store)</div><div><br></div><div>My patch (in attachment) was implementing the idea that the OK state is when no error occurred. With this patch, output is:</div><div>$ file -z two_empty_files.zip</div><div>   z.zip: ERROR:[gzip: Stdin has more than one entry--rest ignored] (Zip archive data, at least v1.0 to extract, compression method=store)</div><div><br></div><div>Amended patch in attachment.</div><div><br></div><div>best regards,<br>vincent mihalkovic</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Sep 13, 2022 at 5:16 PM Vincent Mihalkovic <<a href="mailto:vmihalko@redhat.com">vmihalko@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hello, <div><br></div><div>reproducer from <a href="https://bugzilla.redhat.com/show_bug.cgi?id=2095828" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=2095828</a> is quite simple:<br></div><div>$ touch emptyFile<br>$ zip emptyFileArchive emptyFile</div><div>  adding: emptyFile (stored 0%)<br>$ file -z emptyFileArchive.zip</div><div>emptyFileArchive.zip: ERROR:[gzip: Read failed, No such file or directory] (Zip archive data, at least v1.0 to extract, compression method=store)</div><div><br></div><div>If we consider gzip flags "-cd" <a href="https://github.com/file/file/blob/master/src/compress.c#L134" target="_blank">https://github.com/file/file/blob/master/src/compress.c#L134</a></div><div>-c: Write output on standard output;</div><div>-d: Decompress;</div><div>and run:</div><div>$ gzip -cd emptyFileArchive.zip</div><div><br></div><div>gzip output is, as expected, empty, but due to this condition: <a href="https://github.com/file/file/blob/master/src/compress.c#L993" target="_blank">https://github.com/file/file/blob/master/src/compress.c#L993</a></div><div>file considers gzip output to be erroneous.</div><div><br></div><div>To achieve expected output:</div><div>$ file -z emptyFileArchive.zip</div><div>emptyFileArchive.zip: empty (Zip archive data, at least v1.0 to extract, compression method=store)<br></div><div>I propose the following patch.</div><div><br></div><div>best regards,</div><div>vincent mihalkovic</div></div>
</blockquote></div></div>