Bug 338843 - Dolphin should not display mounts that the user cannot access
Summary: Dolphin should not display mounts that the user cannot access
Status: RESOLVED UNMAINTAINED
Alias: None
Product: solid
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 4.13.2
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Dolphin Bug Assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-09-06 12:35 UTC by Mark Harmstone
Modified: 2018-09-04 15:45 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Patch (1.56 KB, patch)
2014-09-06 12:40 UTC, Mark Harmstone 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Harmstone 2014-09-06 12:35:53 UTC 
On a multi-user system, it's conceivable that that other users might have CIFS shares etc. mounted in their home directories. Dolphin shouldn't display these if the directories cannot be accessed by the current user.

Reproducible: Always
Comment 1 Mark Harmstone 2014-09-06 12:40:09 UTC 
Created attachment 88588 [details]
Patch

Patch to achieve this. I don't think it's possible just by modifying the Solid predicate, so it instead checks using QDir.
Comment 2 Christoph Feck 2014-09-06 20:34:09 UTC 
Mark, can you test what happens if these shares are on a remote host that is not online? Checking every mount might block the user interface for a long time.
Comment 3 Emmanuel Pescosta 2014-09-08 13:50:51 UTC 
Thanks for the bug report!

This bug can also be reproduced with USB storage devices. Simply mount it with your current user, then start a new session with another user and try to access this mount there.

Remember that there are other places, where the same bug exists (like the file dialog or the device notifier). So we should better fix it in Solid.
Comment 4 Emmanuel Pescosta 2014-09-09 22:55:14 UTC 
/proc/mounts provides some useful information related to storage permissions, maybe we can used somehow?
Comment 5 Emmanuel Pescosta 2014-09-09 22:55:53 UTC 
(In reply to Emmanuel Pescosta from comment #4)

permissions, maybe we *can used = can use it
Comment 6 Andrew Crouthamel 2018-09-04 15:45:43 UTC 
Hello! Sorry to be the bearer of bad news, but this project has been unmaintained for many years so I am closing this bug. Please try again with the latest version and submit a new bug to frameworks-solid if your issue persists. Thank you!