Always call archive_write_finish_entry even when copy_data fails

If copy_data throws and returns early, archive_write_finish_entry was
skipped, leaving the file handle open. On Windows this blocks unlink()
in the test cleanup. Call finish_entry unconditionally after
archive_write_header succeeds, then return if copy_data had failed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: cataphract

libarchive.c

@@ -331,11 +331,15 @@ PHP_METHOD(libarchive_Archive, extractCurrent)
         return;
     }
 
-    if (!copy_data(arch_obj->archive, arch_obj->arch_disk)) {
-        return;
-    }
+    bool data_ok = copy_data(arch_obj->archive, arch_obj->arch_disk);
 
+    /* Always finish the entry to close the file handle, even if copy_data
+     * failed — otherwise the handle stays open until the object is freed,
+     * blocking unlink() on Windows. */
     res = archive_write_finish_entry(arch_obj->arch_disk);
+    if (!data_ok) {
+        return;
+    }
     if (res != ARCHIVE_OK) {
         zend_throw_exception_ex(except_ce, archive_errno(arch_obj->arch_disk),
                                 "Could not finish writing entry: %s",