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
lib/vfscore: Prevent superfluous slashes in working directory #1302
lib/vfscore: Prevent superfluous slashes in working directory #1302
Conversation
The mount point can contain a trailing slash (for example the root mount point has a path of "/"). The path that is concatenated to the mount point also contains a slash at the beginning. Therefore, extra care must be taken to prevent two slashes from appearing besides each other. Signed-off-by: Marco Schlumpp <marco@unikraft.io>
838fe88
to
a2c01e2
Compare
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.
Reviewed-by: Stefan Jumarea stefanjumarea02@gmail.com
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.
Looks good.
Reviewed-by: Radu Nichita radunichita99@gmail.com
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.
Approved-by: Razvan Deaconescu razvand@unikraft.io
The mount point can contain a trailing slash (for example the root mount point has a path of "/"). The path that is concatenated to the mount point also contains a slash at the beginning. Therefore, extra care must be taken to prevent two slashes from appearing besides each other. Signed-off-by: Marco Schlumpp <marco@unikraft.io> Reviewed-by: Stefan Jumarea <stefanjumarea02@gmail.com> Reviewed-by: Radu Nichita <radunichita99@gmail.com> Approved-by: Razvan Deaconescu <razvand@unikraft.io> GitHub-Closes: #1302
The mount point can contain a trailing slash (for example the root mount point has a path of "/"). The path that is concatenated to the mount point also contains a slash at the beginning. Therefore, extra care must be taken to prevent two slashes from appearing besides each other.
Prerequisite checklist
checkpatch.uk
on your commit series before opening this PR;Base target
Additional configuration
Description of changes