Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot parse Rstat with a name of / #37

Open
kyrias opened this issue Jun 16, 2024 · 0 comments
Open

Cannot parse Rstat with a name of / #37

kyrias opened this issue Jun 16, 2024 · 0 comments

Comments

@kyrias
Copy link

kyrias commented Jun 16, 2024

As per the stat(5) Plan 9 man page stat.name must be / if the file Tstat is invoked with is the root of the file tree, but trying to parse such an Rstat message with styxproto leads to the following panic:

    styxproto_test.go:92: bad message: slash in path element

I discovered this because I've been using styxproto to validate my own 9P implementation. I haven't investigated the actual implementation of styx or jsonfs so I'm not sure what exactly causes it, but dumping the traffic when mounting a jsonfs exported file tree I see that the root directory has a name of . which is invalid as per stat(5).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant