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.

Sign up for GitHub

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

Jump to bottom

memfs symlink that point outside memfs would cause unpredicatble behavior #64

Closed
maxirmx opened this issue Dec 21, 2021 · 0 comments · Fixed by #81
Closed

memfs symlink that point outside memfs would cause unpredicatble behavior #64

maxirmx opened this issue Dec 21, 2021 · 0 comments · Fixed by #81
Assignees
@maxirmx
Labels
bug Something isn't working
Milestone
v0.1.11

Comments

@maxirmx
Copy link
Member

maxirmx commented Dec 21, 2021 •
edited
Loading

memfs symlink that point outside memfs would cause unpredicatble behavior

Tha issue affects all functions except stat.
Stat works correctly but st.ino returned can not be distinguished between memfs and real filesystem.

Not sure what is correct behavior is such case, probably we should not allow such symlinks in memfs
@maxirmx maxirmx added the bug Something isn't working label Dec 21, 2021
@maxirmx maxirmx mentioned this issue Dec 21, 2021
Closed
@maxirmx maxirmx added this to the v0.1.10 milestone Feb 8, 2022
maxirmx added a commit that referenced this issue Mar 16, 2022
@maxirmx
Correct processin and tests for symlinks pointing outside of memfs #64
512ce35
maxirmx added a commit that referenced this issue Mar 16, 2022
@maxirmx
Correct processing and tests for symlinks pointing outside of memfs #64
f2eb7cf
@maxirmx maxirmx self-assigned this Mar 16, 2022
@maxirmx maxirmx linked a pull request Apr 2, 2022 that will close this issue
Add Alpine Linux build #81
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maxirmx maxirmx

Labels
bug Something isn't working
Projects
None yet
Milestone
v0.1.11
Development

Successfully merging a pull request may close this issue.

Add Alpine Linux build tamatebako/libdwarfs
1 participant
@maxirmx