[lvm-devel] [PATCH 8/7][retry remove] Also add check for filesystem use on a DM/LVM device - "dm_device_has_fs"
Zdenek Kabelac
zkabelac at redhat.com
Wed Sep 21 11:39:47 UTC 2011
Dne 21.9.2011 10:39, Peter Rajnoha napsal(a):
> On 09/20/2011 09:15 PM +0100, Peter Rajnoha wrote:
>> diff --git a/libdm/libdm-common.c b/libdm/libdm-common.c
>> index 7ea78af..37b6902 100644
>> --- a/libdm/libdm-common.c
>> +++ b/libdm/libdm-common.c
>
>> +int dm_device_has_fs(uint32_t major, uint32_t minor)
>> +{
>> + char sysfs_path[PATH_MAX];
>> + char temp_path[PATH_MAX];
>> + size_t size;
>
> I've just noticed - this must be int size!!!
>
>> + char *kernel_dev_name;
>> +
>> + /* Get kernel device name first */
>> + if (dm_snprintf(sysfs_path, PATH_MAX, "%sdev/block/%" PRIu32 ":%" PRIu32,
>> + _sysfs_dir, major, minor) < 0) {
>> + log_error("sysfs_path dm_snprintf failed");
>> + return 0;
>> + }
>> +
>> + if ((size = readlink(sysfs_path, temp_path, PATH_MAX)) < 0) {
>
> ...because of this check.
>
Or even better use types from declaration:
ssize_t readlink(const char *path, char *buf, size_t bufsiz);
Zdenek
More information about the lvm-devel
mailing list