Messages in this thread Patch in this message | | | From | Felipe Balbi <> | Subject | [PATCH] base: core: WARN() about bogus permissions on device attributes | Date | Wed, 20 Feb 2013 10:31:42 +0200 |
| |
Whenever a struct device_attribute is registered with mismatched permissions - read permission without a show routine or write permission without store routine - we will issue a big warning so we catch those early enough.
Signed-off-by: Felipe Balbi <balbi@ti.com> --- drivers/base/core.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/base/core.c b/drivers/base/core.c index a235085..14e6a92 100644 --- a/drivers/base/core.c +++ b/drivers/base/core.c @@ -563,8 +563,15 @@ int device_create_file(struct device *dev, const struct device_attribute *attr) { int error = 0; - if (dev) + + if (dev) { + WARN(((attr->attr.mode & S_IWUGO) && !attr->store), + "Write permission without 'store'\n"); + WARN(((attr->attr.mode & S_IRUGO) && !attr->show), + "Read permission without 'show'\n"); error = sysfs_create_file(&dev->kobj, &attr->attr); + } + return error; } -- 1.8.1.rc1.5.g7e0651a
| |