ext/zip: guard getStream() and callback destructors against re-entrant close

Follow-up to the archive destructor use-after-free fix, plugging two more
paths where a progress or cancel callback firing during zip_close() can
re-enter the archive.

getStream() reached php_zip_archive_addref() with the archive already being
closed (refcount 0), tripping the ZEND_ASSERT(refcount > 0). Guard it with
php_zipobj_closing() so the call throws "Already being closed" instead.

Guard php_zip_progress_callback_free() and php_zip_cancel_callback_free()
like their callback counterparts, so the FCC destructor is not run while the
engine is inactive or in a bailout.

Author: jvoisin

ext/zip/php_zip.c

@@ -1128,6 +1128,10 @@ static void php_zip_progress_callback_free(void *ptr)
 {
 	php_zip_archive *archive = ptr;
 
+	if (UNEXPECTED(!EG(active) || archive->bailout_callback)) {
+		return;
+	}
+
 	if (ZEND_FCC_INITIALIZED(archive->progress_callback)) {
 		zend_fcc_dtor(&archive->progress_callback);
 	}
@@ -1139,6 +1143,10 @@ static void php_zip_cancel_callback_free(void *ptr)
 {
 	php_zip_archive *archive = ptr;
 
+	if (UNEXPECTED(!EG(active) || archive->bailout_callback)) {
+		return;
+	}
+
 	if (ZEND_FCC_INITIALIZED(archive->cancel_callback)) {
 		zend_fcc_dtor(&archive->cancel_callback);
 	}
@@ -3137,6 +3145,10 @@ static void php_zip_get_stream(INTERNAL_FUNCTION_PARAMETERS, int type, bool acce
 
 	ZIP_FROM_OBJECT(intern, self);
 
+	if (php_zipobj_closing(Z_ZIP_P(self))) {
+		RETURN_THROWS();
+	}
+
 	if (type) {
 		PHP_ZIP_STAT_PATH(intern, ZSTR_VAL(filename), ZSTR_LEN(filename), flags, sb);
 	} else {