summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJavi Merino <javi.merino@arm.com>2015-01-15 16:17:45 +0000
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2015-02-03 15:50:31 -0800
commitadf305f77878880fa5868a7179979da93be68d83 (patch)
treef87b173d3848b06bbe620142452803c64b78723c
parent68ff2a00dbf590dc9483c20f85fd1f1faa054604 (diff)
downloadlinux-stable-adf305f77878880fa5868a7179979da93be68d83.tar.gz
linux-stable-adf305f77878880fa5868a7179979da93be68d83.tar.bz2
linux-stable-adf305f77878880fa5868a7179979da93be68d83.zip
sysfs: fix warning when creating a sysfs group without attributes
When attempting to create a gropu without attrs, the warning prints the name of the group. However, the check for name being a NULL pointer is wrong: it uses the pointer to the name when it's NULL. Fix it to use the name if present, otherwise just put an empty string. Cc: Bruno Prémont <bonbons@linux-vserver.org> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Javi Merino <javi.merino@arm.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--fs/sysfs/group.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/sysfs/group.c b/fs/sysfs/group.c
index 7d2a860ba788..2554d8835b48 100644
--- a/fs/sysfs/group.c
+++ b/fs/sysfs/group.c
@@ -99,7 +99,7 @@ static int internal_create_group(struct kobject *kobj, int update,
return -EINVAL;
if (!grp->attrs && !grp->bin_attrs) {
WARN(1, "sysfs: (bin_)attrs not set by subsystem for group: %s/%s\n",
- kobj->name, grp->name ? "" : grp->name);
+ kobj->name, grp->name ?: "");
return -EINVAL;
}
if (grp->name) {