/index and the log escape a control byte, DEL or C1 in a name as \xNN, as they do a byte not UTF-8, so the name decodes to what it is

A control byte in a name read as a space, so /index could not be
decoded back to the name the name file now accepts (\x07 and a space
read the same). Every byte a name holds that is not printable UTF-8 is
now \xNN there; a message's text keeps spaces.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

1 files changed, 4 insertions, 3 deletions

diff --git a/src/ninep/ctl.zig b/src/ninep/ctl.zig
index aa309ad1..9a1a8da1 100644
--- a/src/ninep/ctl.zig
+++ b/src/ninep/ctl.zig
@@ -2964,13 +2964,14 @@ test "every EINVAL a write gets says why, in its err record too; DEL is a contro
     try testing.expect(th.logHas(p, "an empty name"));
 }
 
-test "/index shows a name as the log does: a newline in it is \\n, controls spaces, bytes not UTF-8 \\xNN" {
+test "/index shows a name as the log does: a newline in it is \\n, a control byte, C1 or byte not UTF-8 \\xNN" {
     const p = try th.withTerm(testing.allocator);
     defer p.deinit();
     const term = p.paneBySerial(serialOf(p)).?;
-    p.setCwd(term, "/tmp/two\nlines\x7f\xff");
+    p.setCwd(term, "/tmp/two\nlines\x07\x7f\xc2\x85\xff");
     const index = rd(p, @intFromEnum(tree.TopFile.index), 0, 4096).bytes;
-    try testing.expect(std.mem.indexOf(u8, index, "/tmp/two\\nlines \\xff ") != null);
+    // Each byte as itself, so the name decodes to what it is.
+    try testing.expect(std.mem.indexOf(u8, index, "/tmp/two\\nlines\\x07\\x7f\\xc2\\x85\\xff ") != null);
     try testing.expectEqual(@as(usize, 1), std.mem.count(u8, index, "\n"));
     try testing.expect(index.len > 0);
     try testing.expectEqual(@as(u64, 0), call(p, .{ .tag = 1, .op = .getattr, .node = @intFromEnum(tree.TopFile.index) }).reply.attr.size);