summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStanislav Brabec2015-05-25 18:21:36 +0200
committerKarel Zak2015-05-26 11:42:38 +0200
commitd0dc6c17c5025639a7beef8d52792932fd8da0fc (patch)
tree8d2e8ae96e6cf1447fb790851baa3769d02a1738
parenthwclock: don't allow non-root access (diff)
downloadkernel-qcow2-util-linux-d0dc6c17c5025639a7beef8d52792932fd8da0fc.tar.gz
kernel-qcow2-util-linux-d0dc6c17c5025639a7beef8d52792932fd8da0fc.tar.xz
kernel-qcow2-util-linux-d0dc6c17c5025639a7beef8d52792932fd8da0fc.zip
Fix /sys to /dev node name translation
linux/drivers/base/core.c: device_get_devnode() defines a translation of '!' in sysfs nodes to '/' in /dev nodes. The same translation has to be done to properly support device nodes with slash (e. g. device nodes of cciss driver and several other drivers). Introduce new helper sysfs_devname_to_devno() and use it where appropriate. Fixes for example lsblk -f on devices using cciss driver. Signed-off-by: Stanislav Brabec <sbrabec@suse.cz>
-rw-r--r--include/sysfs.h18
-rw-r--r--lib/sysfs.c2
-rw-r--r--libblkid/src/devno.c1
-rw-r--r--misc-utils/lsblk.c1
4 files changed, 22 insertions, 0 deletions
diff --git a/include/sysfs.h b/include/sysfs.h
index 1de624aad..4564124df 100644
--- a/include/sysfs.h
+++ b/include/sysfs.h
@@ -91,4 +91,22 @@ extern int sysfs_scsi_host_is(struct sysfs_cxt *cxt, const char *type);
extern int sysfs_scsi_has_attribute(struct sysfs_cxt *cxt, const char *attr);
extern int sysfs_scsi_path_contains(struct sysfs_cxt *cxt, const char *pattern);
+/**
+ * sysfs_devname_to_dev_name:
+ * @name: devname to be converted in place
+ *
+ * Linux kernel linux/drivers/base/core.c: device_get_devnode()
+ * defines a replacement of '!' in the /sys device name by '/' in the
+ * /dev device name. This helper replaces all ocurrences of '!' in
+ * @name by '/'.
+ */
+static inline void sysfs_devname_to_dev_name (char *name)
+{
+ char *c;
+
+ if (name)
+ while ((c = strchr(name, '!')))
+ c[0] = '/';
+}
+
#endif /* UTIL_LINUX_SYSFS_H */
diff --git a/lib/sysfs.c b/lib/sysfs.c
index 759d97be6..8417d2d76 100644
--- a/lib/sysfs.c
+++ b/lib/sysfs.c
@@ -131,6 +131,7 @@ char *sysfs_devno_to_devpath(dev_t devno, char *buf, size_t bufsiz)
return NULL;
/* create the final "/dev/<name>" string */
+ sysfs_devname_to_dev_name(name);
memmove(buf + 5, name, sz + 1);
memcpy(buf, "/dev/", 5);
@@ -789,6 +790,7 @@ int sysfs_devno_to_wholedisk(dev_t dev, char *diskname,
if (!name)
goto err;
+ sysfs_devname_to_dev_name(name);
if (diskname && len) {
strncpy(diskname, name, len);
diskname[len - 1] = '\0';
diff --git a/libblkid/src/devno.c b/libblkid/src/devno.c
index f4a36e4f5..3c082271b 100644
--- a/libblkid/src/devno.c
+++ b/libblkid/src/devno.c
@@ -208,6 +208,7 @@ static char *scandev_devno_to_devpath(dev_t devno)
new_list = NULL;
}
}
+ sysfs_devname_to_dev_name(devname);
free_dirlist(&list);
free_dirlist(&new_list);
diff --git a/misc-utils/lsblk.c b/misc-utils/lsblk.c
index 1b4ffc128..d826c778f 100644
--- a/misc-utils/lsblk.c
+++ b/misc-utils/lsblk.c
@@ -421,6 +421,7 @@ static char *get_device_path(struct blkdev_cxt *cxt)
return canonicalize_dm_name(cxt->name);
snprintf(path, sizeof(path), "/dev/%s", cxt->name);
+ sysfs_devname_to_dev_name(path);
return xstrdup(path);
}