misc: close leaked files in m.proj, g.extension, G_debug and an r.in.gdal test

python/grass/pygrass/modules/interface/env.py

@@ -33,4 +33,8 @@ def G_debug(level, *msg):
     if debug_level >= level:
         dfile = os.environ.get("GRASS_DEBUG_FILE")
         fd = sys.stderr if dfile is None else open(dfile, mode="a")
-        print("D%d/%d: " % (level, debug_level), *msg, end="\n", file=fd)
+        try:
+            print("D%d/%d: " % (level, debug_level), *msg, end="\n", file=fd)
+        finally:
+            if dfile is not None:
+                fd.close()

raster/r.in.gdal/testsuite/test_r_in_gdal.py

@@ -307,7 +307,8 @@ def test_netCDF_3d_5(self):
 test_gdal_import_map.0000000105
 """
 
-        text_from_file = open("map_names_file.txt").read()
+        with open("map_names_file.txt") as file:
+            text_from_file = file.read()
 
         self.assertLooksLike(map_list, text_from_file)
 

scripts/g.extension/g.extension.py

@@ -1665,6 +1665,8 @@ def install_extension_win(name):
             if fullname not in old_file_list:
                 file_list.append(fullname)
 
+    if outdev is not sys.stdout:
+        outdev.close()
     return 0, module_list, file_list
 
 
@@ -1780,8 +1782,9 @@ def is_binary_string(bytes):
     for root, unused, files in os.walk(directory):
         for name in files:
             filename = os.path.join(root, name)
-            if is_binary_string(open(filename, "rb").read(1024)):
-                continue  # ignore binary files
+            with open(filename, "rb") as file:
+                if is_binary_string(file.read(1024)):
+                    continue  # ignore binary files
 
             # read content of text file
             data = Path(filename).read_bytes()
@@ -2183,6 +2186,8 @@ def install_extension_std_platforms(name, source, url, branch):
             if fullname not in old_file_list:
                 file_list.append(fullname)
 
+    if outdev is not sys.stdout:
+        outdev.close()
     return ret, module_list, file_list, os.path.join(srcdir)
 
 

scripts/m.proj/m.proj.py

@@ -282,8 +282,14 @@ def main():
                 )
             )
 
+    p.stdout.close()
     p.wait()
 
+    if inf is not sys.stdin:
+        inf.close()
+    if outf is not sys.stdout:
+        outf.close()
+
     if p.returncode != 0:
         gcore.warning(_("Projection transform probably failed, please investigate"))