3 files changed, 65 insertions, 9 deletions

diff --git a/pkg/p9/p9test/BUILD b/pkg/p9/p9test/BUILD
index cf22edde8..6e939a49a 100644
--- a/pkg/p9/p9test/BUILD
+++ b/pkg/p9/p9test/BUILD
@@ -8,7 +8,7 @@ alias(
     actual = "@com_github_golang_mock//mockgen:mockgen",
 )
 
-MOCK_SRC_PACKAGE = "gvisor.googlesource.com/gvisor/pkg/p9"
+MOCK_SRC_PACKAGE = "gvisor.dev/gvisor/pkg/p9"
 
 # mockgen_reflect is a source file that contains mock generation code that
 # imports the p9 package and generates a specification via reflection. The
@@ -64,7 +64,7 @@ go_library(
         "mocks.go",
         "p9test.go",
     ],
-    importpath = "gvisor.googlesource.com/gvisor/pkg/p9/p9test",
+    importpath = "gvisor.dev/gvisor/pkg/p9/p9test",
     visibility = ["//:sandbox"],
     deps = [
         "//pkg/fd",
diff --git a/pkg/p9/p9test/client_test.go b/pkg/p9/p9test/client_test.go
index e00dd03ab..fe649c2e8 100644
--- a/pkg/p9/p9test/client_test.go
+++ b/pkg/p9/p9test/client_test.go
@@ -28,8 +28,8 @@ import (
 	"time"
 
 	"github.com/golang/mock/gomock"
-	"gvisor.googlesource.com/gvisor/pkg/fd"
-	"gvisor.googlesource.com/gvisor/pkg/p9"
+	"gvisor.dev/gvisor/pkg/fd"
+	"gvisor.dev/gvisor/pkg/p9"
 )
 
 func TestPanic(t *testing.T) {
@@ -269,14 +269,14 @@ type fileGenerator func(*Harness, string, p9.File) (*Mock, *Mock, p9.File)
 func walkHelper(h *Harness, name string, dir p9.File) (parentBackend *Mock, walkedBackend *Mock, walked p9.File) {
 	_, parent, err := dir.Walk(nil)
 	if err != nil {
-		h.t.Fatalf("got walk err %v, want nil", err)
+		h.t.Fatalf("Walk(nil) got err %v, want nil", err)
 	}
 	defer parent.Close()
 	parentBackend = h.Pop(parent)
 
 	_, walked, err = parent.Walk([]string{name})
 	if err != nil {
-		h.t.Fatalf("got walk err %v, want nil", err)
+		h.t.Fatalf("Walk(%s) got err %v, want nil", name, err)
 	}
 	walkedBackend = h.Pop(walked)
 
@@ -389,7 +389,7 @@ func checkDeleted(h *Harness, file p9.File) {
 	_, newFile, err := file.Walk(nil)
 	if err == syscall.EBUSY {
 		// We can't walk from here because this reference is open
-		// aleady. Okay, we will also have unopened cases through
+		// already. Okay, we will also have unopened cases through
 		// TestUnlink, just skip the remove operation for now.
 		return
 	} else if err != nil {
@@ -854,6 +854,62 @@ func TestRenameAtInvalid(t *testing.T) {
 	}
 }
 
+// TestRenameSecondOrder tests that indirect rename targets continue to receive
+// Renamed calls after a rename of its renamed parent. i.e.,
+//
+// 1. Create /one/file
+// 2. Create /directory
+// 3. Rename /one -> /directory/one
+// 4. Rename /directory -> /three/foo
+// 5. file from (1) should still receive Renamed.
+//
+// This is a regression test for b/135219260.
+func TestRenameSecondOrder(t *testing.T) {
+	h, c := NewHarness(t)
+	defer h.Finish()
+
+	rootBackend, root := newRoot(h, c)
+	defer root.Close()
+
+	// Walk to /one.
+	_, oneBackend, oneFile := walkHelper(h, "one", root)
+	defer oneFile.Close()
+
+	// Walk to and generate /one/file.
+	//
+	// walkHelper re-walks to oneFile, so we need the second backend,
+	// which will also receive Renamed calls.
+	oneSecondBackend, fileBackend, fileFile := walkHelper(h, "file", oneFile)
+	defer fileFile.Close()
+
+	// Walk to and generate /directory.
+	_, directoryBackend, directoryFile := walkHelper(h, "directory", root)
+	defer directoryFile.Close()
+
+	// Rename /one to /directory/one.
+	rootBackend.EXPECT().RenameAt("one", directoryBackend, "one").Return(nil)
+	expectRenamed(oneBackend, []string{}, directoryBackend, "one")
+	expectRenamed(oneSecondBackend, []string{}, directoryBackend, "one")
+	expectRenamed(fileBackend, []string{}, oneBackend, "file")
+	if err := renameAt(h, root, directoryFile, "one", "one", false); err != nil {
+		h.t.Fatalf("got rename err %v, want nil", err)
+	}
+
+	// Walk to /three.
+	_, threeBackend, threeFile := walkHelper(h, "three", root)
+	defer threeFile.Close()
+
+	// Rename /directory to /three/foo.
+	rootBackend.EXPECT().RenameAt("directory", threeBackend, "foo").Return(nil)
+	expectRenamed(directoryBackend, []string{}, threeBackend, "foo")
+	expectRenamed(oneBackend, []string{}, directoryBackend, "one")
+	expectRenamed(oneSecondBackend, []string{}, directoryBackend, "one")
+	expectRenamed(fileBackend, []string{}, oneBackend, "file")
+	if err := renameAt(h, root, threeFile, "directory", "foo", false); err != nil {
+		h.t.Fatalf("got rename err %v, want nil", err)
+	}
+}
+
 func TestReadlink(t *testing.T) {
 	for name := range newTypeMap(nil) {
 		t.Run(name, func(t *testing.T) {
diff --git a/pkg/p9/p9test/p9test.go b/pkg/p9/p9test/p9test.go
index 1c8eff200..95846e5f7 100644
--- a/pkg/p9/p9test/p9test.go
+++ b/pkg/p9/p9test/p9test.go
@@ -23,8 +23,8 @@ import (
 	"testing"
 
 	"github.com/golang/mock/gomock"
-	"gvisor.googlesource.com/gvisor/pkg/p9"
-	"gvisor.googlesource.com/gvisor/pkg/unet"
+	"gvisor.dev/gvisor/pkg/p9"
+	"gvisor.dev/gvisor/pkg/unet"
 )
 
 // Harness is an attacher mock.