Skip to content
Snippets Groups Projects
Commit 949dd010 authored by Len Brown's avatar Len Brown
Browse files

powercap: restrict energy meter to root access


Remove non-privileged user access to power data contained in
/sys/class/powercap/intel-rapl*/*/energy_uj

Non-privileged users currently have read access to power data and can
use this data to form a security attack. Some privileged
drivers/applications need read access to this data, but don't expose it
to non-privileged users.

For example, thermald uses this data to ensure that power management
works correctly. Thus removing non-privileged access is preferred over
completely disabling this power reporting capability with
CONFIG_INTEL_RAPL=n.

Fixes: 95677a9a ("PowerCap: Fix mode for energy counter")

Signed-off-by: default avatarLen Brown <len.brown@intel.com>
Cc: stable@vger.kernel.org
parent 3d7772ea
No related branches found
No related tags found
No related merge requests found
...@@ -367,9 +367,9 @@ static void create_power_zone_common_attributes( ...@@ -367,9 +367,9 @@ static void create_power_zone_common_attributes(
&dev_attr_max_energy_range_uj.attr; &dev_attr_max_energy_range_uj.attr;
if (power_zone->ops->get_energy_uj) { if (power_zone->ops->get_energy_uj) {
if (power_zone->ops->reset_energy_uj) if (power_zone->ops->reset_energy_uj)
dev_attr_energy_uj.attr.mode = S_IWUSR | S_IRUGO; dev_attr_energy_uj.attr.mode = S_IWUSR | S_IRUSR;
else else
dev_attr_energy_uj.attr.mode = S_IRUGO; dev_attr_energy_uj.attr.mode = S_IRUSR;
power_zone->zone_dev_attrs[count++] = power_zone->zone_dev_attrs[count++] =
&dev_attr_energy_uj.attr; &dev_attr_energy_uj.attr;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment