Fix to make comments, types and initialization easier to understand

Signed-off-by: Kohei Tokunaga <[email protected]>

Author: ktock

crfs.go

@@ -474,7 +474,7 @@ func (n *layerDebugRoot) Lookup(ctx context.Context, name string) (fspkg.Node, e
 		te: root,
 		sr: r,
 		f:  f,
-		children: make(map[string]fspkg.Node),
+		child: make(map[string]*node),
 	}, nil
 }
 
@@ -776,7 +776,7 @@ func (n *layerHostOwnerImageReference) Lookup(ctx context.Context, name string)
 		fs: n.fs,
 		te: root,
 		sr: r,
-		children: make(map[string]fspkg.Node),
+		child: make(map[string]*node),
 	}, nil
 }
 
@@ -903,7 +903,11 @@ type node struct {
 	te *stargz.TOCEntry
 	sr *stargz.Reader
 	f  *os.File // non-nil if root & in debug mode
-	children map[string]fspkg.Node // Remenber child nodes once looked up.
+
+	// children maps from previously-looked up base names (like "foo.txt") to
+	// the *node that was previously returned. This prevents FUSE inode numbers
+	// from getting out of sync
+	child map[string]*node
 }
 
 var (
@@ -969,7 +973,7 @@ func (h *nodeHandle) ReadDirAll(ctx context.Context) (ents []fuse.Dirent, err er
 //
 // See https://godoc.org/bazil.org/fuse/fs#NodeStringLookuper
 func (n *node) Lookup(ctx context.Context, name string) (fspkg.Node, error) {
-    	if c, ok := n.children[name] ; ok {
+    	if c, ok := n.child[name] ; ok {
     		return c, nil
     	}
 
@@ -978,8 +982,13 @@ func (n *node) Lookup(ctx context.Context, name string) (fspkg.Node, error) {
 		return nil, fuse.ENOENT
 	}
 
-    	c := &node{n.fs, e, n.sr, nil, make(map[string]fspkg.Node)}
-    	n.children[name] = c
+    	c := &node{
+		fs: n.fs,
+		te: e,
+		sr: n.sr,
+		child: make(map[string]*node),
+	}
+    	n.child[name] = c
 
     	return c, nil
 }