Remove unnecessary local ID in authfs
AuthFS had the unnecessary flexibility to associate a local file ID /
filesystem entry name with the remote FD. Remove this flexibility since
we can always use the remote FD to name the file entry.
Also, rename some of the "id" with "fd". Although id/identifiler may
make better sense in the API, calling it fd/file descriptor could reduce
mental overhead.
Bug: 203251769
Test: atest AuthFsHostTest
Test: atest ComposHostTestCases
Change-Id: I7a75ef4ed21fb00f5c7f80c560d1f654d21268cd
diff --git a/authfs/service/src/authfs.rs b/authfs/service/src/authfs.rs
index 6d87243..1b05749 100644
--- a/authfs/service/src/authfs.rs
+++ b/authfs/service/src/authfs.rs
@@ -132,11 +132,11 @@
// TODO(b/185178698): Many input files need to be signed and verified.
// or can we use debug cert for now, which is better than nothing?
args.push(OsString::from("--remote-ro-file-unverified"));
- args.push(OsString::from(format!("{}:{}", conf.fd, conf.fd)));
+ args.push(OsString::from(conf.fd.to_string()));
}
for conf in out_fds {
args.push(OsString::from("--remote-new-rw-file"));
- args.push(OsString::from(format!("{}:{}", conf.fd, conf.fd)));
+ args.push(OsString::from(conf.fd.to_string()));
}
if debuggable {
args.push(OsString::from("--debug"));