reverse: use incrementing inode number for gocryptfs.longname.*.name files

ed0a12b7337c2d88c027329f64e73070da17d5b3 already fixed the kernel side,
now we also want the .name files to NOT appear hardlinked when just
looking at the inode number.

Relates-to: https://github.com/rfjakob/gocryptfs/issues/802

2 files changed, 18 insertions, 2 deletions

diff --git a/tests/reverse/correctness_test.go b/tests/reverse/correctness_test.go
index 930048f..b335456 100644
--- a/tests/reverse/correctness_test.go
+++ b/tests/reverse/correctness_test.go
@@ -298,6 +298,10 @@ func TestSeekData(t *testing.T) {
 // gocryptfs.longname.*.name of hardlinked files should not appear hardlinked (as the
 // contents are different).
 //
+// This means that
+// 1) They have a different NodeID, hence the kernel knows it's different files
+// 2) They have a different inode number, hence userspace knows they are not hard-linked.
+//
 // https://github.com/rfjakob/gocryptfs/issues/802
 func TestHardlinkedLongname(t *testing.T) {
 	if plaintextnames {
@@ -335,4 +339,16 @@ func TestHardlinkedLongname(t *testing.T) {
 	if test_helpers.Md5fn(matches[0]) == test_helpers.Md5fn(matches[1]) {
 		t.Errorf("Files %q are identical - that's wrong!", matches)
 	}
+
+	var st0 syscall.Stat_t
+	if err := syscall.Stat(matches[0], &st0); err != nil {
+		t.Fatal(err)
+	}
+	var st1 syscall.Stat_t
+	if err := syscall.Stat(matches[1], &st1); err != nil {
+		t.Fatal(err)
+	}
+	if st0.Ino == st1.Ino {
+		t.Errorf("Files %q have the same inode number - that's wrong!", matches)
+	}
 }
diff --git a/tests/reverse/inomap_test.go b/tests/reverse/inomap_test.go
index aadb782..ff78f4c 100644
--- a/tests/reverse/inomap_test.go
+++ b/tests/reverse/inomap_test.go
@@ -143,7 +143,7 @@ func TestVirtualFileIno(t *testing.T) {
 	if origInos.child == cipherInos.name {
 		t.Errorf("name ino collision: %d == %d", origInos.child, cipherInos.name)
 	}
-	if origInos.child&mask != cipherInos.name&mask {
-		t.Errorf("name ino mismatch: %#x vs %#x", origInos.child, cipherInos.name)
+	if cipherInos.name < 1<<63 {
+		t.Errorf("name ino should be in spill space, but is actually %#x", cipherInos.name)
 	}
 }