-
Notifications
You must be signed in to change notification settings - Fork 7
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
Fix mounting special files #56
Conversation
This fixes sandboxing exceptions for special files on Linux. Previously birdcage would explicitly panic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs a CHANGELOG entry, I think.
This bug technically doesn't exist in a released version since 0.4 was yanked. But the tag and CHANGELOG entry still exist for 0.4, so it's still a good idea to explain the changes between 0.4 and 0.4+next
let tempdir = TempDir::new().unwrap(); | ||
let symlink_str = tempdir.path().to_string_lossy() + "_tmpfile"; | ||
let symlink_path = PathBuf::from(symlink_str.as_ref()); | ||
unixfs::symlink(&tempdir, &symlink_path).unwrap(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There might be some value in adding a broken symlink to one of these tests just to make sure that birdcage.lock()
still succeeds. Although the symlink should remain broken inside the sandbox, so maybe that is pointless.
No description provided.