Bug 23823 – NULL dereference

Bug 23824 – Unused variable

Bug 23825 – Leak in update_info_mount_state

Bug 23826 – NULL dereference

Bug 23827 – Undefined value

Bug 23828 – Undefined value in umount-devkit.c

Bug 23829 – Undefined value in partutil.c

Use WindowsStyleCaps for property names

Technically any UTF-8 string can be used for a property name. However,
some tools are incapable of handling property names with dashes ('-')
in them. So change to using the recommending naming. See

http://lists.freedesktop.org/archives/devkit-devel/2009-September/000430.html

for discussion.

This is based on a patch from Davide Bettio <davide.bettio@kdemail.net>.

Also be paranoid about the UUID containing '/' characters

This can't really happen (for known filesystems) so this "fix" is
really only for people playing along at home. See

 http://bugs.freedesktop.org/show_bug.cgi?id=23235

for more details.

Rework ATA SMART handling

Basically make everything a lot simpler. Also don't store blobs in a
database. For now at least - we might want to add that feature back at
some point.

Bug 22914 – Require password to eject a CD or DVD mounted by the same user

Apparently this got messed up in the port to polkit1.

Post-release version bump to 007

Update NEWS for release

Bug 23235 – Cannot mount disc with / (slash) in label

Make FilesystemMount() accept an 'auth_no_user_interaction' option

This allows automounters to request that no user interaction should ever
take place. See

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

for discussion.

Ignore loop and ram devices for now

Add a new :device-presentation-nopolicy property

Desktops should read this property and avoid automounting etc. if it
is set to 1.

Bug 23309 – Udev rules probe floppy drives when they shouldn't

Fix up object path mapping

In particular this failed with AoE devices which has basenames of the
form "etherd!e0.1". Should work for everything now.

Hide recovery, boot, and hidden partitions from desktops

Set DKD_PRESENTATION_HIDE for partitions which desktops should hide, such as
EFI, Apple bootstrap, or recovery partitions.

For recovery partitions is a direct port of the earlier hal fdis:
http://cgit.freedesktop.org/hal/tree/fdi/policy/10osvendor/20-storage-methods.fdi#n80

Apple boostrap, EFI, and hidden partitions now don't rely on labels any more,
but directly check partitioning schema (mbr, GUID) and types.

libgdu evaluates this and exports it as device-presentation-hide
property, which gvfs respects.

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

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

Require latest libatasmart

This version of libatasmart has a new ABI/API with new features. We
won't initially take advantage of these new features so just do a
simple port for now.

Rely on the disk to spin itself down

In other words, instead of playing games with watching
/sys/block/sdX/stat files and sending a STANDBY IMMEDIATE command when
these haven't changed for a while, just use the STANDBY command itself
with the timeout value.

This avoids having to wake up the daemon every five seconds. In
addition, this change ensures that device specific minimums are
respected.

We didn't do this in the original commit because while testing the
feature, some of disks, notably ones from WD, appeared to not respect
the STANDBY command. However, when moved to another enclosure (the
disks was initially in a SATA PM enclosure, it got moved to a SAS
enclosure) things started working again. Also turns out the WD disk in
question has a device specific timeout of 10 minutes.

Add LinuxMdCreate method

Also fix a bug in PartitionTableCreate so we don't return until the
partition table has actually been created.

Add Device:drive-is-rotational property

This new property can be used to make a distinction between hard disks
and SSDs, see

http://people.freedesktop.org/~david/palimpsest-new-treeview.png

for an example.

Fix up ATA SMART detection

Recent udev started using ata_id instead of scsi_id.

Also migrate from SYSFS to ATTR/ATTRS.

Misc fixes for optical disc detection

Move data files into a separate directory

Move implementation of DrivePollMedia into a separate process

Much much better to do this from a helper process since some
devices/drivers may hang the process in state D (uninterruptible IO)
until the device was removed. This was the last place where we opened
the device from the daemon.

Work around media detection problems

Seems that the kernel won't revalidate media unless O_NONBLOCK is
passed. This contradicts how things are supposed to work (O_NONBLOCK
is a hint to avoid testing things like media state), see
http://bugzilla.kernel.org/show_bug.cgi?id=13029 for more information.

So to be on the safe side, open the device both with and without
O_NONBLOCK.

Add --poll-for-media option to devkit-disks(1)

Pass size of buffer to avoid Z_BUF_ERROR when compressing ATA SMART blob

Fix polling process title and turn off noisy debugging spew

Fix segfault when starting a Linux MD array

Add a way to spin down drives

Post-release version bump to 006

Update NEWS for release

Also includes news about the 004 release since that slipped in the
previous release...

Use AM_SILENT_RULES if available

Clean up build a bit

Remove unused SMART test files

Sort out dbus-glib registration mess

We sometimes segfaulted when stopping a RAID device. The problem here
was that we kept the exported D-Bus object around until it was
finalized (by using weak refs). This was problematic because several
change events were treated as add events but the state of the RAID
device was such that we didn't want to add the object anyway. So we
ended up unreffing the object. Which means that the job completion
func accessed a unreferenced object.

The solution here is to remove the object from our hashes as soon as
we want to remove it.

Since we don't have dbus_g_connection_unregister_g_object() just yet
(available in dbus-glib >= 0.81) we just keep the object on the
bus. If we later wants to add a new object but the old object is still
around we use an undocumented hack to unregister the object.

You will also need eggdbus >= 0.5, otherwise introspection won't work.

Add API to check/repair Linux MD RAID arrays

Use ID_CDROM_MEDIA to detect optical drives

This is available since udev 142 that we already depend on. This
change makes my USB stick from Sandisk "work" as "expected":

http://people.freedesktop.org/~david/thanks-sandisk.png

Update udev rules

 - depend on udev >= 142 since that will pull in /sbin/blkid
 - avoid probing dm or md if it has been done already
 - don't create symlinks for dm

GIO modules, for the polkit action lookup module, need to be prefixed with lib

Better handling of Floppy and Zip drives

Welcome to the 1980s

Bug 22149 – Do not poll floppy drives

Avoid polling PC floppy drives since that makes a lot of
noise. DrivePollMedia() still works, e.g.

 dbus-send --system --print-reply --dest=org.freedesktop.DeviceKit.Disks  \
           /org/freedesktop/DeviceKit/Disks/devices/fd0                   \
           org.freedesktop.DeviceKit.Disks.Device.DrivePollMedia

Note, though, that udev will not run /sbin/blkid on the device. So
desktops wanting to support this needs to ignore the fstype for floppy
drives and just attempt to mount it anyway.

Fix bug when retrieving historical ATA SMART data

Use gudev-1.0 instead of devkit-gobject

Can't believe how easy this port was...

Remove :drive-requires-eject and read udev props for detached/ejectable

Suspend USB device on DriveDetach() if it only has one interface

This is nice for turning the lights off...

Don't insert NULL pointers in details

Add a DriveDetach() method to properly power down USB hard disk enclosures

This work is inspired by this thread

 http://thread.gmane.org/gmane.linux.hotplug.devel/14079

that is referencing this blog entry

 http://elliotli.blogspot.com/2009/01/safely-remove-usb-hard-drive-in-linux.html

The only difference here is that instead of unbinding the usb driver
from the usb device, we unbind the usb-storage driver from the usb
interface in question. This is to better handle multi-function devices
(e.g. multiple USB interfaces).

Things like GVfs (ie. Nautilus) can use this new method to offer an
"Eject" option for USB enclosures that can invoke DriveDetach() on
this service. (In GVfs, such devices currently have no eject option
since they don't support removable media).

The PolicyKit extensions dir has moved

Add a pkg-config file

Use PolkitDetails, not GHashTable to pass details to PolicyKit

Only check for ATA SMART if the device reports non-removable media

Catch up with latest polkit API changes

Update to latest polkit API where PolkitAuthorizationResult is an object

Update to latest PolicyKit API

Remove unneeded code for PolicyKit

Post-release version bump

Port to polkit-1

Move to using a capital letter for commit messages

only consider Prefail attibutes when checking for bad attributes

add :device-media-detection-time property

This property aids desktop automounters in determining they shouldn't
automount devices that are created as a result of
formatting/partitioning.

add support for ext2

This is sometimes wanted for filesystems on flash devices.

properly detect devices with vfat on the whole disk

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

allow the 'flush' mount option for vfat

fix up some warnings

add property device-detection-time

document influential udev properties in the DeviceKit-disks man page

add device-presentation-hide property

This is useful for a number of things

 - hiding firmware/update partitions (similar to how volume.ignore is used
   in HAL to hide e.g. HP_RECOVERY, RECOVERY etc. partitions)

 - hiding unwanted things used in things like live cd environments, cf.
   https://bugzilla.redhat.com/show_bug.cgi?id=495170

be more careful when probing for ATA SMART

See https://bugzilla.redhat.com/show_bug.cgi?id=495286 for where we
trash a CD-RW medium when running off a live cd.

rework how we detect whether a device is busy

Commit c1447a2e8a0d83347330bbfd2810f98172e8b962 was a bit overeager
insofor that it made it impossible to create/delete/modify partitions
if other partitions on the same device are busy.

add temporary workaround for btrfs

See

 https://bugzilla.redhat.com/show_bug.cgi?id=495152#c31
 http://article.gmane.org/gmane.comp.file-systems.btrfs/2851

for details.

remove debug spew

also check if partitions on a device is busy

This fixes a bug where it was possible to erase /dev/sdb while
/dev/sdb1 was mounted.

use correct dmode for iso9660 mounts

Reported by yaneti@declera.com

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

add command line options to devkit-disks for refreshing ATA SMART data

fix some bugs in the man page

change default mount options for vfat to utf8=1 and shortname=lower

See https://bugzilla.redhat.com/show_bug.cgi?id=489222 for details.

fix a race where /etc/fstab mount points where sometimes removed

bump version

Should have done this right after releasing 003 but meh.

use /var/run/DeviceKit-disks, not /tmp as a temporary directory

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

remove hack for working around broken USB card reader

This add was added since one of my card readers appear to be broken;
other devices are fine.

On closer investigation this looks like a kernel bug. Filed here for
now

http://bugzilla.kernel.org/show_bug.cgi?id=13029

If needed we can add code to do the equivalent of 'touch /dev/sdXXX'
in the poller on the first ENOMEDIUM for a device being polled. But
let's see what the kernel guys says before resorting to such hacks.

remove debug spew from mount monitor

fix grammar/spelling for ATA SMART output in devkit-disks(1)

install udev rules into /lib/udev/rules.d not /etc/udev/rules.d

change devkit-disks(1) to use device node paths instead of object paths

also move umount.devkit over to use the correct object paths

add FindDeviceByMajorMinor() to look up a device by dev_t

prefix all D-Bus objects paths with /org/freedesktop/DeviceKit/Disks

fix udev rules to keep udev from complaining

rework some of the ATA SMART stuff, go back to using sqlite

remove erase= option and the notion of tasks within jobs

Secure erase is a bit K-rad and we never really implemented anything
but 0 and 1 passes. If we want this feature back (and I think we do;
e.g. it is useful to scrub the disks before selling a computer) we
should add a dedicated SecureErase() method that (optionally) can take
advantage of things like the ATA Secure Erase command-set.

While it is cute that jobs consist of tasks it is not used anymore now
that erase= is gone. So remove it to simplify the code.

remove a lot of unnecessary errors

For example it doesn't make sense to return this error

 org.freedesktop.DeviceKit.Disks.Error.NotPartition

since the client is already able to check in advance if the given
object really is a partition. So trim the error list to only contain
errors that the client can't predict. This leaves us with

 DEVKIT_DISKS_ERROR_FAILED,
 DEVKIT_DISKS_ERROR_BUSY,
 DEVKIT_DISKS_ERROR_CANCELLED,
 DEVKIT_DISKS_ERROR_INHIBITED,
 DEVKIT_DISKS_ERROR_INVALID_OPTION,
 DEVKIT_DISKS_ERROR_NOT_SUPPORTED,
 DEVKIT_DISKS_ERROR_ATA_SMART_WOULD_WAKEUP,

remove device-is-busy property

This should never have been a property since

 - we are not really ready to make that kind of guarantee; another
   program may have the device open and then we're going to fail
   with EBUSY and return a Busy D-Bus error

 - it is computed on demand and that makes it difficult to determine
   when it changes value and thus trigger a PropertyChanged event

Currently this property is not used by gnome-disk-utility. If it turns
out that we need something like this we can add a Device.IsBusy()
method.

export device-major and device-minor properties

This makes it easier for clients wanting to fool around with
/proc/self/mountinfo themselves.

use /proc/self/mountinfo and handle a device being mounted in multiple places

This is an effort to fix

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

Basically we have to somehow cope with a device being mounted in
multiple places. So we replace the property

 String device-mount-path

with an array of mount paths

 String device-mount-paths[]

We also try to make an effort to ensure that the shortest mount point
is shown before others. E.g. if you bind mount / on /foo then

 device-mount-paths = ["/", "/foo"]

This means that desktop clients can just take the first mount path.

Also make a note that this is the mount point in the root
namespace. On systems clients running in other namespaces care has to
be taken; in this case the client probably needs to examine
/proc/self/mountinfo on his own to determine if a given block device
is mounted or not.

Also drop support for the 'remount' option; we might add it back later
but since we now support multiple mount points it makes everything a
bit weird.

print the scheme of the main partition table, never the nested one

To avoid e.g. 'embr' for the scheme for logical partitions.