From dc9d09a8218718c447e9f33ef834d0c6f7ad5363 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tim=20D=C3=BCsterhus?= <tim@bastelstu.be>
Date: Tue, 22 Apr 2025 20:29:14 +0200
Subject: [PATCH] fileinfo: Stop calling `finfo_close()` in tests

This function is a noop and will be proposed for deprecation. This patch
removes the useless calls.
---
 ext/fileinfo/tests/bug79756.phpt                 | 1 -
 ext/fileinfo/tests/cve-2014-1943-mb.phpt         | 2 --
 ext/fileinfo/tests/cve-2014-1943.phpt            | 2 --
 ext/fileinfo/tests/cve-2014-3538-mb.phpt         | 1 -
 ext/fileinfo/tests/cve-2014-3538-nojit.phpt      | 1 -
 ext/fileinfo/tests/cve-2014-3538.phpt            | 1 -
 ext/fileinfo/tests/finfo_buffer_basic-mb.phpt    | 1 -
 ext/fileinfo/tests/finfo_buffer_basic.phpt       | 1 -
 ext/fileinfo/tests/finfo_set_flags_basic-mb.phpt | 2 --
 ext/fileinfo/tests/finfo_set_flags_basic.phpt    | 2 --
 ext/fileinfo/tests/finfo_upstream.phpt           | 1 -
 11 files changed, 15 deletions(-)

diff --git a/ext/fileinfo/tests/bug79756.phpt b/ext/fileinfo/tests/bug79756.phpt
index b4ea28baaa9b2..ad5478b62de8e 100644
--- a/ext/fileinfo/tests/bug79756.phpt
+++ b/ext/fileinfo/tests/bug79756.phpt
@@ -7,7 +7,6 @@ fileinfo
 $filename = __DIR__ . '/bug79756.xls';
 $finfo = finfo_open(FILEINFO_MIME);
 $mime = finfo_file($finfo, $filename);
-finfo_close($finfo);
 echo $mime;
 ?>
 --EXPECT--
diff --git a/ext/fileinfo/tests/cve-2014-1943-mb.phpt b/ext/fileinfo/tests/cve-2014-1943-mb.phpt
index 9dd4a9ec18a5a..01cf70b811bea 100644
--- a/ext/fileinfo/tests/cve-2014-1943-mb.phpt
+++ b/ext/fileinfo/tests/cve-2014-1943-mb.phpt
@@ -15,13 +15,11 @@ $m =  "0           byte        x\n".
 file_put_contents($fd, $a);
 $fi = finfo_open(FILEINFO_NONE);
 var_dump(finfo_file($fi, $fd));
-finfo_close($fi);
 
 file_put_contents($fd, $b);
 file_put_contents($fm, $m);
 $fi = finfo_open(FILEINFO_NONE, $fm);
 var_dump(finfo_file($fi, $fd));
-finfo_close($fi);
 ?>
 Done
 --CLEAN--
diff --git a/ext/fileinfo/tests/cve-2014-1943.phpt b/ext/fileinfo/tests/cve-2014-1943.phpt
index 35f66148ae79c..1aee196d16ffc 100644
--- a/ext/fileinfo/tests/cve-2014-1943.phpt
+++ b/ext/fileinfo/tests/cve-2014-1943.phpt
@@ -15,13 +15,11 @@ $m =  "0           byte        x\n".
 file_put_contents($fd, $a);
 $fi = finfo_open(FILEINFO_NONE);
 var_dump(finfo_file($fi, $fd));
-finfo_close($fi);
 
 file_put_contents($fd, $b);
 file_put_contents($fm, $m);
 $fi = finfo_open(FILEINFO_NONE, $fm);
 var_dump(finfo_file($fi, $fd));
-finfo_close($fi);
 ?>
 Done
 --CLEAN--
diff --git a/ext/fileinfo/tests/cve-2014-3538-mb.phpt b/ext/fileinfo/tests/cve-2014-3538-mb.phpt
index e6c63e35ac852..3b158a26b7264 100644
--- a/ext/fileinfo/tests/cve-2014-3538-mb.phpt
+++ b/ext/fileinfo/tests/cve-2014-3538-mb.phpt
@@ -19,7 +19,6 @@ $fi = finfo_open(FILEINFO_NONE);
 $t = microtime(true);
 var_dump(finfo_file($fi, $fd));
 $t = microtime(true) - $t;
-finfo_close($fi);
 if ($t < 3) {
     echo "Ok\n";
 } else {
diff --git a/ext/fileinfo/tests/cve-2014-3538-nojit.phpt b/ext/fileinfo/tests/cve-2014-3538-nojit.phpt
index 2010d538da951..73ecd09561d84 100644
--- a/ext/fileinfo/tests/cve-2014-3538-nojit.phpt
+++ b/ext/fileinfo/tests/cve-2014-3538-nojit.phpt
@@ -23,7 +23,6 @@ $fi = finfo_open(FILEINFO_NONE);
 $t = microtime(true);
 var_dump(finfo_file($fi, $fd));
 $t = microtime(true) - $t;
-finfo_close($fi);
 if ($t < 1.5) {
     echo "Ok\n";
 } else {
diff --git a/ext/fileinfo/tests/cve-2014-3538.phpt b/ext/fileinfo/tests/cve-2014-3538.phpt
index f15e745fc05d3..d19278d1dc325 100644
--- a/ext/fileinfo/tests/cve-2014-3538.phpt
+++ b/ext/fileinfo/tests/cve-2014-3538.phpt
@@ -19,7 +19,6 @@ $fi = finfo_open(FILEINFO_NONE);
 $t = microtime(true);
 var_dump(finfo_file($fi, $fd));
 $t = microtime(true) - $t;
-finfo_close($fi);
 if ($t < 1.5) {
     echo "Ok\n";
 } else {
diff --git a/ext/fileinfo/tests/finfo_buffer_basic-mb.phpt b/ext/fileinfo/tests/finfo_buffer_basic-mb.phpt
index 7426c909d9763..0ed6d48c803cb 100644
--- a/ext/fileinfo/tests/finfo_buffer_basic-mb.phpt
+++ b/ext/fileinfo/tests/finfo_buffer_basic-mb.phpt
@@ -27,7 +27,6 @@ foreach( $options as $option ) {
     foreach( $buffers as $string ) {
         var_dump( finfo_buffer( $finfo, $string, $option ) );
     }
-    finfo_close( $finfo );
 }
 
 ?>
diff --git a/ext/fileinfo/tests/finfo_buffer_basic.phpt b/ext/fileinfo/tests/finfo_buffer_basic.phpt
index e338583fab702..6a6e6a7a862cb 100644
--- a/ext/fileinfo/tests/finfo_buffer_basic.phpt
+++ b/ext/fileinfo/tests/finfo_buffer_basic.phpt
@@ -27,7 +27,6 @@ foreach( $options as $option ) {
     foreach( $buffers as $string ) {
         var_dump( finfo_buffer( $finfo, $string, $option ) );
     }
-    finfo_close( $finfo );
 }
 
 ?>
diff --git a/ext/fileinfo/tests/finfo_set_flags_basic-mb.phpt b/ext/fileinfo/tests/finfo_set_flags_basic-mb.phpt
index f366972b613e1..a4d723f86c9ca 100644
--- a/ext/fileinfo/tests/finfo_set_flags_basic-mb.phpt
+++ b/ext/fileinfo/tests/finfo_set_flags_basic-mb.phpt
@@ -12,8 +12,6 @@ echo "*** Testing finfo_set_flags() : basic functionality ***\n";
 var_dump( finfo_set_flags( $finfo, FILEINFO_NONE ) );
 var_dump( finfo_set_flags( $finfo, FILEINFO_SYMLINK ) );
 
-finfo_close( $finfo );
-
 // OO way
 $finfo = new finfo( FILEINFO_NONE, $magicFile );
 var_dump( $finfo->set_flags( FILEINFO_MIME ) );
diff --git a/ext/fileinfo/tests/finfo_set_flags_basic.phpt b/ext/fileinfo/tests/finfo_set_flags_basic.phpt
index 948031a0b8006..db71447ae2393 100644
--- a/ext/fileinfo/tests/finfo_set_flags_basic.phpt
+++ b/ext/fileinfo/tests/finfo_set_flags_basic.phpt
@@ -12,8 +12,6 @@ echo "*** Testing finfo_set_flags() : basic functionality ***\n";
 var_dump( finfo_set_flags( $finfo, FILEINFO_NONE ) );
 var_dump( finfo_set_flags( $finfo, FILEINFO_SYMLINK ) );
 
-finfo_close( $finfo );
-
 // OO way
 $finfo = new finfo( FILEINFO_NONE, $magicFile );
 var_dump( $finfo->set_flags( FILEINFO_MIME ) );
diff --git a/ext/fileinfo/tests/finfo_upstream.phpt b/ext/fileinfo/tests/finfo_upstream.phpt
index ef5ec17b03706..2a28bd7f776c0 100644
--- a/ext/fileinfo/tests/finfo_upstream.phpt
+++ b/ext/fileinfo/tests/finfo_upstream.phpt
@@ -18,7 +18,6 @@ foreach($lst as $p) {
 	if ($i !== $exp) {
 		echo "'$p' failed\nexp: '$exp'\ngot: '$i'\n";
 	} 
-	finfo_close($finfo);
 }
 
 echo "==DONE==";