[fix] Avoid resource leaks in case a MountSource constructor throws

core/ratarmountcore/SQLiteIndex.py

@@ -306,6 +306,11 @@ def openWritable(self):
     def __enter__(self):
         return self
 
+    def __del__(self):
+        # This is important in case SQLiteIndex is not used with a context manager
+        # and the constructor raises an exception. There is no clean way to close it in that case!
+        self.close()
+
     def __exit__(self, exception_type, exception_value, exception_traceback):
         self.close()
 

ratarmount.py

@@ -676,13 +676,15 @@ def __del__(self) -> None:
         try:
             if self.mountPointFd is not None:
                 os.close(self.mountPointFd)
-        except Exception:
-            pass
+        except Exception as exception:
+            if self.printDebug >= 1:
+                print("[Warning] Failed to close mount point folder descriptor because of:", exception)
 
         try:
             self.mountSource.__exit__(None, None, None)
-        except Exception:
-            pass
+        except Exception as exception:
+            if self.printDebug >= 1:
+                print("[Warning] Failed to tear down root mount source because of:", exception)
 
     def _addNewHandle(self, handle, flags):
         # Note that fh in fuse_common.h is 64-bit and Python also supports 64-bit (long integers) out of the box.