Add workaround annotation for udisks_client_get_block_for_dev()

The gobject-introspection parser does not know how to properly handle dev_t
arguments (https://bugzilla.gnome.org/show_bug.cgi?id=584517) and treats them
as gint. But as the native API expects a 64 bit integer, you'll get garbage in
the upper word on 32 bit platforms, which cause
UDisks.Block.get_block_for_dev() to never find a match when using through
introspection.

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

integration-test: Robustify test_luks_forced_removal()

Wait a bit for the object path to disappear after yanking the device away.
Also, ensure that the device is always re-added after this, even if there are
test failures or exceptions in between, otherwise all subsequent tests will
fail as well.

integration-test: Add test case for mounting removable device

This should only require the org.freedesktop.udisks2.filesystem-mount
privilege, not -system.

integration-test: Create a read-only fake CD-ROM device

We will need this for further tests, e. g. to reproduce
https://bugs.freedesktop.org/show_bug.cgi?id=53237 or to check required
privileges for a removable device.

scsi_debug is not capable enough to emulate the whole SCSI CD-ROM command set,
thus we need a temporary udev rule (in /run/udev/rules.d/) to ensure that
blkid runs on scsi_debug CD devices.

test_polkitd.py: Add --replace option

We do not need this for udisks' test case (as we run that on a private bus),
but it's useful for system integration tests which use this mock polkitd.

Use all-caps for RAID

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

Add textual descriptions for IMSM Raid members

http://people.freedesktop.org/~david/imsm_before.png
http://people.freedesktop.org/~david/imsm_after.png

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

Add support for creating exFAT filesystems and changing exFAT labels

Tested with the exFAT userspace utilities available from

 http://code.google.com/p/exfat/

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

integration-test: Consistent mount path validation

Do not require mount paths to be in /run/media/, leave that as an
implementation detail (the code supports both cases). The other mount point
tests already do this.

Update list of recovery/system partitions

In particular add "System Reserved" (Windows 7). Also, since
ID_FS_LABEL nowadays always has underscores, replace spaces with
underscores in some of the existing patterns.

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

Post-release version bump to 1.100.0

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

Update NEWS for release

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

Update docs

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

Update translations from Transifex

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

Update /run/udisks2/loop file before setting up the loop device

Otherwise we'll race to read back from it (to set the SetupByUID
property) when we get an uevent on the loop device.

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

Ignore non-Linux software on SanDisk Cruzer

We've been happily ignoring this because of firmware-bugs in its
CD-ROM emulation but this may soon change, see

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

In fact, newer revisions of the Cruzer (including the one Kay has) has
fixes for this (making the fake CD-ROM appear on the desktop).

The CD-ROM will still appear in GNOME Disks and it can be mounted that
way if you really want to access these files. It just won't appear in
the desktop, e.g. the file manager or file chooser until it's manually
mounted.

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

Create cleanup object before coldplugging

Otherwise things like Loop.SetupByUID will not be set correctly.

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

Use ATA PASS-THROUGH (16) command instead ATA PASS-THROUGH (12) command

When needed (ie. when someone files a bug), we'll add support for
trying 12-byte commands (I don't know how it'd work right now).

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

Add "Linux Filesystem" GPT partition type

According to http://en.wikipedia.org/wiki/GUID_Partition_Table there
is now a new dedicated Linux partition type so Microsoft's Basic Data
no longer has to be overloaded.

 Previously Linux used the same GUID for the data partitions as
 Windows. Linux never had a separate unique partition type GUID
 defined for its data partitions. This created problems when
 dual-booting Linux and Windows in UEFI-GPT setup. This new GUID was
 defined jointly by GPT fdisk and GNU Parted developers. It is
 identified as type code 8300 in GPT fdisk.

So add this new type and remove "Basic Data" from the Linux section
(it was listed in both the Linux and Microsoft section until now).

For now (and the foreseeable future) we will use "Basic Data" instead
of "Microsoft Basic Data" since a lot of systems in the wild is using
this type already and it would cause confusion to display the string
"Microsoft Basic Data" on Linux systems

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

Add a Block.Rescan() method

This is needed when restoring a disk image - the kernel won't reread
the partition table on its own and BLKRRPART is unfortunately not
permitted if not uid 0.

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

Ignore "Apple Boot" GPT partitions

My USB disk set up for FileVault on Lion, has one of these. It's not
useful to display in the user interface.

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

client: Add Apple Core Storage GPT partition type

This is used for FileVault partitions and probably other things as
well.

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

Black-list seemingly invalid WWN for SAMSUNG SP1604N hard disks

In this case, use the serial number as the VPD (the serial number does
vary) otherwise we'll end up only creating a single Drive object.

This was reported in https://bugzilla.redhat.com/show_bug.cgi?id=838691

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

Update compiler warning flags

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

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>