After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 163165 - Bad parsing for "user" in fstab
Bad parsing for "user" in fstab
Status: RESOLVED WONTFIX
Product: gnome-vfs
Classification: Deprecated
Component: Other
cvs (head)
Other All
: High normal
: ---
Assigned To: Christian Neumair
gnome-vfs maintainers
gnome[unmaintained]
Depends on:
Blocks:
 
 
Reported: 2005-01-06 20:24 UTC by Sebastien Bacher
Modified: 2018-08-17 13:51 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
Proposed Patch (HEAD) (907 bytes, patch)
2005-04-30 14:39 UTC, Christian Neumair
needs-work Details | Review
Proposed (not really tested) patch (4.95 KB, patch)
2006-04-03 16:35 UTC, Christian Neumair
none Details | Review

Description Sebastien Bacher 2005-01-06 20:24:21 UTC
This bug has been opened here: https://bugzilla.ubuntu.com/4023

"Nautilus will show any mounts with the word "user" in their fstab line as drives
on the desktop.  Unfortunately, one of the samba options is "username", so any
samba exports mounted as a particular user will be shown on the desktop rather
than just slotted into the directory tree as normal mounts."
Comment 1 Martin Wehner 2005-03-03 22:43:31 UTC
gnome-vfs issue, reassigning.
Comment 2 Christian Neumair 2005-04-30 14:39:53 UTC
Created attachment 45862 [details] [review]
Proposed Patch (HEAD)

mntent indeed matches all strings beginning with "name", so we have to filter
this one out.
Comment 3 Christian Neumair 2005-04-30 14:40:57 UTC
...beginning with "user", I mean.
Comment 4 Lionel Dricot 2006-04-03 13:46:32 UTC
Up as there is a patch a no comments for a year.

Is this bug still valid ? Has the bug been commited ?
Comment 5 Christian Neumair 2006-04-03 13:59:25 UTC
Thanks for pinging us. The current semantics are actually a bit odd, because hasmntopt() does substring matching, and the patch just special-cases particular cases. I'll look into it. Reopening, assigning to myself.
Comment 6 Christian Neumair 2006-04-03 16:35:26 UTC
Created attachment 62678 [details] [review]
Proposed (not really tested) patch

The attached patch adds a getmntopt_strcmp helper which doesn't do substring matching but matches keys where each mntopt is either "key=value" or "key".

Requires testing, feedback appreciated.
Comment 7 Daniel Holbach 2006-12-08 16:36:24 UTC
Did the patch get some attention?
Comment 8 Baptiste Mille-Mathias 2007-09-03 19:35:52 UTC
I know gvfs is on the rails, but would be nice to have some feedback on this patch if possible.

Thanks for your attention

cheers
Comment 9 André Klapper 2018-08-17 13:51:49 UTC
gnome-vfs got deprecated in 2008.

gnome-vfs is not under active development anymore and had its last code changes
in 2011. Its codebase has been archived:
https://gitlab.gnome.org/Archive/gnome-vfs/commits/master

gio (in glib) and gvfs are its successors. See https://developer.gnome.org/gio/stable/ch33.html and https://people.gnome.org/~gicmo/gio-migration-guide/ for porting info.

Closing this report as WONTFIX as part of Bugzilla Housekeeping to reflect
reality. Feel free to open a task in GNOME Gitlab if the issue described in this task still applies to a recent + supported version of glib/gio/gvfs. Thanks!