Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:45063
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 72.14.220.157 as permitted sender)
DomainKey-Status: bad
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:date:message-id:subject:from:to:content-type
         :content-transfer-encoding;
        b=JUMy4qIDHgJwuksQsPGeIR4SGesu4xkJusjKQ5gQxNoq94qdEYro6QnzU3bsgIyAAh
         2c5lrIb/sn4jVBlpGrCqPhiuEWxd9ZneH1iT0KM3vEZVRiBnWhDAx11rwPTZBkZgMAiF
         WKTV7sxXoPgNAoTVhuoeqPCwAjxYJ+WgOBWgQ=
MIME-Version: 1.0
Date: Thu, 23 Jul 2009 16:20:54 +0300
Message-ID: <8d3051340907230620t27c82e8eve9e21e9ae06604e0@mail.gmail.com>
To: PHP Development <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Subject: ZipArchive problems. Possible bug (Livelock)
From: torokalpar@gmail.com (=?UTF-8?B?QWxww6FyIFTDtnLDtmsg?=)

Hi all,

The short version is : on a corrupted zip, getStream never returns,
and eats up the whole processor. CHECKCONS doesn't help.

The long version:
  I have scripts that process data. Data arrives as zip archives. The
data import process halts, eating up all server CPU. I have analyzed
the incoming zips  using the test option of 7zip [1] , i found that
the the problem occurs with zips that contain one or more files for
witch 7zip reports that CRC has failed. These zips open without an
error, and once i try to get stream, for the faulty file, the call
never returns, and eats up the cpu. I tried to use the   CHECKCON flag
at opening, in the hope that the zip won't open, and thus i can just
throw it away, but it doesn't seem to help. Looking in the source i
wound on line 1174 of php_zip.c [2] , that no flag parameter is passed
to zip_open. I don't have a developement environment, so i can
recompile / and or debug this problem.

I have tested with 5.2.10



[1] http://www.7-zip.org/ - The 7zip homepage
[2] the line reads : rsrc_int->za = zip_open(resolved_path, 0, &err);
-- 
Alpar Torok