Bug 51439 – udisks should hide lvm PVs

Hide partitions marked as containing LVM and RAID. This is only useful
for encrypted partitions.

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

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add erase parameter to Block.Format() to zero out device before formatting it

... and also provide new job properties so it's possible to build a
meaningful UI around it. This makes things like this possible

 http://people.freedesktop.org/~david/gnome-disks-erase-overwrite-option.png
 http://people.freedesktop.org/~david/gnome-disks-long-running-operations.png

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Move Drive:Configuration property up a bit

Makes the docs look better.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Drop the 'drive-' prefix for drive configuration files

In the future we may want to use the same configuration file format
for block devices.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Default to mode 0600 for drive config files

This is important because we *may* want to store passwords in them at
some point (such as the ATA security stuff).

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add DriveAta.PmWakeup() method

The UI already has a "Standby" so for symmetri it's desirable to also
have a "Wakeup" menu item.

http://people.freedesktop.org/~david/gnome-disks-resume-from-standby.png

For now it seems to be good enough to just read some sectors from the
disk. We can always change this to something more sophisticated later
if needed.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Don't require auth for Standby'ing non-system drives on own seat

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add drive configuration interfaces and configuration files

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

integration-test: Fix parsing of daemon path in system mode

Only take the first word of the Exec= line in the udisks D-BUS service file, so
that extra arguments like --no-debug get ignored.

With that, integration-test runs fine against the system installed binaries
again.

Ship integration-test in the tarball

integration-test: Fix Polkit.test_internal_fs_nobody test case

Do not call tools/udisksctl libtool wrapper as nobody, call the actual ELF
binary in tools/.libs. Otherwise libtool cannot write its temporary file as
user nobody.

Add Block.OpenForBenchmark() method

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Mark Realtek rts5229 based card readers as flash drives.

Signed-off-by: Martin Pitt <martinpitt@gnome.org>

integration-test: Check polkit denial

Add a test case for a polkit-denied operation: we try to mount an internal disk
as user "nobody". Technically the system polkit configuration could allow this,
but let's hope that this is not the case (this is an integration test, after
all).

We need to supply our own D-BUS configuration for this to allow nobody to
access our temporary fake system bus. Thus drop the usage of Gio.TestDBus and
replace it with calling dbus-launch.

integration-test: Explicit polkit tests

Since commit 203533a0 udisks can run without polkit, so run most of the
testsuite without polkitd. Introduce a new "Polkit" test that checks operation
with polkitd running. This only has a very shallow test for now, as further
tests require some reorganization.

integration-test: Fix existance test of local state dirs

Add Documentation key to systemd unit file

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Catch up with latest polkit guidance

See

 http://cgit.freedesktop.org/polkit/commit/?id=acf3a06e55f9ca8a7f7bfa012c24e8794d27c85f
 https://bugzilla.gnome.org/show_bug.cgi?id=677718

for details.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Post-release version bump to 1.99.0

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Update NEWS for release

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

udisksd: work if polkitd is not available

Upstream polkit now recommends this, see

http://cgit.freedesktop.org/polkit/commit/?id=84e42887d413cba0039f92f0c56a25c70c20fe6e

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

integration-test: Run on temporary local DBus

Run a temporary local session DBus, using the new Gio.TestDBus API from glib
2.34. Pretend that it is the system DBus, and run our udisksd as well as a
polkitd instance on that. This avoids interacting with the real system
DBus/udisksd and any currently running session, and obsoletes the need for
suspending gvfs-udisks2-volume-monitor (which has always been a GNOME specific
hack).

The polkitd debug output will now appear in the dameon log as well, if you use
the --log-file/-l option.

integration-test: Quiesce wipefs output

It previously cluttered the test output with the erased bytes, like:

test_ext3 (__main__.FS)
fs: ext3 ... [cli] 2 bytes were erased at offset 0x438 (ext3)
they were: 53 ef
[ud] ok

integration-test: Update expected Luks mount points

Mounts are now done in /run/media when there is a known user name. Update the
Luks tests to get along with both /media and /run/media/user.

Wait for the correct device after formatting

When creating a LUKS device, the object to wait on after mkfs (cleartext
device-mapper) is not the same as we originally started with (crypto_LUKS
device). Always wait on the object we called mkfs on.

integration-test: Mitigate race after mounting

Also wait for udev and the rest of the system after calling mount before
asserting the mount path exists. Fixes a common failure of FS tests.

Rework part of the polkit variable documentation

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add id.* and partition.* polkit variables

These variables can be useful for making authorization decisions.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Use "device" instead of "block_device" in the polkit variables

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

If a block device has ID_PATH set, consider it to be a drive

This fixes problems with the devices where ID_SERIAL or
ID_WWN_WITH_EXTENSION is not set. For example, block devices from the
rts_pstor driver.

See https://bugzilla.redhat.com/show_bug.cgi?id=828492

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Move table with polkit variables to the "Authorization Checks" chapter

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

integration-test: Update expected exception names

integration tests: Suspend gvfs-udisks2-volume-monitor

... instead of the old gdu one.

integration tests: Update expected mount points

mounts are now done in /run/media when there is a known user name. Update the
tests to get along with both /media and /run/media/user.

Rename current polkit variables and also add some new ones

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

Add some documentation about polkit variables

In particular, guarantee that $(udisks2.device) is always a device
file, if set.

We may want to add more variables in the future such as
udisks2.drive.{vendor,model,serial,wwn}.

This is related to the new experimental JS-based polkit authority, see

 https://plus.google.com/u/0/110773474140772402317/posts/LB1QjaoaEn2

for more details.

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

Drive: Refuse to eject drives that appear to be in use

Unfortunately eject(1) is trying to be "helpful" insofar that it will
unmount partitions in use. While this is indeed handy when running
eject(1) from the command-line it's not a very good idea for us since
with the way its used in udisks, it effectively allows unmounting
partitions possibly mounted by other users.

So check if block devices for the drive is either mounted or unlocked
and bail if this is the case.

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

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>