Fix typo

... now 'make distcheck' works again.

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

Disable -Wformat-nonliteral for a tiny block of code

... in this case, we happen to know better than the compiler what we
are doing.

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

Add more compiler warnings

You'd think GNOME_COMPILE_WARNINGS([maximum]) would be good enough but
apparently not. I hate build systems.

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

Fix some uninitialized variable warnings

These only occur when building with -O2, not -O0, ofcourse, thanks gcc.

Pointed out by Colin Walters <walters@verbum.org>.

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

Don't return uninitialized variable on inhibitor error path

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

Inhibit system shutdown, sleep and idle when formatting and erasing devices

Formatting and ATA secure erase are both potentially long-running
tasks that can easily take hours to complete. Especially for ATA
secure erase, losing power while the command is pending can be very
very bad.

Here's udisks taking an inhibitor lock:

 # gdbus call --system --dest org.freedesktop.login1 --object-path /org/freedesktop/login1 --method org.freedesktop.login1.Manager.ListInhibitors
 ([('shutdown:sleep:idle', 'Disk Manager', 'Formatting Device', 'block', uint32 0, uint32 14172)],)

On non-systemd systems, this does nothing.

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

When setting up LUKS, erase the cleartext device rather than the cryptotext one

That way the cryptotext device will be full of noise instead of
zeroes, that way not conveying that there isn't a lot of data on the
device.

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

Don't report progress for ATA Secure Erase when expected time is >508 minutes

I have a Samsung drive, see [1], that reports 510 in its IDENTIFY data
but the secure erase only takes around 214 minutes:

 Jul  3 00:18:18 mars udisksd[9480]: Commencing ATA enhanced secure erase of /dev/sdd (SAMSUNG-HD103UJ-S13PJ90S703014). This operation is expected to take at least 510 minutes to complete
 [...]
 Jul  3 03:52:01 mars udisksd[9480]: Finished securely erasing /dev/sdd (SAMSUNG-HD103UJ-S13PJ90S703014)

See

 http://people.freedesktop.org/~david/gnome-disks-secure-erase-with-greater-than-508-min.png

for an example of how GNOME Disks interprets this and compare it with

 http://people.freedesktop.org/~david/gnome-disks-ata-secure-erase-progress.png

for when progress is available.

[1] :

/org/freedesktop/UDisks2/drives/SAMSUNG_HD103UJ_S13PJ90S703015:
  org.freedesktop.UDisks2.Drive:
    Configuration:              {}
    ConnectionBus:
    Ejectable:                  false
    Id:                         SAMSUNG-HD103UJ-S13PJ90S703015
    Media:
    MediaAvailable:             true
    MediaChangeDetected:        true
    MediaCompatibility:
    MediaRemovable:             false
    Model:                      SAMSUNG HD103UJ
    Optical:                    false
    OpticalBlank:               false
    OpticalNumAudioTracks:      0
    OpticalNumDataTracks:       0
    OpticalNumSessions:         0
    OpticalNumTracks:           0
    Removable:                  false
    Revision:                   1AA01181
    RotationRate:               -1
    Seat:                       seat0
    Serial:                     S13PJ90S703015
    Size:                       1000204886016
    SortKey:                    01hotplug/1341418168551507
    TimeDetected:               1341418168551507
    TimeMediaDetected:          1341418168551507
    Vendor:
    WWN:                        0x50024e9200c0c694
  org.freedesktop.UDisks2.Drive.Ata:
    AamEnabled:                                 false
    AamSupported:                               true
    AamVendorRecommendedValue:                  254
    ApmEnabled:                                 false
    ApmSupported:                               true
    PmEnabled:                                  true
    PmSupported:                                true
    SecurityEnhancedEraseUnitMinutes:           510
    SecurityEraseUnitMinutes:                   510
    SecurityFrozen:                             false
    SmartEnabled:                               true
    SmartFailing:                               false
    SmartNumAttributesFailedInThePast:          0
    SmartNumAttributesFailing:                  0
    SmartNumBadSectors:                         0
    SmartPowerOnSeconds:                        31629600
    SmartSelftestPercentRemaining:              0
    SmartSelftestStatus:                        success
    SmartSupported:                             true
    SmartTemperature:                           295.14999999999998
    SmartUpdated:                               1341418168

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

integration-test: More robust syncing

Repeatedly settle the client until all main loop events have been processed,
instead of just calling it once. This avoids a lot of spurious, but obstinate
test failures due to properties not being updated yet.

Also add an extra sleep after the manual mount and dd call to allow some more
time for the changes to get picked up. This is a hack, but not easily fixed in
a cleaner way.

Don't use g_propagate_error() with a NULL GError

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

Add support for VMWare filesystem types and GPT partition types

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

integration-test: Use Gio.TestDBus() again

Now that we do not need to call tests under a different UID any more, we can
drop the custom D-BUS config and dbus-launch, and go back to the much simpler
Gio.TestDBus().

integration-test: Use mock polkitd

Add src/tests/test_polkitd.py which provides a simple mock polkitd on a given
bus, as well as some convenience API. This allows test cases to precisely
control which actions get permitted and forbidden without having to assume
anything about the system polkit configuration, users, active/inactive consoles
etc. This is meant to go into polkit itself at some point.

Simplify integration-test to use test_polkitd and get rid of the uid changing
and calling of udisksctl.

Add ATA Secure Erase functionality

This is available via a new Drive.Ata.SecurityEraseUnit() method but
also via the Block.Format() method by setting the @erase parameter to
'ata-secure-erase' or 'ata-secure-erase-enhanced'. The latter is the
preferred way of using the new functionality.

Secure erase requires the calling user to be authorized for the (newly
added) org.freedesktop.udisks2.ata-secure-erase polkit action. By
default this requires administrator authentication no matter what kind
of drive is being used.

Since a secure erase cannot be canceled, a new Job:Cancelable property
has been added so the UI can avoid displaying a cancel button.

Considerable care have been taken in the implementation so things are
properly logged to the system log in case something goes wrong - for
example, if a device is yanked it may remain locked and needs to be
unlocked using the temporary 'xxxx' password that is used.

http://people.freedesktop.org/~david/gnome-disks-ata-secure-erase.png
http://people.freedesktop.org/~david/gnome-disks-ata-secure-erase-warning.png
http://people.freedesktop.org/~david/gnome-disks-ata-secure-erase-confirmation.png
http://people.freedesktop.org/~david/gnome-disks-ata-secure-erase-progress.png

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

Init interface proxies on hotplug too

GDBusObjectManagerClient only emits object-added for an object with N
interface (not N separate interface-added signals).

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

Issue BLKRRPART if wiping a device with an existing partition table

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

Set Job:ExpectedEndTime when erasing a device

Do this by introducing some plumbing in UDisksBaseJob to estimate the
end time based on the UDisksJob:progress property. We do this by using
a 100-period moving average (actually, up to 100 periods). Works out
great in practice.

Also switch the format of the time used on the org.fd.UDisks.Job
interface to be micro-seconds instead of seconds. This is not really a
break as these properties were always zero in any released udisks2
version.

http://people.freedesktop.org/~david/gnome-disks-estimated-time-remaining.png

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

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>