From: Pierre Joye <pierre.php@gmail.com>
Date: Fri, 17 May 2013 09:45:13 +0000 (+0200)
Subject: other fix for #64214, unmodified file may not have old fp set, causing other possible... 
X-Git-Tag: php-5.3.26RC1~6
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3eb1745643e6774dcd589705bf566516561d1cff;p=php

other fix for #64214, unmodified file may not have old fp set, causing other possible crashes
---

diff --git a/ext/phar/zip.c b/ext/phar/zip.c
index c8057e3bbe..6ba745e9cb 100644
--- a/ext/phar/zip.c
+++ b/ext/phar/zip.c
@@ -937,10 +937,11 @@ is_compressed:
 		PHAR_SET_32(local.uncompsize, entry->uncompressed_filesize);
 		PHAR_SET_32(central.compsize, entry->compressed_filesize);
 		PHAR_SET_32(local.compsize, entry->compressed_filesize);
-
-		if (-1 == php_stream_seek(p->old, entry->offset_abs, SEEK_SET)) {
-			spprintf(p->error, 0, "unable to seek to start of file \"%s\" while creating zip-based phar \"%s\"", entry->filename, entry->phar->fname);
-			return ZEND_HASH_APPLY_STOP;
+		if (p->old) {
+			if (-1 == php_stream_seek(p->old, entry->offset_abs, SEEK_SET)) {
+				spprintf(p->error, 0, "unable to seek to start of file \"%s\" while creating zip-based phar \"%s\"", entry->filename, entry->phar->fname);
+				return ZEND_HASH_APPLY_STOP;
+			}
 		}
 	}
 not_compressed: