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

Test "mntdir with space" fails on Alpine Linux #106

Open
dgerber opened this issue Dec 13, 2021 · 4 comments
Open

Test "mntdir with space" fails on Alpine Linux #106

dgerber opened this issue Dec 13, 2021 · 4 comments
Labels
fuse-bug Most likely a bug in a FUSE implementation

Comments

@dgerber
Copy link

dgerber commented Dec 13, 2021 •
edited
Loading

This tests fails when running as uid not 0 and with --no-allow-other.

--- mntdir with space ---
[    ]
fusermount3: entry for .../lib/alpine/aports/testing/bindfs/src/bindfs-1.15.1/tests/tmp_test_bindfs/a b not found in /etc/mtab
ERROR: failed to umount

Not sure if the issue is in libfuse.
@dgerber dgerber changed the title Test "mntdir with space" fails Test "mntdir with space" fails on Alpine Linux Dec 13, 2021
@mpartel
Copy link
Owner

mpartel commented Dec 13, 2021

I added --no-allow-other to the test case on Debian 10 (FUSE 3.4.1) and Debian 11 (FUSE 3.10.3) and it passed.

I agree that the problem is likely in libfuse or, looking at may_unmount there, possibly in musl's implementation of getmntent.

@mpartel mpartel added the fuse-bug Most likely a bug in a FUSE implementation label Dec 13, 2021
@mpartel
Copy link
Owner

mpartel commented Dec 13, 2021

Vagrant setup where it repros: 6d762ab
Can add it to test suite when fixed.

@matthiasgoergens
Copy link

Attempt at a fix is at https://www.openwall.com/lists/musl/2023/03/29/2

matthiasgoergens added a commit to matthiasgoergens/libfuse that referenced this issue Apr 2, 2023
@matthiasgoergens
Workaround musl bug when mountdir has whitespace
7f3b328 
Fixes mpartel/bindfs#106
@matthiasgoergens matthiasgoergens mentioned this issue Apr 2, 2023
Merged
matthiasgoergens added a commit to matthiasgoergens/libfuse that referenced this issue Apr 2, 2023
@matthiasgoergens
Workaround musl bug when mountdir has whitespace
e6d4ae3 
Fixes mpartel/bindfs#106
matthiasgoergens added a commit to matthiasgoergens/libfuse that referenced this issue Apr 2, 2023
@matthiasgoergens
Workaround musl bug when mountdir has whitespace
a13138d 
Fixes mpartel/bindfs#106
matthiasgoergens added a commit to matthiasgoergens/libfuse that referenced this issue Apr 2, 2023
@matthiasgoergens
Workaround musl bug when mountdir has whitespace
657cd5f 
Fixes mpartel/bindfs#106
@matthiasgoergens
Copy link

matthiasgoergens commented Apr 2, 2023 •
edited
Loading

@mpartel I have a work-around while we wait for musl: libfuse/libfuse#761

EDIT: It's merged into libfuse master, but not released.

matthiasgoergens added a commit to matthiasgoergens/libfuse that referenced this issue Apr 2, 2023
@matthiasgoergens
Workaround musl bug when mountdir has whitespace
ab638c6 
Fixes libfuse#634 and mpartel/bindfs#106
Nikratio pushed a commit to libfuse/libfuse that referenced this issue Apr 2, 2023
@matthiasgoergens
Workaround musl bug when mountdir has whitespace (#761)
7be56c5 
Fixes #634 and mpartel/bindfs#106
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
fuse-bug Most likely a bug in a FUSE implementation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@matthiasgoergens @mpartel @dgerber