diff --git a/ext/zip/php_zip.c b/ext/zip/php_zip.c index a723a187dbb..99adb723e38 100644 --- a/ext/zip/php_zip.c +++ b/ext/zip/php_zip.c @@ -1495,6 +1495,7 @@ static ZIPARCHIVE_METHOD(close) struct zip *intern; zval *self = getThis(); ze_zip_object *ze_obj; + int err; if (!self) { RETURN_FALSE; @@ -1504,7 +1505,8 @@ static ZIPARCHIVE_METHOD(close) ze_obj = Z_ZIP_P(self); - if (zip_close(intern)) { + if (err = zip_close(intern)) { + php_error_docref(NULL TSRMLS_CC, E_WARNING, zip_strerror(intern)); zip_discard(intern); } @@ -1513,7 +1515,11 @@ static ZIPARCHIVE_METHOD(close) ze_obj->filename_len = 0; ze_obj->za = NULL; - RETURN_TRUE; + if (!err) { + RETURN_TRUE; + } else { + RETURN_FALSE; + } } /* }}} */ diff --git a/ext/zip/tests/bug70322.phpt b/ext/zip/tests/bug70322.phpt new file mode 100644 index 00000000000..a9b4915914d --- /dev/null +++ b/ext/zip/tests/bug70322.phpt @@ -0,0 +1,29 @@ +--TEST-- +Bug #70322 (ZipArchive::close() doesn't indicate errors) +--DESCRIPTION-- +We want to test whether ZipArchive::close() returns FALSE and raises a warning +on failure, so we force the failure by adding a file to the archive, which we +delete before closing. +--SKIPIF-- + +--FILE-- +open($zipfile, ZipArchive::CREATE); +$zip->addFile($textfile); +unlink($textfile); +var_dump($zip->close()); +?> +--CLEAN-- + +--EXPECTF-- +Warning: ZipArchive::close(): Read error: No such file or directory in %s%ebug70322.php on line %d +bool(false) diff --git a/ext/zip/tests/oo_delete.phpt b/ext/zip/tests/oo_delete.phpt index 9eac8217348..69c6151d5ce 100644 --- a/ext/zip/tests/oo_delete.phpt +++ b/ext/zip/tests/oo_delete.phpt @@ -63,7 +63,8 @@ $sb = $zip->statIndex(1); var_dump($sb); $sb = $zip->statIndex(2); var_dump($sb); -$zip->close(); +// suppress irrelevant error message: +@$zip->close(); unset($zip); if (file_exists($file)) {