Bug 24264 – Crash on removing NULL value from hash in device_remove()

Properly handle the situtation where we're processing a 'change' event
but the 'remove' event has happened but is still queued up in our own
process. Also handle device file renames and 'move' uevents.

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

Bug 24351 – Firewire hard drive is considered system-internal

Firewire subsystem is called "ieee1394" in current Linux. Now check for both
"ieee1394" and "firewire". This fixes firewire drives to not be considered
system internal any more.

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

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

Bug 24343 – Unintended side-effect when calling DriveDetach()

Some laptops like the Dell Mini 10 have builtin SD card readers which
are wired through an internal USB bus. The current behaviour is that
if you click the eject button in nautilus, the card reader device gets
ejected and disconnected, and cannot be reactivated ever, except by a
reboot.

The cause of this is that DK-d's update_info_drive() sets "can_detach"
for all USB devices currently, which causes
devkit-disks-helper-drive-detach to be called on them.

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

Bug 24129 – Please mount vfat disks with shortname=mixed

The previous default, shortname=lower, breaks all-uppercase file names ("touch
FOO" creates "foo"), thus breaks rsync, and Windows compatibility.

The default was changed in the Linux kernel as well, see
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=955234755ce4a2c33cfc558912aa8f2148cc1fc6

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

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

Bug 24054 – Devickit-disks doesn't support changing ntfs label

Bug 24053 – MMC cards aren't automounted.

Show MMC/MemStick cards by default. The previous whitelist in the
udev rules ignored them.

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

Signed-off-by: David Zeuthen <davidz@redhat.com>
Signed-off-by: Martin Pitt <martin.pitt@ubuntu.com>

Bug 24052 – CDROM eject button is locked while CDROM is mounted

Unlike in the hal world, we do not have a daemon polling CD drives for eject
button presses. In order to make hardware tray eject buttons work, unlock the
tray after mounting a CD.

This is pretty much equivalent to yanking out USB sticks, which we already
handle reasonably (detecting disappeared device, force-unmounting).

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

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

Actually inhibit the daemon when Inhibit() is called

Guess this was forgotten with the polkit-1 porting effort.

Post-release version bump to 008

Update NEWS for release

Bug 23926 – Error creating partition

When creating a partition we may already have received the uevent for
the new partition. Check for this before waiting for it.

Bug 23541 – PartitionTableCreate method times out

... when 'none' is specified as a parameter.

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

Bug 23579 – Must allow DBus Properties interface for g-d-u to work

Bug 23761 – Some leaks

g_hash_table_get_values returns a newly-allocated GList. The callers
of devkit_disks_daemon_local_get_all_devices need to free it.

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

Bug 23762 – Fix typo

Bug 23764 – Small leak in update_drive_properties_from_sysfs

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