Bug 49842 – Unhandled rootfs on bind mount

Don't ignore a mount just because if a subtree of the filesystem is
mounted. Because if we do this, we may wrongly conclude the device is
not mounted at all, which then means that it's visible in the desktop
UI and subject to automounting.

In the future, we may want a "a(ss) MountPointsFull" property that in
addition to the mount point also contains the subtree that is mounted
at said mountpoint.

https://bugs.freedesktop.org/show_bug.cgi?id=49842

Signed-off-by: David Zeuthen <davidz@redhat.com>

Call it "Disk Manager" and reword some sections of udisks(8) man page

Signed-off-by: David Zeuthen <davidz@redhat.com>

udisksctl: describe 'loop-setup' and 'loop-delete' verbs

For some reason the man page didn't describe these verbs. Also remove
some references to udisksd(8) for the unmount and lock verbs.

Signed-off-by: David Zeuthen <davidz@redhat.com>

udisksctl: add 'smart-simulate' verb

This makes it a lot easier to simulate failing disks. Also add proper
polkit support so the user don't have to run it as root.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Post-release version bump to 1.98.0

Signed-off-by: David Zeuthen <davidz@redhat.com>

Update NEWS for release

Signed-off-by: David Zeuthen <davidz@redhat.com>

Loop: try writing to loop/autoclear sysfs file before LO_{GET,SET}_STATUS

This will only work if the following kernel patch is applied

 http://people.freedesktop.org/~david/linux-3.4.0-rc5+-loop-make-it-possible-to-set-autoclear-through-sysfs.patch

however with this patch it's now possible to set autoclear for loop
devices backed on files on FUSE filesystems - without this patch we'll
fail when doing LO_GET_STATUS64.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Simplify loop device checking

... and never automatically tear down loop devices.

Signed-off-by: David Zeuthen <davidz@redhat.com>

UDisksCleanup: Remove stale entries when adding new ones

Signed-off-by: David Zeuthen <davidz@redhat.com>

Update path to mounted-fs file in documentation

It's been /run/udisks2, not /var/lib/udisks2, for a while now.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Revert "Add loop.autoclear option to Filesystem.Mount() method"

Now that we have a Loop.SetAutoclear() method, this is no longer
needed.

This reverts commit 9a3939800e280efbac0efe22dca26799da894f4d.

UDisksDaemon: rework wait_for_object_sync() method

Simpler and more intuitive this way. Also update all current users to
the new API and semantics.

Signed-off-by: David Zeuthen <davidz@redhat.com>

UDisksDaemon: add get_objects() method

Mostly convenience to avoid casting and getting the object-manager.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Manager: wait for partitions to appear before returning in LoopSetup()

This is needed for the Disk Image Mounter when mounting all
filesystems in a partitioned disk image.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Daemon: Add new find_block_*() convenience methods

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add work-around to show FS on CDs/USB sticks created using isohybrid

This happens at least in two cases - when using loop-devices with
LO_PARTSCAN and when dd(1)'ing the image to a USB stick.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Loop: add SetupByUID property

This is useful in GVfs where we don't want to show loop devices set up
by other users.

Signed-off-by: David Zeuthen <davidz@redhat.com>

UDisksClient: add get_loop_for_block() convenience method

Signed-off-by: David Zeuthen <davidz@redhat.com>

Warn if /run/udisks2/loop has an existing entry when setting up a new loop device

Signed-off-by: David Zeuthen <davidz@redhat.com>

Manager: Add locking around LoopSetup()

Surprisingly, contention did happen when trying to attach 20-some disk
images in a row.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Manager: Add new 'no-part-scan' option in LoopSetup() method

Signed-off-by: David Zeuthen <davidz@redhat.com>

Fix up how we check if use is authorized for Loop.SetAutoclear()

Signed-off-by: David Zeuthen <davidz@redhat.com>

Don't assign the Loop D-Bus interface to partitions of a loop device

Signed-off-by: David Zeuthen <davidz@redhat.com>

Loop: add :Autoclear property and SetAutoclear() method

Also add some UI in Disks for this

 http://people.freedesktop.org/~david/disks-loop-autoclear-switch.png

Signed-off-by: David Zeuthen <davidz@redhat.com>

Also check for "target is busy" when checking umount(8) output

Apparently this changed recently... ideally umount(8) would simply
return a well-defined exit code but that's not the way things work
right now.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add loop.autoclear option to Filesystem.Mount() method

The main user of this could be the desktop shell / file manager
offering a "Attach Disk Image" option for e.g. ISO files - it would
first use Manager.LoopSetup() and then, on the returned block device,
invoke Filesystem.Mount() with this new option (assuming the returned
loop device has any interesting content). With this, the file manager
need not worry about the loop device - it goes away as soon as the
filesystem is unmounted.

Why is this an option to Filesystem.Mount() and not Manager.LoopSetup()?
Because right after setting up the loop device, the OS opens the
device so it can populate the udev database with the detected contents
of the newly created loop device and also to maintain symlinks in the
/dev/disk tree. If we had created the loop device with LO_AUTOCLEAR
the kernel would clear the loop device right after blkid(8) closes the
device. However, if we set the flag after successfully mounting the
file system on the loop device, there is already a single reference to
the block device.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Post-release version bump to 1.97.0

Signed-off-by: David Zeuthen <davidz@redhat.com>

Update NEWS for release

Signed-off-by: David Zeuthen <davidz@redhat.com>

Actually link with libsystemd-login

Signed-off-by: David Zeuthen <davidz@redhat.com>

Post-release version bump to 1.96.0

Signed-off-by: David Zeuthen <davidz@redhat.com>

Update NEWS for release

Signed-off-by: David Zeuthen <davidz@redhat.com>

Minor doc fixes

Signed-off-by: David Zeuthen <davidz@redhat.com>

Remove unused .filesystem-unmount-others-shared polkit action

Signed-off-by: David Zeuthen <davidz@redhat.com>

Remove support for ID_SEAT=all

We might bring this back in one shape or the other, but remove it for
now until I get a chance to talk to Kay and Lennart some more.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add Transifex config and pull latest translations

Signed-off-by: David Zeuthen <davidz@redhat.com>

Remove danish translation

We'll get this from Transifex going forward.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Make $(udisks2.device) in authentication messages include the vendor/model

... if available. Here's an example of what it looks like

 http://people.freedesktop.org/~david/udisks2-detailed-auth-message.png

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add some more translation context and strings

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add a bunch of translation comments

Signed-off-by: David Zeuthen <davidz@redhat.com>

Reword parts of the udisks(8) man page

Signed-off-by: David Zeuthen <davidz@redhat.com>

Fix typo

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add separate polkit actions for ejecting media

It's a little weird to overload the modify-device action especially
since this doesn't change any bits on the media.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Fix typos in policy file

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add missing "on" in udisks(8) man page

Signed-off-by: David Zeuthen <davidz@redhat.com>

Remove section about ID_DRIVE* properties

It is really inappropriate to mention these here. Instead these
properties should be mentioned in the udev docs, along with other key
properties such as ID_FS_TYPE, ID_MODEL, ID_VENDOR, ID_WWN, ID_CDROM*
and so on. I've talked to Kay about this several times and he seems to
agree, just needs to happen.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Use upper-case for all man page sections

Signed-off-by: David Zeuthen <davidz@redhat.com>

udisksctl: Make man page point to API STABILITY section of udisks(8) man page

Signed-off-by: David Zeuthen <davidz@redhat.com>

Move API STABILITY remarks to udisks(8) man page

Signed-off-by: David Zeuthen <davidz@redhat.com>

Move audience blurb from README to the udisks(8) man page

Also point people there from README.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Rearrange docs so man pages come first

In particular, ensure that the udisks(8) man page comes first and use
"Introduction to udisks" as its purpose.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Fix 'make distcheck'

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add section about polkit actions to the docs

Also,

 - Add more comments to the .policy file (since it's included verbatim)
 - Explicitly state that polkit actions are not considered stable
 - Don't reference polkit actions in D-Bus docs
 - Cross-reference polkit actions chapter from udisks(8) man page
 - Mention ID_SEAT in udisks(8) man page

Signed-off-by: David Zeuthen <davidz@redhat.com>

Interpret ID_SEAT=all to mean that a device is available on all seats

This changes a couple of things

 - when mounting the device, we use different defaults so other users can
   access the filesystem too. We also mount the device in /media, not
   /run/media/$USER (unless an /etc/fstab entry exists for the device,
   of course)

 - if the drive is shared, we allow other users to unmount a
   filesystem mounted by another user (through a new polkit action)

 - we need to serialize multiple concurrent mount requests so we don't
   end up mounting the device twice... with this change, one one of the
   requests will succeed, the other one will get the AlreadyMounted error

   (basically, add locking, we needed that anyway .. it just only
   became apparent with this change because multiple sessions tried to
   mount the shared drive.)

Signed-off-by: David Zeuthen <davidz@redhat.com>

Fix reference count mistake in previous commit

Signed-off-by: David Zeuthen <davidz@redhat.com>

Make on_same_seat() also work for drive objects

Otherwise it won't work when used for the Drive.Eject() method which
operates on a Drive object instead of a Block object.

Also use Drive:seat instead of peeking into the udev database.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add versioning macros

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add multi-seat support

The first change is that we require additional authorization to
mount/unlock/modify a device, if it is on another seat than the
caller.

The other change is that we export the seat id as a property on the
org.freedesktop.UDisks2.Drive D-Bus interface. This can be used by
auto-mounter infrastructure (such as GVfs) to hint that a device
should not be automatically mounted unless it's plugged into the same
seat as where the environment is running.

All this is based on the libsystemd-login library - if this library is
not available at build-time, multi-seat support will not be enabled.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Work around missing serial/wwn on VMware hard disks

This is pretty much like how we work around the same for VirtIO so it
looks like a hard disk instead of a fake block device.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Use libacl library instead of setfacl(1)

https://bugs.freedesktop.org/show_bug.cgi?id=48842

Signed-off-by: David Zeuthen <davidz@redhat.com>

Update link to GVolumeMonitor docs

Stable/unstable in GLib means something completely different from what
we are discussing here, so to avoid confusion, point to the stable API
docs for GVolumeMonitor, not the unstable ones.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Mention udisks audience (and non-audience) in README

See https://plus.google.com/u/0/110773474140772402317/posts/eeJF4AHoh7a
for why this is a good idea.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Post-release version bump to 1.95.0

Signed-off-by: David Zeuthen <davidz@redhat.com>

Update NEWS for release

Signed-off-by: David Zeuthen <davidz@redhat.com>

Make escaping work properly with non-ASCII UTF-8 strings

Basically, we need to only unescape \ and " otherwise things won't
work with non-ASCII characters.

https://bugs.freedesktop.org/show_bug.cgi?id=48502

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add udev rules for identifying devices which should not be shown

Copied from udisks1 branch, see

 http://cgit.freedesktop.org/udisks/tree/data/80-udisks.rules?h=udisks1&id=1.0.4#n178

and adjusted to use the partition properties set by blkid and to use
UDISKS_IGNORE instead of UDISKS_PRESENTATION_HIDE.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Fix unmounting large disks when not asked to

udisks2 was "Cleaning up mount point" claiming the device no longer existed.
Disk size may be larger than INT_MAX for large disks.

https://bugs.freedesktop.org/show_bug.cgi?id=48155

Signed-off-by: David Zeuthen <davidz@redhat.com>

Convert some g_warning() uses to udisks_warning()

Signed-off-by: David Zeuthen <davidz@redhat.com>

Change some D-Bus types from bytestring ('ay') to UTF8 ('s')

We are 100% sure that the changed items are always UTF-8 especially
because udisks_device_udev_string() guarantees this and this function
is used before setting the affected properties.

This breaks the D-Bus API but there is no change in the libudisks2.so
ABI since byte-strings and UTF-8 strings are mapped the same way when
GDBus is being used. It may require restarting the system, but apart
from that there is minimal interference from this change.

The remaining byte-strings in the API are things that either comes
from configuration files (such as /etc/fstab and /etc/crypttab), the
kernel (e.g. sysfs paths) or udev rules (e.g. device symlinks). We
could require UTF-8 here and just reject items that are not valid
UTF-8 but there's really no point in that.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Clarify docs for Filesystem.Mount() and Filesystem.Unmount()

Signed-off-by: David Zeuthen <davidz@redhat.com>

Update docs

Signed-off-by: David Zeuthen <davidz@redhat.com>

Fix how we sign the tarballs

This mistake was pointed out by Adam Sampson <ats@offog.org>, thanks
for noticing!

Signed-off-by: David Zeuthen <davidz@redhat.com>

Pass --readonly to cryptsetup(8) if device to unlock is read-only

With this fix, locked SD cards with LUKS now work.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Post-release version bump to 1.94.0

Signed-off-by: David Zeuthen <davidz@redhat.com>

Update NEWS for release

Signed-off-by: David Zeuthen <davidz@redhat.com>

Tweak the docs some

Signed-off-by: David Zeuthen <davidz@redhat.com>

Properly escape all device files

At least in the default install, udev never includes a space or quote
in the device file but better safe than sorry.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Ensure that the loop file name we pass to the kernel is always NUL-terminated

Signed-off-by: David Zeuthen <davidz@redhat.com>

Ensure that whatever we pass as -t to mount(8) is in a whitelist

IOW, ensure that you can't pass e.g. 'nfs../blah' to load the whole NFS
stack. Now we error out, like this

 $ udisksctl mount --block-device /dev/sdc3 -t nfs../blah
 Error mounting /dev/sdc3: GDBus.Error:org.freedesktop.UDisks2.Error.OptionNotPermitted: Requested filesystem type `nfs../blah' is neither well-known nor in /proc/filesystems nor in /etc/filesystems

This is a bug that was already fixed for udisks 1.x, see

 https://bugs.freedesktop.org/show_bug.cgi?id=32232
 CVE-2010-4661: Arbitrary kernel module load

But we apparently forgot to fix this in the move from udisks 1.x to
2.x. Pointed out by Sebastian Krahmer <krahmer@suse.de>, thanks!

Signed-off-by: David Zeuthen <davidz@redhat.com>

Force MediaRemovable to TRUE for e.g. SD cards using the mmc layer

Otherwise udisks_client_get_drive_info() will return

 drive-removable-media-flash-sd

for /dev/mmcblk0p1 instead of

 media-flash-sd

and we want the latter.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Run gtkdoc-rebase to fix up links for GLib types

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add a section about /org/freedesktop/UDisks2/jobs/* objects

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add a section about /org/freedesktop/UDisks2/drives/* objects

Signed-off-by: David Zeuthen <davidz@redhat.com>

Reword the section about block device objects a little bit

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add a section about /org/freedesktop/UDisks2/block_devices/* objects

Signed-off-by: David Zeuthen <davidz@redhat.com>

Don't include sys/acl.h - it's not needed

Signed-off-by: David Zeuthen <davidz@redhat.com>

Post-release version bump to 1.93.0

Signed-off-by: David Zeuthen <davidz@redhat.com>

Update NEWS for release

Signed-off-by: David Zeuthen <davidz@redhat.com>

Move to /usr-only setup and get rid of hardcoded paths to /lib and sbin

With the whole OS in /usr, tricks like these are no longer
needed. Howver, distros with different content in e.g. /sbin and
/usr/sbin will have to move things like the /usr/sbin/umount.udisks2
binary themselves etc.

Signed-off-by: David Zeuthen <davidz@redhat.com>

client: Fix bit shifts of flags on 32 bit

Also, the flags should be unsigned, not signed.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Use /run/media/$USER for mounting

Also use ACLs on the $USER sub-directory so only $USER (and not any
other user) can access the mounted filesystem.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Don't free object twice when deleting a loop device

Duh!

Signed-off-by: David Zeuthen <davidz@redhat.com>

Avoid using $XDG_RUNTIME_DIR/media for now

Needs a bit more thought on the security side.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Post-release version bump to version 1.92.0

Signed-off-by: David Zeuthen <davidz@redhat.com>

Update NEWS for release

Signed-off-by: David Zeuthen <davidz@redhat.com>

Fix a couple of uninitialized warnings and simplify SMART self-test handling

In particular, this fixes a crash when running a SMART self-test on a
drive and then yanking it.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Shut up valgrind complaing about LOOP_GET_STATUS64 ioctl

Signed-off-by: David Zeuthen <davidz@redhat.com>

Use g_dbus_interface_dup_object() and check return value

This fixes potential (multi-threading related) problems where the
GDBusObject is removed just before or in the middle of handling a
call. Also bump required GLib version to (unreleased) 2.31.13 since
that's the version this API was introduced in.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Do coldplug for block devices twice

Otherwise we may miss some associations between LUKS devices and other
block devices. This bug only became evident once we started sorting
block devices before processing them.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Make sure that Drive:SortKey sorts e.g. sdz before sdaa

Also make sure that we e.g. process sdb before sdad to get
Drive:SortKey set using sdb rather than sdad.

With this change we get the expected output that corresponds to kernel
probing order. Which (on good days) may correspond to slot and
enclosure order.

MODEL                     REVISION  SERIAL               DEVICE
--------------------------------------------------------------------------
HP LOGICAL VOLUME         3.00      5001438008F61790     sda
SEAGATE ST3300657SS       0006      3SJ1S7K600009051M0CE sdad sdb
SEAGATE ST3300657SS       0006      3SJ1S4MH00009052RG6Z sdae sdc
SEAGATE ST3300657SS       0006      3SJ1RFP900009051HZ5S sdaf sdd
SEAGATE ST3300657SS       0006      3SJ1S66L00009052QGZA sdag sde
SEAGATE ST3300657SS       0006      3SJ1RPFJ00009052BZ10 sdah sdf
SEAGATE ST3300657SS       0006      3SJ1RWZ100009101T12H sdai sdg
SEAGATE ST3300657SS       0006      3SJ1S7C500009052RKPP sdaj sdh
SEAGATE ST3300657SS       0006      3SJ1S40K00009101T1XU sdak sdi
SEAGATE ST3300657SS       0006      3SJ1RJS700009050VYEA sdal sdj
SEAGATE ST3300657SS       0006      3SJ1RSJC00009052MWTB sdam sdk
SEAGATE ST3300657SS       0006      3SJ1QNMQ00009052NECM sdan sdl
SEAGATE ST3300657SS       0006      3SJ1S3NE00009101TB5Y sdao sdm
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30012560      sdn
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30064749      sdo
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30039741      sdp
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30064055      sdq
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30030732      sdr
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30010853      sds
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30024753      sdt
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30065104      sdu
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30031826      sdv
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30025874      sdw
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30063724      sdx
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30064486      sdy
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30032003      sdz
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30021345      sdaa
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30067287      sdab
WDC WD1002FAEX-00Y9A0     01.01V10  WD-WCAW30014347      sdac

Signed-off-by: David Zeuthen <davidz@redhat.com>

udisksctl: sort the drives using Drive:SortKey for 'status' verb

Signed-off-by: David Zeuthen <davidz@redhat.com>