summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2007-07-31 00:38:16 -0700
committerWilly Tarreau <w@1wt.eu>2007-08-25 17:24:05 +0200
commit037f00bce0965f3291bf80595c6a0bcc33cca3a1 (patch)
treeab8d50a007dd4b5ed1352fe5dd78bec42917a2e6
parent219d43111f781f42116d2a9b3b0e7fa2336a8fba (diff)
downloadlwn-037f00bce0965f3291bf80595c6a0bcc33cca3a1.tar.gz
lwn-037f00bce0965f3291bf80595c6a0bcc33cca3a1.zip
[PATCH] CPU online file permission
Is there a reason why the "online" file in the subdirectories for the CPUs in /sys/devices/system isn't world-readable? I cannot imagine it to be security relevant especially now that a getcpu() syscall can be used to determine what CPUa thread runs on. The file is useful to correctly implement the sysconf() function to return the number of online CPUs. In the presence of hotplug we currently cannot provide this information. The patch below should to it. Signed-off-by: Ulrich Drepper <drepper@redhat.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> Signed-off-by: Willy Tarreau <w@1wt.eu>
-rw-r--r--drivers/base/cpu.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/base/cpu.c b/drivers/base/cpu.c
index 7fd095efaebd..be31b4358059 100644
--- a/drivers/base/cpu.c
+++ b/drivers/base/cpu.c
@@ -53,7 +53,7 @@ static ssize_t store_online(struct sys_device *dev, const char *buf,
ret = count;
return ret;
}
-static SYSDEV_ATTR(online, 0600, show_online, store_online);
+static SYSDEV_ATTR(online, 0644, show_online, store_online);
static void __devinit register_cpu_control(struct cpu *cpu)
{