try not to leak

plugin/plugins/filesystem/nodes/base.go

@@ -144,15 +144,7 @@ func (ib *IPFSBase) close() error {
 	}
 
 	if ib.filesystemCancel != nil {
-		/* TODO: only do this on the last close to the root
-		if ib.proxy != nil {
-		    if err := ib.proxy.Close(); err != nil {
-				ib.Logger.Error(err)
-			}
-			lastErr = err
-		}
 		ib.filesystemCancel()
-		*/
 	}
 
 	if ib.operationsCancel != nil {

plugin/plugins/filesystem/nodes/keyfs.go

@@ -2,6 +2,7 @@ package fsnodes
 
 import (
 	"context"
+	"runtime"
 
 	"github.com/hugelgupf/p9/p9"
 	"github.com/hugelgupf/p9/unimplfs"
@@ -39,6 +40,11 @@ func KeyFSAttacher(ctx context.Context, core coreiface.CoreAPI, ops ...nodeopts.
 
 	kd.proxy = subsystem.(fsutils.WalkRef)
 
+	// detach from our proxied system when we fall out of memory
+	runtime.SetFinalizer(kd, func(keyRoot *KeyFS) {
+		keyRoot.proxy.Close()
+	})
+
 	return kd
 }
 

plugin/plugins/filesystem/nodes/pinfs.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"fmt"
 	gopath "path"
+	"runtime"
 
 	"github.com/hugelgupf/p9/p9"
 	"github.com/hugelgupf/p9/unimplfs"
@@ -43,6 +44,11 @@ func PinFSAttacher(ctx context.Context, core coreiface.CoreAPI, ops ...nodeopts.
 
 	pd.proxy = subsystem.(fsutils.WalkRef)
 
+	// detach from our proxied system when we fall out of memory
+	runtime.SetFinalizer(pd, func(pinRoot *PinFS) {
+		pinRoot.proxy.Close()
+	})
+
 	return pd
 }
 

plugin/plugins/filesystem/nodes/root.go

@@ -2,6 +2,7 @@ package fsnodes
 
 import (
 	"context"
+	"runtime"
 
 	"github.com/hugelgupf/p9/p9"
 	"github.com/hugelgupf/p9/unimplfs"
@@ -116,6 +117,13 @@ func RootAttacher(ctx context.Context, core coreiface.CoreAPI, ops ...nodeopts.A
 		}
 	}
 
+	// detach from our proxied systems when we fall out of memory
+	runtime.SetFinalizer(ri, func(root *RootIndex) {
+		for _, ss := range ri.subsystems {
+			ss.file.Close()
+		}
+	})
+
 	return ri
 }